#3537: [with patch] sage-env should set RM="rm -f"
-------------------------+--------------------------------------------------
    Reporter:  gfurnish  |         Owner:  gfurnish       
        Type:  defect    |        Status:  positive_review
    Priority:  major     |     Milestone:  sage-4.7       
   Component:  build     |    Resolution:                 
    Keywords:            |   Work_issues:                 
    Upstream:  N/A       |      Reviewer:  Mariah Lenox   
      Author:  gfurnish  |        Merged:                 
Dependencies:            |  
-------------------------+--------------------------------------------------
Changes (by mariah):

  * status:  needs_review => positive_review
  * reviewer:  => Mariah Lenox


Old description:

> The env variable RM is set to `rm` instead of `rm -f`. This breaks newer
> libtools, for example anything in Fedora 12 or later (libtool 2.2.6,
> autoconf 2.63, automake 1.11.1). They assume that `$RM` is either unset
> or `RM="rm -f"`, that is, deleting non-existing files must not cause an
> error.
>
> One of the symptoms of this breakage is that configure ends with
> {{{
> rm: cannot remove `libtoolT': No such file or directory
> }}}
> Compile will break later on...
>
> -------
>
> There are two approaches presented here:
>
>  - change $RM from "rm" to "rm -f"  (this is
> [attachment:trac_3537_scripts.patch])
>  - keep $RM as it is, but document it (this is
> [attachment:trac_3537-doc.patch])
>
> Only one should be applied, not both.

New description:

 The env variable RM is set to `rm` instead of `rm -f`. This breaks newer
 libtools, for example anything in Fedora 12 or later (libtool 2.2.6,
 autoconf 2.63, automake 1.11.1). They assume that `$RM` is either unset or
 `RM="rm -f"`, that is, deleting non-existing files must not cause an
 error.

 One of the symptoms of this breakage is that configure ends with
 {{{
 rm: cannot remove `libtoolT': No such file or directory
 }}}
 Compile will break later on...

 -------

 There are two approaches presented here:

  - change $RM from "rm" to "rm -f"  (this is
 [attachment:trac_3537_scripts.patch])
  - keep $RM as it is, but document it (this is
 [attachment:trac_3537-doc.patch])

 Only one should be applied, not both.

 '''Apply''' [attachment:trac_3537-doc.patch]

--

Comment:

 When 50 percent or more of spkgs require $RM to be "rm -f" then
 change $RM from "rm" to "rm -f".  Until then document.

 Positive review.

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