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

Remove empty directories after PARI upgrade #9973

Closed
jdemeyer opened this issue Sep 23, 2010 · 6 comments
Closed

Remove empty directories after PARI upgrade #9973

jdemeyer opened this issue Sep 23, 2010 · 6 comments
Assignees
@jasongrout
Labels
p: major / 3 t: bug
Milestone
sage-4.6

Comments

@jdemeyer
Copy link

In #9343, all files in the directory data/extcode/pari/SEA and data/extcode/pari/cremona were removed, but the directory structure is still there.

There is no patch for this, the release manager should remove the directories manually.

CC: @qed777

Component: misc

Author: Jeroen Demeyer

Reviewer: David Kirkby

Merged: sage-4.6.alpha2

Issue created by migration from https://trac.sagemath.org/ticket/9973
@jdemeyer jdemeyer added this to the sage-4.6 milestone Sep 23, 2010
@jdemeyer

This comment has been minimized.

Sign in to view
@jdemeyer jdemeyer changed the title Remove empty directory data/extcode/pari/SEA Remove empty directories after PARI upgrade Sep 23, 2010
@sagetrac-drkirkby
Copy link
Mannequin

sagetrac-drkirkby mannequin commented Sep 23, 2010

comment:2

This is not very difficult to review.

@sagetrac-drkirkby
Copy link
Mannequin

sagetrac-drkirkby mannequin commented Sep 23, 2010

Reviewer: David Kirkby

@sagetrac-drkirkby
Copy link
Mannequin

sagetrac-drkirkby mannequin commented Sep 23, 2010

Author: Jeroen Demeyer

@jdemeyer
Copy link
Author

comment:4

Replying to @sagetrac-drkirkby:

This is not very difficult to review.

Thanks. Please see also #10001.

@qed777
Copy link
Mannequin

qed777 mannequin commented Sep 29, 2010

Merged: sage-4.6.alpha2

@qed777 qed777 mannequin removed the s: positive review label Sep 29, 2010
@qed777 qed777 mannequin closed this as completed Sep 29, 2010
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jasongrout jasongrout

Labels
p: major / 3 t: bug
Projects
None yet
Milestone
sage-4.6
Development

No branches or pull requests
2 participants
@jasongrout @jdemeyer