You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@mesos.apache.org by Vinod Kone <vi...@gmail.com> on 2014/07/17 23:18:27 UTC
Re: Review Request 23086: Update system check to unbundle protobuf
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/23086/#review48049
-----------------------------------------------------------
configure.ac
<https://reviews.apache.org/r/23086/#comment84305>
end with period.
configure.ac
<https://reviews.apache.org/r/23086/#comment84336>
would PKG_CHECK_MODULES work if protobuf is installed in a non-standard location that user specified with --with-protobuf=<dir>?
mpi/mpiexec-mesos.in
<https://reviews.apache.org/r/23086/#comment84321>
period at the end.
src/examples/java/test-exception-framework.in
<https://reviews.apache.org/r/23086/#comment84333>
What happens when the user gives --with-protobuf=/path/to/non-standard/location?
is the jar still expected in /usr/share/java?
is it possible to plumb through the user provided protobuf path to these scripts? that seems better than silently trying to pick the jar from a standard location?
src/examples/python/test-framework.in
<https://reviews.apache.org/r/23086/#comment84335>
ditto. saying "failed" and proceeding seems weird. is the egg expected to be on the python path? if yes, we should at least tell the users so?
- Vinod Kone
On June 26, 2014, 7:31 p.m., Timothy St. Clair wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/23086/
> -----------------------------------------------------------
>
> (Updated June 26, 2014, 7:31 p.m.)
>
>
> Review request for mesos and Benjamin Hindman.
>
>
> Bugs: MESOS-1174
> https://issues.apache.org/jira/browse/MESOS-1174
>
>
> Repository: mesos-git
>
>
> Description
> -------
>
> Enable mesos to build against system installed version of protobuf.
>
>
> Diffs
> -----
>
> 3rdparty/libprocess/3rdparty/Makefile.am 429c956
> 3rdparty/libprocess/configure.ac 27c6ede
> configure.ac 41f3c2b
> mpi/mpiexec-mesos.in da0733f
> src/Makefile.am 3c8b91a
> src/examples/java/test-exception-framework.in 065f3c0
> src/examples/java/test-executor.in 35559d2
> src/examples/java/test-framework.in 9af48f9
> src/examples/java/test-log.in 8afed0d
> src/examples/java/test-multiple-executors-framework.in 235eb23
> src/examples/python/test-containerizer.in 569519b
> src/examples/python/test-executor.in 7e8875f
> src/examples/python/test-framework.in c4683b9
> src/python/setup.py.in b996dfe
>
> Diff: https://reviews.apache.org/r/23086/diff/
>
>
> Testing
> -------
>
> make check
>
>
> Thanks,
>
> Timothy St. Clair
>
>
Re: Review Request 23086: Update system check to unbundle protobuf
Posted by "Timothy St. Clair" <ts...@redhat.com>.
> On July 17, 2014, 9:18 p.m., Vinod Kone wrote:
> > configure.ac, line 315
> > <https://reviews.apache.org/r/23086/diff/1/?file=618608#file618608line315>
> >
> > would PKG_CHECK_MODULES work if protobuf is installed in a non-standard location that user specified with --with-protobuf=<dir>?
Added the check above on prefix.
> On July 17, 2014, 9:18 p.m., Vinod Kone wrote:
> > src/examples/java/test-exception-framework.in, line 18
> > <https://reviews.apache.org/r/23086/diff/1/?file=618611#file618611line18>
> >
> > What happens when the user gives --with-protobuf=/path/to/non-standard/location?
> >
> > is the jar still expected in /usr/share/java?
> >
> > is it possible to plumb through the user provided protobuf path to these scripts? that seems better than silently trying to pick the jar from a standard location?
Overloaded to use AC_SUBST
> On July 17, 2014, 9:18 p.m., Vinod Kone wrote:
> > src/examples/python/test-framework.in, line 31
> > <https://reviews.apache.org/r/23086/diff/1/?file=618618#file618618line31>
> >
> > ditto. saying "failed" and proceeding seems weird. is the egg expected to be on the python path? if yes, we should at least tell the users so?
This actually doesn't matter so much as they user can adjust their PYTHONPATH. For 99% of cases where folks will use the system installation of protobuf, this is a non-issue.
- Timothy
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/23086/#review48049
-----------------------------------------------------------
On June 26, 2014, 7:31 p.m., Timothy St. Clair wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/23086/
> -----------------------------------------------------------
>
> (Updated June 26, 2014, 7:31 p.m.)
>
>
> Review request for mesos and Benjamin Hindman.
>
>
> Bugs: MESOS-1174
> https://issues.apache.org/jira/browse/MESOS-1174
>
>
> Repository: mesos-git
>
>
> Description
> -------
>
> Enable mesos to build against system installed version of protobuf.
>
>
> Diffs
> -----
>
> 3rdparty/libprocess/3rdparty/Makefile.am 429c956
> 3rdparty/libprocess/configure.ac 27c6ede
> configure.ac 41f3c2b
> mpi/mpiexec-mesos.in da0733f
> src/Makefile.am 3c8b91a
> src/examples/java/test-exception-framework.in 065f3c0
> src/examples/java/test-executor.in 35559d2
> src/examples/java/test-framework.in 9af48f9
> src/examples/java/test-log.in 8afed0d
> src/examples/java/test-multiple-executors-framework.in 235eb23
> src/examples/python/test-containerizer.in 569519b
> src/examples/python/test-executor.in 7e8875f
> src/examples/python/test-framework.in c4683b9
> src/python/setup.py.in b996dfe
>
> Diff: https://reviews.apache.org/r/23086/diff/
>
>
> Testing
> -------
>
> make check
>
>
> Thanks,
>
> Timothy St. Clair
>
>
Re: Review Request 23086: Update system check to unbundle protobuf
Posted by "Timothy St. Clair" <ts...@redhat.com>.
> On July 17, 2014, 9:18 p.m., Vinod Kone wrote:
> > src/examples/java/test-exception-framework.in, line 18
> > <https://reviews.apache.org/r/23086/diff/1/?file=618611#file618611line18>
> >
> > What happens when the user gives --with-protobuf=/path/to/non-standard/location?
> >
> > is the jar still expected in /usr/share/java?
> >
> > is it possible to plumb through the user provided protobuf path to these scripts? that seems better than silently trying to pick the jar from a standard location?
>
> Timothy St. Clair wrote:
> Overloaded to use AC_SUBST
The logic for this is weird due to: https://lists.gnu.org/archive/html/bug-autoconf/2013-02/msg00008.html
- Timothy
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/23086/#review48049
-----------------------------------------------------------
On June 26, 2014, 7:31 p.m., Timothy St. Clair wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/23086/
> -----------------------------------------------------------
>
> (Updated June 26, 2014, 7:31 p.m.)
>
>
> Review request for mesos and Benjamin Hindman.
>
>
> Bugs: MESOS-1174
> https://issues.apache.org/jira/browse/MESOS-1174
>
>
> Repository: mesos-git
>
>
> Description
> -------
>
> Enable mesos to build against system installed version of protobuf.
>
>
> Diffs
> -----
>
> 3rdparty/libprocess/3rdparty/Makefile.am 429c956
> 3rdparty/libprocess/configure.ac 27c6ede
> configure.ac 41f3c2b
> mpi/mpiexec-mesos.in da0733f
> src/Makefile.am 3c8b91a
> src/examples/java/test-exception-framework.in 065f3c0
> src/examples/java/test-executor.in 35559d2
> src/examples/java/test-framework.in 9af48f9
> src/examples/java/test-log.in 8afed0d
> src/examples/java/test-multiple-executors-framework.in 235eb23
> src/examples/python/test-containerizer.in 569519b
> src/examples/python/test-executor.in 7e8875f
> src/examples/python/test-framework.in c4683b9
> src/python/setup.py.in b996dfe
>
> Diff: https://reviews.apache.org/r/23086/diff/
>
>
> Testing
> -------
>
> make check
>
>
> Thanks,
>
> Timothy St. Clair
>
>