Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Plain View
Sqoop >> mail # dev >> Review Request: SQOOP-949: Allow Sqoop to build on Windows OS


+
Ahmed El Baz 2013-03-15, 19:00
+
Venkat Ranganathan 2013-03-15, 19:15
+
Ahmed El Baz 2013-03-15, 19:42
+
Venkat Ranganathan 2013-03-15, 20:40
+
Ahmed El Baz 2013-03-15, 21:41
+
Venkat Ranganathan 2013-03-15, 23:53
Copy link to this message
-
Re: Review Request: SQOOP-949: Allow Sqoop to build on Windows OS

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/9961/#review17991
-----------------------------------------------------------
Hi Ahmed,
thank you very much for all your work to get Sqoop compiled on Windows. I'm wondering whether there is need to document any extra steps or limitations?
build.xml
<https://reviews.apache.org/r/9961/#comment38008>

    Is there a need to rename this?

src/scripts/create-tool-scripts.cmd
<https://reviews.apache.org/r/9961/#comment38006>

    Nit: Trailing white space characters

src/scripts/tool-script.cmd.template
<https://reviews.apache.org/r/9961/#comment38007>

    I've missed the place where the sqoop.cmd gets generated, would you mind pointing me?
Jarcec

- Jarek Cecho
On March 15, 2013, 7 p.m., Ahmed El Baz wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/9961/
> -----------------------------------------------------------
>
> (Updated March 15, 2013, 7 p.m.)
>
>
> Review request for Sqoop.
>
>
> Description
> -------
>
> Allow Sqoop to build on Windows OS
> Associated Jira: https://issues.apache.org/jira/browse/SQOOP-949
>
> Sqoop fails to build on Windows due to dependencies on Linux scripts like write-version-info.sh and create-tool-scripts.sh which fail to build on Windows without Cygwin.
>
> The patch does the following:
> - provide Windows scripts src/scripts/create-tool-scripts.cmd and src/scripts/write-version-info.cmd which perform the same logic as the .sh scripts on Linux.
> - Update build.xml to invoke the appropriate script based on the host OS.
> - Skip real-docs target on Windows OS builds since as it invokes Linux specific "make" executable
>
>
> This addresses bug SQOOP-949.
>     https://issues.apache.org/jira/browse/SQOOP-949
>
>
> Diffs
> -----
>
>   build.xml c097fe8
>   src/scripts/create-tool-scripts.cmd PRE-CREATION
>   src/scripts/tool-script.cmd.template PRE-CREATION
>   src/scripts/write-version-info.cmd PRE-CREATION
>
> Diff: https://reviews.apache.org/r/9961/diff/
>
>
> Testing
> -------
>
> Verified builds are successful on both Linux and Windows, and Unit tests are fully passing on Linux. Fixes to unit tests on Windows will go in a separate patch
>
>
> Thanks,
>
> Ahmed El Baz
>
>

+
Ahmed El Baz 2013-03-15, 20:26
+
Venkat Ranganathan 2013-03-22, 06:15
+
Venkat Ranganathan 2013-04-06, 00:39