osx packaging: issues with fixlib() in osx-app.sh

Bug #532765 reported by su_v on 2010-03-05
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Undecided
Unassigned

Bug Description

current issues with fixlib() in 'packaging/macosx/osx-app.sh' when building the application package from current bzr trunk (r9145):

1) fails if not run with 'sudo' rights because OpenSSL libs are installed with limited permissions ('r-xr-xr-x') in MacPorts (libcrypto.0.9.8.dylib, libz.1.dylib)

2) fails if custom MacPorts tree is installed in user account or on (mounted) partition (i.e. custom MacPorts prefix has more than 2 dir levels).
 works with: /opt/local-macports-with-a-really-very-quite-long-directory-name
 fails with: /Volumes/blue/mp-inkscape/with-a-long-long-long-directory-name
due to hard-coded level of directories (using e.g. "relative=`echo $lib | cut -d/ -f4-`")

3) redundant second-level 'for'-loop: rewrites paths of all linked dylibs that had been copied to 'Contents/Resources/lib' every time a linking *.so executable is changed (i.e. all linked dylibs are touched over and over again - but they are parsed at the end of rewritelibpaths() anyway, so this second-level loop is not needed AFAIU)

4) incorrect id of many 'Contents/Resources/lib/*.dylib' executables: because id change is part of the first level 'for'-loop for the linked dylibs and not for the current lib, every loop incorrectly sets the install name of the linked lib to the name of the linking lib.
Do we really need to change the install name at all? AFAIU it is only required when a dynamic library is linked into another executable.

Mac OS X 10.5.8
Inkscape 0.47+devel r9145
MacPorts 1.8.99 (trunk)

su_v (suv-lp) wrote :

attached is the output from

 $ cd Inkscape.app/Contents/Resources
 $ otool -D lib/*.dylib

of Inkscape 0.47 r22583

su_v (suv-lp) wrote :

patch of the changes in osx-app.sh used for packaging Inkscape 0.47+devel r9145 (uploaded to modevia)

su_v (suv-lp) wrote :

…and the modified file itself.

su_v (suv-lp) wrote :

issue 1) "file permissions" fixed in r9170

Changed in inkscape:
assignee: nobody → Michael Wybrow (mjwybrow)
su_v (suv-lp) wrote :

Closing - out-of-date.

Changed in inkscape:
assignee: Michael Wybrow (mjwybrow) → nobody
status: New → Invalid
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers