Re: Review Request 36910: Patch configure.ac to include $LIBS in the CRAM-MD5 check
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36910/#review94252 --- Ship it! Ship It! - Till Toenshoff On Aug. 4, 2015, 1:49 p.m., Chris Heller wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36910/ > --- > > (Updated Aug. 4, 2015, 1:49 p.m.) > > > Review request for mesos and Till Toenshoff. > > > Bugs: MESOS-3170 > https://issues.apache.org/jira/browse/MESOS-3170 > > > Repository: mesos > > > Description > --- > > [MESOS-3170] Add $LIBS to build path of the CRAM-MD5 test > > > Diffs > - > > configure.ac 9438d88 > > Diff: https://reviews.apache.org/r/36910/diff/ > > > Testing > --- > > See https://github.com/apache/mesos/pull/51 > > Verified build against statically linked OpenSSL 1.0.1e and Cyrus-SASL 2.1.26 > > > Thanks, > > Chris Heller > >
Re: Review Request 36910: Patch configure.ac to include $LIBS in the CRAM-MD5 check
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36910/#review94121 --- Patch looks great! Reviews applied: [36910] All tests passed. - Mesos ReviewBot On Aug. 4, 2015, 1:49 p.m., Chris Heller wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36910/ > --- > > (Updated Aug. 4, 2015, 1:49 p.m.) > > > Review request for mesos and Till Toenshoff. > > > Bugs: MESOS-3170 > https://issues.apache.org/jira/browse/MESOS-3170 > > > Repository: mesos > > > Description > --- > > [MESOS-3170] Add $LIBS to build path of the CRAM-MD5 test > > > Diffs > - > > configure.ac 9438d88 > > Diff: https://reviews.apache.org/r/36910/diff/ > > > Testing > --- > > See https://github.com/apache/mesos/pull/51 > > Verified build against statically linked OpenSSL 1.0.1e and Cyrus-SASL 2.1.26 > > > Thanks, > > Chris Heller > >
Re: Review Request 36910: Patch configure.ac to include $LIBS in the CRAM-MD5 check
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36910/ --- (Updated Aug. 4, 2015, 1:49 p.m.) Review request for mesos and Till Toenshoff. Changes --- Rebased Bugs: MESOS-3170 https://issues.apache.org/jira/browse/MESOS-3170 Repository: mesos Description --- [MESOS-3170] Add $LIBS to build path of the CRAM-MD5 test Diffs (updated) - configure.ac 9438d88 Diff: https://reviews.apache.org/r/36910/diff/ Testing --- See https://github.com/apache/mesos/pull/51 Verified build against statically linked OpenSSL 1.0.1e and Cyrus-SASL 2.1.26 Thanks, Chris Heller
Re: Review Request 36910: Patch configure.ac to include $LIBS in the CRAM-MD5 check
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36910/#review94054 --- Re-based the PR - Chris Heller On July 29, 2015, 1:41 p.m., Chris Heller wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36910/ > --- > > (Updated July 29, 2015, 1:41 p.m.) > > > Review request for mesos and Till Toenshoff. > > > Bugs: MESOS-3170 > https://issues.apache.org/jira/browse/MESOS-3170 > > > Repository: mesos > > > Description > --- > > [MESOS-3170] Add $LIBS to build path of the CRAM-MD5 test > > > Diffs > - > > b/configure.ac 546c9bbf775a4ef481fafb3a58c85c6d80e19500 > > Diff: https://reviews.apache.org/r/36910/diff/ > > > Testing > --- > > See https://github.com/apache/mesos/pull/51 > > Verified build against statically linked OpenSSL 1.0.1e and Cyrus-SASL 2.1.26 > > > Thanks, > > Chris Heller > >
Re: Review Request 36910: Patch configure.ac to include $LIBS in the CRAM-MD5 check
> On July 30, 2015, 12:28 p.m., Chris Heller wrote: > > It's unclear what failed in that auto build. It appears unrelated > > (potentially). As a test I pulled my branch, and rebased from master, then > > configured a build and ran `make -j3 distcheck` and was successful in > > building. Can the auto check be re-run? nothing wrong with your patch. there was another problem on the master branch that we pushed a fix for. if you can rebase off the latest master and update this review, the bot will run again. - Vinod --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36910/#review93579 --- On July 29, 2015, 1:41 p.m., Chris Heller wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36910/ > --- > > (Updated July 29, 2015, 1:41 p.m.) > > > Review request for mesos and Till Toenshoff. > > > Bugs: MESOS-3170 > https://issues.apache.org/jira/browse/MESOS-3170 > > > Repository: mesos > > > Description > --- > > [MESOS-3170] Add $LIBS to build path of the CRAM-MD5 test > > > Diffs > - > > b/configure.ac 546c9bbf775a4ef481fafb3a58c85c6d80e19500 > > Diff: https://reviews.apache.org/r/36910/diff/ > > > Testing > --- > > See https://github.com/apache/mesos/pull/51 > > Verified build against statically linked OpenSSL 1.0.1e and Cyrus-SASL 2.1.26 > > > Thanks, > > Chris Heller > >
Re: Review Request 36910: Patch configure.ac to include $LIBS in the CRAM-MD5 check
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36910/#review93579 --- It's unclear what failed in that auto build. It appears unrelated (potentially). As a test I pulled my branch, and rebased from master, then configured a build and ran `make -j3 distcheck` and was successful in building. Can the auto check be re-run? - Chris Heller On July 29, 2015, 1:41 p.m., Chris Heller wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36910/ > --- > > (Updated July 29, 2015, 1:41 p.m.) > > > Review request for mesos and Till Toenshoff. > > > Bugs: MESOS-3170 > https://issues.apache.org/jira/browse/MESOS-3170 > > > Repository: mesos > > > Description > --- > > [MESOS-3170] Add $LIBS to build path of the CRAM-MD5 test > > > Diffs > - > > b/configure.ac 546c9bbf775a4ef481fafb3a58c85c6d80e19500 > > Diff: https://reviews.apache.org/r/36910/diff/ > > > Testing > --- > > See https://github.com/apache/mesos/pull/51 > > Verified build against statically linked OpenSSL 1.0.1e and Cyrus-SASL 2.1.26 > > > Thanks, > > Chris Heller > >
Re: Review Request 36910: Patch configure.ac to include $LIBS in the CRAM-MD5 check
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36910/ --- (Updated July 29, 2015, 1:41 p.m.) Review request for mesos and Till Toenshoff. Changes --- Added JIRA issue. Bugs: MESOS-3170 https://issues.apache.org/jira/browse/MESOS-3170 Repository: mesos Description --- [MESOS-3170] Add $LIBS to build path of the CRAM-MD5 test Diffs - b/configure.ac 546c9bbf775a4ef481fafb3a58c85c6d80e19500 Diff: https://reviews.apache.org/r/36910/diff/ Testing --- See https://github.com/apache/mesos/pull/51 Verified build against statically linked OpenSSL 1.0.1e and Cyrus-SASL 2.1.26 Thanks, Chris Heller
Review Request 36910: Patch configure.ac to include $LIBS in the CRAM-MD5 check
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36910/ --- Review request for mesos. Repository: mesos Description --- [MESOS-3170] Add $LIBS to build path of the CRAM-MD5 test Diffs - b/configure.ac 546c9bbf775a4ef481fafb3a58c85c6d80e19500 Diff: https://reviews.apache.org/r/36910/diff/ Testing --- See https://github.com/apache/mesos/pull/51 Verified build against statically linked OpenSSL 1.0.1e and Cyrus-SASL 2.1.26 Thanks, Chris Heller