#11280: check XCode version on MacOSX, and bail out if it is 4
-----------------------+----------------------------------------------------
   Reporter:  dimpase  |          Owner:  GeorgSWeber      
       Type:  defect   |         Status:  needs_work       
   Priority:  blocker  |      Milestone:  sage-4.7         
  Component:  build    |       Keywords:  XCode 4          
Work_issues:           |       Upstream:  N/A              
   Reviewer:           |         Author:  Dmitrii Pasechnik
     Merged:           |   Dependencies:                   
-----------------------+----------------------------------------------------
Changes (by drkirkby):

  * status:  needs_review => needs_work


Comment:

 If this Xcode version checking is performed, then the version of prereq
 needs to be incremented to 0.9, otherwise we are going to have different
 code with the same version number. That means creating a new tar file
 (prereq-0.8.tar), and changing the TARGET at the top of prereq-0.9-install
 to:

 {{{
 TARGET=prereq-0.9
 }}}

 Also, I think the bit of code following:

 {{{
     if [ $XCODE_VERS_MAJOR -gt 2 ]; then
         echo "You are using XCode version " $XCODE_VERS
         echo "WARNING: You are strongly advised to install Apple's latest
 XCode 3,"
         echo "unless you already have it. You can download this from "
         echo "http://developer.apple.com/xcode/";
     fi
 }}}

 should make it clear '''not''' to install Xcode 4. Although the message
 says to use the latest Xcode 3, someone reading it is likely to interpret
 that as to just upgrade.

 Also, why use the -e option to sed? That's for when you want to specify
 the name of a script. If you put the script inline, as you have done here,
 the -e can be removed.

 Dave

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