On 3/5/14, 21:24 , Mario Juric wrote:
> On 2/27/14, 9:21 , Fabrice Jammes wrote:
>> Hello Mario,
>>
>> eupspkg file contains function default_install() (see below). In this
>> function, the install procedure for a scons script doesn't run
>> *install_ups*, whereas all other install procedures launch it.
>> Do you think it could be added, or do i have to add and ups/eupspkg.sh
>> in Qserv build procedure ? Indeed Qserv relies on standard eupspkg build
>> procedure and scons, but installing it print next message :
>>
>
> Hmm, OK, good point.
>
> The reason that install_ups wasn't there was because packages that use
> sconsUtils install it automatically. I never thought (which I should, my
> mistake!) that a vanilla SCons project would need it.
>
> I'll fix it tomorrow.
>
Fabrice,
A proposed fix is now on the tip of my EUPS' eupspkg branch (commit
3fb8491). Let me know if it works (didn't test it well yet). It's a
simple change:
============
[mjuric@lsst-dev [eupspkg] /ssd/mjuric/eupspkg/sources/eups]$ git diff
HEAD^..
diff --git a/lib/eupspkg.sh b/lib/eupspkg.sh
index 39f187f..56923f8 100755
--- a/lib/eupspkg.sh
+++ b/lib/eupspkg.sh
@@ -738,22 +738,26 @@ default_install()
scons -j$NJOBS prefix="$PREFIX" version="$VERSION"
cc="$CC" install
elif [[ -f configure ]]; then
make -j$NJOBS $MAKE_INSTALL_TARGETS
- install_ups
elif [[ -f Makefile || -f makefile || -f GNUmakefile ]]; then
make -j$NJOBS prefix="$PREFIX" version="$VERSION"
$MAKE_INSTALL_TARGETS
- install_ups
elif [[ -f setup.py ]]; then
PYDEST="$PREFIX/lib/python"
mkdir -p "$PYDEST"
PYTHONPATH="$PYDEST:$PYTHONPATH" python setup.py install
$PYSETUP_INSTALL_OPTIONS
evil_setuptools_pth_fix "$PYDEST"
- install_ups
else
- # just copy everything
+ # just copy everything, except for the ups directory
mkdir -p "$PREFIX"
cp -a ./ "$PREFIX"
+ rm -rf "$PREFIX/ups"
msg "Copied the product into '$PREFIX'"
fi
+
+ # Install ups if the native build system hasn't done it already.
+ # We do this check to avoid expanding the table file twice (EUPS
has a bug there)
+ if [[ -d "ups" && ! -d "$PREFIX/ups" ]]; then
+ install_ups
+ fi
}
============
Cheers,
--
Mario Juric,
Data Mgmt. Project Scientist, Large Synoptic Survey Telescope
Web : http://research.majuric.org Phone : +1 617 744 9003
########################################################################
Use REPLY-ALL to reply to list
To unsubscribe from the QSERV-L list, click the following link:
https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=QSERV-L&A=1
|