> On 2012-01-30 04:39:53, Charles Reiss wrote:
> > src/Makefile.am, line 57
> > <https://reviews.apache.org/r/3672/diff/1/?file=71186#file71186line57>
> >
> >     Since nobase_* is used here, the scripts are already installed under 
> > $(pkgdatadir)/webui (as you can test by running 'make install 
> > DESTDIR=/tmp/some-path'). Is it your intention to install them in 
> > $(pkgdatadir)/webui/webui ?
> 
> Benjamin Hindman wrote:
>     This doesn't appear to be the case. After first doing a configure with 
> --prefix=/usr/local an then a 'make install DESTDIR=/home/benh/tmp' I get a 
> directory layout like this:
>     
>     [benh@hostname ~]$ find /home/benh/tmp/usr/local
>     /home/benh/tmp/usr/local
>     /home/benh/tmp/usr/local/lib
>     /home/benh/tmp/usr/local/lib/java
>     /home/benh/tmp/usr/local/lib/java/mesos.jar
>     /home/benh/tmp/usr/local/lib/python2.6
>     /home/benh/tmp/usr/local/lib/python2.6/site-packages
>     
> /home/benh/tmp/usr/local/lib/python2.6/site-packages/mesos-0.0.1-py2.6-linux-x86_64.egg
>     /home/benh/tmp/usr/local/lib/libmesos.la
>     /home/benh/tmp/usr/local/lib/libmesos.so
>     /home/benh/tmp/usr/local/lib/libmesos.so.0.0.0
>     /home/benh/tmp/usr/local/lib/libmesos.so.0
>     /home/benh/tmp/usr/local/bin
>     /home/benh/tmp/usr/local/bin/mesos-log
>     /home/benh/tmp/usr/local/bin/mesos-mesos
>     /home/benh/tmp/usr/local/bin/mesos-local
>     /home/benh/tmp/usr/local/libexec
>     /home/benh/tmp/usr/local/libexec/mesos
>     /home/benh/tmp/usr/local/libexec/mesos/mesos-launcher
>     /home/benh/tmp/usr/local/libexec/mesos/killtree.sh
>     /home/benh/tmp/usr/local/include
>     /home/benh/tmp/usr/local/include/mesos
>     /home/benh/tmp/usr/local/include/mesos/mesos.proto
>     /home/benh/tmp/usr/local/include/mesos/mesos.pb.h
>     /home/benh/tmp/usr/local/include/mesos/mesos.hpp
>     /home/benh/tmp/usr/local/include/mesos/scheduler.hpp
>     /home/benh/tmp/usr/local/include/mesos/executor.hpp
>     /home/benh/tmp/usr/local/share
>     /home/benh/tmp/usr/local/share/mesos
>     /home/benh/tmp/usr/local/share/mesos/webui
>     /home/benh/tmp/usr/local/share/mesos/webui/common
>     /home/benh/tmp/usr/local/share/mesos/webui/common/webui_lib.py
>     /home/benh/tmp/usr/local/share/mesos/webui/slave
>     /home/benh/tmp/usr/local/share/mesos/webui/slave/index.tpl
>     /home/benh/tmp/usr/local/share/mesos/webui/slave/executor.tpl
>     /home/benh/tmp/usr/local/share/mesos/webui/slave/framework.tpl
>     /home/benh/tmp/usr/local/share/mesos/webui/slave/webui.py
>     /home/benh/tmp/usr/local/share/mesos/webui/master
>     /home/benh/tmp/usr/local/share/mesos/webui/master/index.tpl
>     /home/benh/tmp/usr/local/share/mesos/webui/master/framework.tpl
>     /home/benh/tmp/usr/local/share/mesos/webui/master/webui.py
>     /home/benh/tmp/usr/local/share/mesos/webui/static
>     /home/benh/tmp/usr/local/share/mesos/webui/static/stylesheet.css
>     /home/benh/tmp/usr/local/share/mesos/webui/bottle-0.8.3
>     /home/benh/tmp/usr/local/share/mesos/webui/bottle-0.8.3/bottle.py
>     /home/benh/tmp/usr/local/sbin
>     /home/benh/tmp/usr/local/sbin/mesos-slave
>     /home/benh/tmp/usr/local/sbin/mesos-master

Scratch that last comment ... the Makefile.in was not regenerated!


- Benjamin


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/3672/#review4681
-----------------------------------------------------------


On 2012-01-28 00:23:01, Benjamin Hindman wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/3672/
> -----------------------------------------------------------
> 
> (Updated 2012-01-28 00:23:01)
> 
> 
> Review request for mesos, Charles Reiss and John Sirois.
> 
> 
> Summary
> -------
> 
> An installed webui has the wrong path, this should fix that.
> 
> 
> Diffs
> -----
> 
>   src/Makefile.am 1026758 
> 
> Diff: https://reviews.apache.org/r/3672/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Benjamin
> 
>

Reply via email to