#9083: 'make distclean' fails to remove some files or directories.
------------------------+---------------------------------------------------
   Reporter:  drkirkby  |       Owner:  drkirkby    
       Type:  defect    |      Status:  needs_review
   Priority:  major     |   Milestone:  sage-4.4.3  
  Component:  build     |    Keywords:              
     Author:            |    Upstream:  N/A         
   Reviewer:            |      Merged:              
Work_issues:            |  
------------------------+---------------------------------------------------
Changes (by drkirkby):

  * keywords:  beginner =>


Comment:

 Replying to [comment:4 ddrake]:
 > Replying to [comment:2 drkirkby]:
 > > I stuck 'beginner' since the change to makefile was trivial, but I do
 take your point that it is a bit different to most other changes in Sage.
 The line
 > >
 > > {{{
 > >     rm -rf spkg/installed/*
 > > }}}
 > >
 > > can actually be removed, as your
 > >
 > > {{{
 > >     rm -rf spkg/installed
 > > }}}
 > >
 > > will do it, but it does not make much difference.
 >
 > I think those actually are different because of the trailing slash -- if
 we do
 > {{{
 > rm -rf spkg/installed*
 > }}}
 > I think that will get everything, right? If so, I can change the patch.

 The trailing slash should make no difference

 {{{
 rm -rf spkg/installed
 }}}

 will remove the directory spkg/installed and of course anything in any
 subdirectory of spkg/installed.

 In contrast

 {{{
 rm -rf spkg/installed*
 }}}

 would remove anything starting with spkg/installed, such as

 {{{
 spkg/installeda
 spkg/installedb
 spkg/installedc
 spkg/installed-but-do-not-delete-this-directory
 }}}

 I have removed the 'beginner' tag.

 Dave

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