Opened 10 years ago

Last modified 10 years ago

#12637 closed defect

Follow-up to #4949: don't delete the current working directory — at Version 3

Reported by: jhpalmieri Owned by: GeorgSWeber
Priority: blocker Milestone: sage-5.0
Component: build Keywords:
Cc: Merged in:
Authors: John Palmieri Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by jhpalmieri)

On Solaris, running rm -rf DIR fails if DIR is in the path of the current working directory. This happens in sage-spkg, so the temporary build directories $SAGE_BUILD_DIR/$PKG_NAME don't get deleted. Fix this by running cd ... first.

Apply trac_12637-cd-before-rm.patch​ to the root repo.

Change History (4)

comment:1 Changed 10 years ago by jhpalmieri

  • Description modified (diff)
  • Status changed from new to needs_review

comment:2 Changed 10 years ago by jdemeyer

Could you just add a comment like 

# On Solaris, the current working directory cannot be deleted,
# so we "cd" out of $SAGE_BUILD_DIR/$PKG_NAME, see #12637.

comment:3 Changed 10 years ago by jhpalmieri

  • Description modified (diff)

Done. (Also changed the name of the patch file to match the ticket number.)

Changed 10 years ago by jhpalmieri

root repo

Note: See TracTickets for help on using tickets.