New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
doc: admin/build-doc: make paths absolute #7119
Conversation
|
||
if [ ! -e $vdir ]; then | ||
virtualenv --system-site-packages $vdir | ||
fi | ||
$vdir/bin/pip install --quiet -r ../admin/doc-requirements.txt | ||
$vdir/bin/pip install --quiet -r $TOPDIR/admin/doc-requirements.txt | ||
|
||
install -d -m0755 \ | ||
output/html \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should these be absolute too?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yep
lgtm |
Build failure on docs.dhc because pip apparently was changing directory before building rbd with cython, causing the relative library path references to fail. Bite the bullet and make everything absolute (since we already have TOPDIR to use). Signed-off-by: Dan Mick <dan.mick@redhat.com>
admin/build-doc: make paths absolute Reviewed-by: Josh Durgin <jdurgin@redhat.com>
This looks to have broken the centos7 gitbuilder, guys: http://gitbuilder.sepia.ceph.com/gitbuilder-ceph-rpm-centos7-amd64-basic/log.cgi?log=7dc7c423bb7e7f2dfbce674187ee7863f27729b0 |
Actually, hmm, maybe something just didn't get set right for the first build. Stuff on top of the commit seems to be getting past that point (http://gitbuilder.sepia.ceph.com/gitbuilder-ceph-rpm-centos7-amd64-basic/log.cgi?log=ca07d1db3edb9e59d485a3a5799c2ff9c13bcadf) so we may be good |
Thanks @dmick for fixing this |
Build failure on docs.dhc because pip apparently was changing directory
before building rbd with cython, causing the relative library path
references to fail. Bite the bullet and make everything absolute (since
we already have TOPDIR to use).
Signed-off-by: Dan Mick dan.mick@redhat.com