[jira] [Commented] (THRIFT-1680) make install requires GNU make
[ 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
[ 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
[ 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
[ 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