#9603: Build iconv in parallel, install it on HP-UX and make it work properly on
Solaris 64-bit.
--------------------------------------------+-------------------------------
   Reporter:  drkirkby                      |       Owner:  drkirkby    
       Type:  defect                        |      Status:  needs_review
   Priority:  major                         |   Milestone:  sage-4.6    
  Component:  build                         |    Keywords:              
     Author:  David Kirkby                  |    Upstream:  N/A         
   Reviewer:  Peter Jeremy, Leif Leonhardy  |      Merged:              
Work_issues:                                |  
--------------------------------------------+-------------------------------

Comment(by leif):

 Replying to [comment:44 drkirkby]:
 > The package I reference at
 >
 >
 http://boxen.math.washington.edu/home/kirkby/patches/iconv-1.13.1.p3.spkg
 >
 > has a couple of changes that are not checked in.

 I mentioned that somewhere above. I've committed your changes in your name
 (these were only those related to #9718 - hopefully).

 > When I apply your patches to that, so I get a reject.

 Ooops? Don't know why; should not happen... 8|

 > Can you create a complete package and provide a link. Then I'll check
 it.

 Yes, I already have it (actually two), just have to upload one of them.

 > I'll go with your second reviewer patch too,

 Fine, thanks.

 > though is there not a mistake on line 21 of {{{spkg-check}}} ?

 You mean {{{;&}}}? That's "the opposite" of {{{;;}}} ("break"), i.e. fall-
 through.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9603#comment:45>
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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/sage-trac?hl=en.

Reply via email to