Spud

Inconsistent handling of prefix during install

Bug #921608 reported by Tim Bond
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Spud
Fix Released
Medium
Tim Greaves

Bug Description

Method from a clean spud checkout:

 ./configure --prefix=/data/installdir1
 cd diamond
 python ./setup.py install --prefix=/data/installdir2

This results in bin/ and lib/ being placed in installdir2, but share/ being placed in installdir1.

A slightly esoteric setup, but this is what happens inside Fluidity where the Fluidity configure overrides the user-supplied prefix and forces a prefix of the Fluidity source directory, followed by directly running setup.py to install diamond in the user-supplied prefix directory.

The end result is that the GUI and plugin directories don't get properly installed.

Is this a bug? Is it something that could be fixed?

Related branches

lp:~tim-greaves/spud/diamond-setup-py-fix2
Patrick Farrell: Approve
Diff: 46 lines (+23/-4)
1 file modified
diamond/setup.py.in (+23/-4)
Revision history for this message
Patrick Farrell (pefarrell) wrote :

Try lp:~pefarrell/libspud/diamond-setup-py ?

Revision history for this message
Tim Bond (timothy-bond) wrote :

That works very well in terms of installing to the 'prefix' supplied to setup.py, thanks!

It does still install to the configure-prefix directory as well - is that intentional?

(Ie, you get a double install of the 'share' files)

Revision history for this message
Tim Greaves (tim-greaves) wrote :

I was overoptimistic - I think. Trying copying in the setup.py.in to the fluidity/libspud/diamond directory inside Fluidity, it partially works in that prefix/share/diamond/plugs is created correctly, but the gui directory still only gets installed in a 'local' version as defined by the original configure's prefix.

I'll attempt to work out what's going on there and report back here.

Tim Greaves (tim-greaves)
Changed in spud:
status: New → Fix Committed
importance: Undecided → Medium
assignee: nobody → Tim Greaves (tim-greaves)
Tim Greaves (tim-greaves)
Changed in spud:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.