#11696: The libpng12 spkg should also create symbolic links from `libpng.*` for 
its
libraries
-----------------------------------------------------+----------------------
       Reporter:  leif                               |         Owner:  leif     
        
           Type:  defect                             |        Status:  
needs_work       
       Priority:  major                              |     Milestone:  sage-5.8 
        
      Component:  packages                           |    Resolution:           
        
       Keywords:  PNG libpng spkg                    |   Work issues:           
        
Report Upstream:  N/A                                |     Reviewers:  Dmitrii 
Pasechnik
        Authors:  Leif Leonhardy, Jean-Pierre Flori  |     Merged in:           
        
   Dependencies:                                     |      Stopgaps:           
        
-----------------------------------------------------+----------------------

Comment (by jpflori):

 Replying to [comment:36 dimpase]:
 > This results in the name clash we see on OSX. I suppose you can  tweak
 the spkg  to remove all the {{{libpng.*}}}, they are apparently not
 needed.
 > (This of course defeats part of the purpose of this ticket :-)).
 Exactly, and the solution to delete files is somehow "dirty"...
 In particular if a package looks for a png rather than png12 it will fail,
 which may be considered bad.
 Or good as the problem with R mentioned here shows... if R finds png then
 it somehow triggers the use of the evil ImageIO which wants a symbol
 prefixed png, but its our non prefixed version which is loaded.

 Here are two different preliminary plans to get this merged.
 First one:
 * delete libpng.* from $SAGE_LOCAL/lib on Darwin
 and potentially (although quite unrelated):
 * include the fix from John from
 http://sage.math.washington.edu/home/leif/Sage/spkgs/matplotlib-1.0.1.p1.spkg
 to fix the problem described here http://groups.google.com/group/sage-
 release/msg/ab91298b636b947a caused by broken pkg-config (needs to be
 rebased on top of #11915).
 or for this second point:
 * modify the changes introduced in #11686 to matplotlib so that it only
 directly links to png12 rather than png when pkg-config is not installed
 and not unconditionally, because I feel it was not really needed (the real
 problem was that some pc file got broken, that should be now fixed in
 #11760) but kept because it was cleaner that way, i.e. only patch when
 needed, although http://groups.google.com/group/sage-
 release/msg/ab91298b636b947a showed it might be beneficial in other
 situations.

 Or second one:
 * do not delete libpng.* from $SAGE_LOCAL/lib
 and:
 * use sage-native-execute to run hg
 * find a similar workaround for R and other things which magically trigger
 the use of ImageIO on Darwin
 and potentially:
 * get rid of all fixes/hacks which make packages link directly to png12
 rather than png as this won't be needed anymore.

 The first solution looks infinitely more realistic and simpler, so I'll
 craft something based on that.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11696#comment:37>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to