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 pbuild files #12635

Closed
jdemeyer opened this issue Mar 6, 2012 · 5 comments
Closed

Remove pbuild files #12635

jdemeyer opened this issue Mar 6, 2012 · 5 comments
Assignees
@jasongrout
Labels
p: major / 3 t: enhancement
Milestone
sage-5.0

Comments

@jdemeyer
Copy link

jdemeyer commented Mar 6, 2012

The "pbuild" system was some different system for building Sage. It hasn't been used since sage-3.1.2 (#4124), so it can safely be deleted.

Component: misc

Author: Jeroen Demeyer

Reviewer: Punarbasu Purkayastha

Merged: sage-5.0.beta8

Issue created by migration from https://trac.sagemath.org/ticket/12635
@jdemeyer jdemeyer added this to the sage-5.0 milestone Mar 6, 2012
@jdemeyer
Copy link
Author

jdemeyer commented Mar 6, 2012

Attachment: 12635_remove_pbuild.patch.gz

Apply to EXTCODE repository

@jdemeyer
Copy link
Author

jdemeyer commented Mar 6, 2012

Author: Jeroen Demeyer

@ppurka
Copy link
Member

ppurka commented Mar 7, 2012

comment:2

No harm in removing cruft. Positive review from me. There doesn't seem to be any mention of the removed files in the remaining files. Typical command I used (in zsh):

...tions/sage-5.0.beta2 [1] > LC_ALL="C" grep -rnIw 'sageenv' ^(devel)/**/*.py | grep -v 'local/lib'
...tions/sage-5.0.beta2 [1] >

@ppurka
Copy link
Member

ppurka commented Mar 7, 2012

Reviewer: Punarbasu Purkayastha

@jdemeyer
Copy link
Author

Merged: sage-5.0.beta8

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: enhancement
Projects
None yet
Milestone
sage-5.0
Development

No branches or pull requests
3 participants
@jasongrout @ppurka @jdemeyer