[jira] [Commented] (THRIFT-1680) make install requires GNU make

2012-09-28 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/THRIFT-1680?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13466071#comment-13466071
 ] 

Hudson commented on THRIFT-1680:


Integrated in Thrift #540 (See [https://builds.apache.org/job/Thrift/540/])
Thrift-1680:Make install requires GNU make
Client: build
patch: Jake Farrell

Updates boost and libevent configure messages, updates thrift help to display 
error and smaller message, moves help to --help or -help and takes care of // 
TODO(dreiss): Delete these when everyone is using the new hotness. Welcome to 
the new hotness everyone. (Revision 1391705)

 Result = FAILURE

> make install requires GNU make
> --
>
> Key: THRIFT-1680
> URL: https://issues.apache.org/jira/browse/THRIFT-1680
> Project: Thrift
>  Issue Type: Bug
>  Components: C++ - Compiler
>Affects Versions: 0.8
> Environment: NetBSD
>Reporter: James K. Lowden
>Assignee: Jake Farrell
>Priority: Minor
>  Labels: bsdmake, gmake, make
> Attachments: Thrift-1680.patch
>
>   Original Estimate: 3h
>  Remaining Estimate: 3h
>
> test/cpp/Makefile.am includes targets whose dependency rule relies on the 
> variable "$<" to pick up the first dependency.  This variable is specific to 
> GNU make; it is not supported by most BSD make programs.  AFAICT this is the 
> only such dependency on GNU make.  I was able to build thrift using BSD make; 
> I was prevented only from installing it (because install invokes tests).  
> BSD make has no analog for $<.  The simplest fix I know of is to repeat the 
> name of the first dependency the command.  The second easiest fix would be to 
> document the limitation.  ;-)  
> Three other suggestions, if I may: 
> 1.  It would be nice if the DIR for --with-boost=DIR were more clearly 
> specified.  It's not obvious that --with-boost=/usr/pkg is what's needed if 
> the Boost headers are in /usr/pkg/include/boost.  
> 2.  The defaults indicated by configure --help are IMO misleading because 
> they're the defaults *if* configure detects them.  I would prefer all default 
> to No, and for configure to stop with an error if --with-foo is specified and 
> fails.  It is otherwise exceedingly difficult to detect automatically whether 
> the specified configuration was in fact installed.  
> 3.  The thrift error message is ridiculous:
> $ thrift --foo 2>&1 | wc -l
>   78
> The first line "No output language(s) specified" (less the "!!!", please) 
> suffices to indicate the syntax error.  The user can more easily consult the 
> manual to learn how to use the program than read 78 lines of output on his 
> terminal.  The "help" obscures the error.  
> Thanks.  

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (THRIFT-1680) make install requires GNU make

2012-09-28 Thread Jake Farrell (JIRA)

[ 
https://issues.apache.org/jira/browse/THRIFT-1680?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13466047#comment-13466047
 ] 

Jake Farrell commented on THRIFT-1680:
--

initial patch committed fixing sub-issues. 

> make install requires GNU make
> --
>
> Key: THRIFT-1680
> URL: https://issues.apache.org/jira/browse/THRIFT-1680
> Project: Thrift
>  Issue Type: Bug
>  Components: C++ - Compiler
>Affects Versions: 0.8
> Environment: NetBSD
>Reporter: James K. Lowden
>Assignee: Jake Farrell
>Priority: Minor
>  Labels: bsdmake, gmake, make
> Attachments: Thrift-1680.patch
>
>   Original Estimate: 3h
>  Remaining Estimate: 3h
>
> test/cpp/Makefile.am includes targets whose dependency rule relies on the 
> variable "$<" to pick up the first dependency.  This variable is specific to 
> GNU make; it is not supported by most BSD make programs.  AFAICT this is the 
> only such dependency on GNU make.  I was able to build thrift using BSD make; 
> I was prevented only from installing it (because install invokes tests).  
> BSD make has no analog for $<.  The simplest fix I know of is to repeat the 
> name of the first dependency the command.  The second easiest fix would be to 
> document the limitation.  ;-)  
> Three other suggestions, if I may: 
> 1.  It would be nice if the DIR for --with-boost=DIR were more clearly 
> specified.  It's not obvious that --with-boost=/usr/pkg is what's needed if 
> the Boost headers are in /usr/pkg/include/boost.  
> 2.  The defaults indicated by configure --help are IMO misleading because 
> they're the defaults *if* configure detects them.  I would prefer all default 
> to No, and for configure to stop with an error if --with-foo is specified and 
> fails.  It is otherwise exceedingly difficult to detect automatically whether 
> the specified configuration was in fact installed.  
> 3.  The thrift error message is ridiculous:
> $ thrift --foo 2>&1 | wc -l
>   78
> The first line "No output language(s) specified" (less the "!!!", please) 
> suffices to indicate the syntax error.  The user can more easily consult the 
> manual to learn how to use the program than read 78 lines of output on his 
> terminal.  The "help" obscures the error.  
> Thanks.  

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (THRIFT-1680) make install requires GNU make

2012-09-28 Thread Jake Farrell (JIRA)

[ 
https://issues.apache.org/jira/browse/THRIFT-1680?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13466044#comment-13466044
 ] 

Jake Farrell commented on THRIFT-1680:
--

Having a mix of on/off would complicate things unnecessarily, and how do we 
decide what is production ready (os x vs ubuntu vs ...). I prefer the all on 
and disable any that can not be build due to lacking dependencies or are 
specified using a --without flag. An option to only build the compiler would be 
a nice addition though, --compiler-only perhaps 

> make install requires GNU make
> --
>
> Key: THRIFT-1680
> URL: https://issues.apache.org/jira/browse/THRIFT-1680
> Project: Thrift
>  Issue Type: Bug
>  Components: C++ - Compiler
>Affects Versions: 0.8
> Environment: NetBSD
>Reporter: James K. Lowden
>Assignee: Jake Farrell
>Priority: Minor
>  Labels: bsdmake, gmake, make
> Attachments: Thrift-1680.patch
>
>   Original Estimate: 3h
>  Remaining Estimate: 3h
>
> test/cpp/Makefile.am includes targets whose dependency rule relies on the 
> variable "$<" to pick up the first dependency.  This variable is specific to 
> GNU make; it is not supported by most BSD make programs.  AFAICT this is the 
> only such dependency on GNU make.  I was able to build thrift using BSD make; 
> I was prevented only from installing it (because install invokes tests).  
> BSD make has no analog for $<.  The simplest fix I know of is to repeat the 
> name of the first dependency the command.  The second easiest fix would be to 
> document the limitation.  ;-)  
> Three other suggestions, if I may: 
> 1.  It would be nice if the DIR for --with-boost=DIR were more clearly 
> specified.  It's not obvious that --with-boost=/usr/pkg is what's needed if 
> the Boost headers are in /usr/pkg/include/boost.  
> 2.  The defaults indicated by configure --help are IMO misleading because 
> they're the defaults *if* configure detects them.  I would prefer all default 
> to No, and for configure to stop with an error if --with-foo is specified and 
> fails.  It is otherwise exceedingly difficult to detect automatically whether 
> the specified configuration was in fact installed.  
> 3.  The thrift error message is ridiculous:
> $ thrift --foo 2>&1 | wc -l
>   78
> The first line "No output language(s) specified" (less the "!!!", please) 
> suffices to indicate the syntax error.  The user can more easily consult the 
> manual to learn how to use the program than read 78 lines of output on his 
> terminal.  The "help" obscures the error.  
> Thanks.  

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (THRIFT-1680) make install requires GNU make

2012-09-28 Thread Roger Meier (JIRA)

[ 
https://issues.apache.org/jira/browse/THRIFT-1680?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13465800#comment-13465800
 ] 

Roger Meier commented on THRIFT-1680:
-

+1 for this patch Jake!

another thing I had in mind is a disable all languages within the configure.ac
or just enable production ready languages by default?

> make install requires GNU make
> --
>
> Key: THRIFT-1680
> URL: https://issues.apache.org/jira/browse/THRIFT-1680
> Project: Thrift
>  Issue Type: Bug
>  Components: C++ - Compiler
>Affects Versions: 0.8
> Environment: NetBSD
>Reporter: James K. Lowden
>Assignee: Jake Farrell
>Priority: Minor
>  Labels: bsdmake, gmake, make
> Attachments: Thrift-1680.patch
>
>   Original Estimate: 3h
>  Remaining Estimate: 3h
>
> test/cpp/Makefile.am includes targets whose dependency rule relies on the 
> variable "$<" to pick up the first dependency.  This variable is specific to 
> GNU make; it is not supported by most BSD make programs.  AFAICT this is the 
> only such dependency on GNU make.  I was able to build thrift using BSD make; 
> I was prevented only from installing it (because install invokes tests).  
> BSD make has no analog for $<.  The simplest fix I know of is to repeat the 
> name of the first dependency the command.  The second easiest fix would be to 
> document the limitation.  ;-)  
> Three other suggestions, if I may: 
> 1.  It would be nice if the DIR for --with-boost=DIR were more clearly 
> specified.  It's not obvious that --with-boost=/usr/pkg is what's needed if 
> the Boost headers are in /usr/pkg/include/boost.  
> 2.  The defaults indicated by configure --help are IMO misleading because 
> they're the defaults *if* configure detects them.  I would prefer all default 
> to No, and for configure to stop with an error if --with-foo is specified and 
> fails.  It is otherwise exceedingly difficult to detect automatically whether 
> the specified configuration was in fact installed.  
> 3.  The thrift error message is ridiculous:
> $ thrift --foo 2>&1 | wc -l
>   78
> The first line "No output language(s) specified" (less the "!!!", please) 
> suffices to indicate the syntax error.  The user can more easily consult the 
> manual to learn how to use the program than read 78 lines of output on his 
> terminal.  The "help" obscures the error.  
> Thanks.  

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira