#11246: flint-1.5.0.p5's extraneous #includes break typedef ulong in sys/types.h
-----------------------------------+----------------------------------------
   Reporter:  dimpase              |          Owner:  tbd            
       Type:  defect               |         Status:  positive_review
   Priority:  major                |      Milestone:  sage-4.7.2     
  Component:  packages             |       Keywords:  cygwin         
Work_issues:                       |       Upstream:  N/A            
   Reviewer:  Karl-Dieter Crisman  |         Author:  Dima Pasechnik 
     Merged:                       |   Dependencies:                 
-----------------------------------+----------------------------------------

Comment(by kcrisman):

 Replying to [comment:23 leif]:
 > Replying to [comment:19 kcrisman]:
 > > Do we ever do error checking for these?  Most spkg-install files seem
 to have just "cp" or "patch".   What should be added?
 >
 > (`cp` will almost always succeed, so that's more or less irrelevant,
 meaning a different situation.)
 >
 > From the "proof-of-concept" spkg (Sphinx):
 > {{{
 > #!sh
 > # Apply patches
 > cd "$CUR/src"
 > echo "Patching Sphinx..."
 > for p in ../patches/*.patch; do
 >         patch -p1 <$p
 >         success "Error applying patch $p"
 > done
 > }}}
 >
 > (The `success` function exits `spkg-install` whenever `$? != 0`.)
 >

 This sounds like a great idea on a ''different'' ticket.  Getting the
 Cygwin things in is going to be hard enough!

 Unless you are volunteering to test the rest of them!   That would be very
 convenient :)

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11246#comment:26>
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