Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

testcflags.sh leaves a directory in spkg/build #13945

Closed
jhpalmieri opened this issue Jan 11, 2013 · 6 comments
Closed

testcflags.sh leaves a directory in spkg/build #13945

jhpalmieri opened this issue Jan 11, 2013 · 6 comments
Assignees
@sagetrac-GeorgSWeber
Labels
c: build p: minor / 4 t: bug
Milestone
sage-5.6

Comments

@jhpalmieri
Copy link
Member

After running ./sage -f spkg/standard/libm4rie-20120613.spkg on OS X 10.8.2, there is a directory with a name like SAGE_ROOT/spkg/build/sage-testcflags-24223.dSYM/, created by the script testcflags.sh. The attached patch cleans this up.

CC: @jdemeyer

Component: build

Author: John Palmieri

Reviewer: Jeroen Demeyer

Merged: sage-5.6.rc1

Issue created by migration from https://trac.sagemath.org/ticket/13945
@jhpalmieri jhpalmieri added this to the sage-5.6 milestone Jan 11, 2013
@jdemeyer
Copy link

comment:2

positive review provided you add a comment along the lines

# Some OS X systems create a directory with debug info, so we really need -r here (#13945)

@jhpalmieri
Copy link
Member Author

Attachment: trac_13945-testcflags.patch.gz

@jhpalmieri
Copy link
Member Author

Reviewer: Jeroen Demeyer

@jhpalmieri
Copy link
Member Author

comment:3

Okay, done.

@jdemeyer
Copy link

Author: John Palmieri

@jdemeyer jdemeyer modified the milestones: sage-5.6, sage-5.7 Jan 12, 2013
@jdemeyer
Copy link

Merged: sage-5.6.rc1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sagetrac-GeorgSWeber sagetrac-GeorgSWeber

Labels
c: build p: minor / 4 t: bug
Projects
None yet
Milestone
sage-5.6
Development

No branches or pull requests
2 participants
@jhpalmieri @jdemeyer