8000 Supported FreeBSD10 as per #225 by pelson · Pull Request #982 · matplotlib/matplotlib · GitHub
[go: up one dir, main page]
Skip to content

Supported FreeBSD10 as per #225 #982

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
Closed
wants to merge 1 commit into from
Closed

Supported FreeBSD10 as per #225 #982

wants to merge 1 commit into from

Conversation

pelson
Copy link
Member
@pelson pelson commented Jun 30, 2012

Capability requested in #225.

@jkseppan
Copy link
Member
jkseppan commented Jul 1, 2012

+1

@pelson
Copy link
Member Author
pelson commented Jul 1, 2012

Note: This PR is untested.
@peterjeremy are you able to test? If not, I suggest we merge this PR and see how it fares.

@ivanov
Copy link
Member
ivanov commented Jul 3, 2012

hang on, @pelson, I've got an alternative proposal coming shortly.

@ivanov ivanov mentioned this pull request Jul 3, 2012
Merged
@ivanov
Copy link
Member
ivanov commented Jul 3, 2012

ok, done, see #985 for the alternatives which removes redundancy and allows us to ignore innocent version bumps that won't otherwise break anything in the future.

@pelson
Copy link
Member Author
pelson commented Jul 3, 2012

Thanks @ivanov. I think there is a small discussion to be had on #985 which will help us decide to either merge this PR or #985.

@ivanov
Copy link
Member
ivanov commented Jul 3, 2012

As I said in the other PR, I just don't like the idea of duplication of effort. According to this CVS history, FreeBSD has been patching this issue in their ports tree since 2008. So every time FreeBSD get a new release, they will patch it, and then we will also end up having to patch it once someone tries to build from git on FreeBSD and not from ports, and Sage is also patching it.

@pelson
Copy link
Member Author
pelson commented Jul 13, 2012

Closed in favour of #985

@pelson pelson closed this Jul 13, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@pelson @jkseppan @ivanov
0