Skip to content
This repository has been archived by the owner on Jan 16, 2025. It is now read-only.

Formula commonmark fails to build with Python 3.5 #44197

Closed
zmwangx opened this issue Sep 21, 2015 · 4 comments
Closed

Formula commonmark fails to build with Python 3.5 #44197

zmwangx opened this issue Sep 21, 2015 · 4 comments

Comments

@zmwangx
Copy link
Contributor

zmwangx commented Sep 21, 2015

Actually, make works fine, but make test fails, due to some incompatibility with Python 3.5 in the tests (or perhaps something deeper, but I doubt it — I haven't looked too deep into the issue, and I only have a partial fix at the moment). This is clearly an upstream issue, and I have reported this upstream: commonmark/cmark#83.

This affects installing on 10.11, since there's no bottle for it. Installing on earlier systems works fine, since the formula was built against Python 3.4 when it was last bottled.

Btw, for future reference, I've never been clear on this: when there's a failure due to an upstream issue (that is already reported upstream), should I report in Homebrew's issue tracker also? That might help with troubleshooting, but that might also pollute the issue tracker with stuff for which nothing can be done here.

@tdsmith
Copy link
Contributor

tdsmith commented Sep 22, 2015

I think we can reasonably choose not to run make test unless this will be fixed soon; what do you think?

I'm not clear on when it's useful to report upstream bugs here either; there's probably not a bright line. Many upstream bugs are actionable in some way. The most useful upstream-bug tickets are if there are upstream commits or sane-looking pull requests that we can apply as patches between releases. This report strikes me as useful since it's suddenly broken because of a change we (in this case: I) recently made to another package. Stuff that's visibly broken to a lot of Homebrew users is good to have a ticket for, since it'll be reported anyway.

@zmwangx
Copy link
Contributor Author

zmwangx commented Sep 22, 2015

I think we can reasonably choose not to run make test unless this will be fixed soon; what do you think?

No objection, provided that someone — probably I — will remember to add it back when the issue is fixed; it's still good to have in my opinion. I have no idea of when this will be fixed.

Submitted #44246.

@tdsmith
Copy link
Contributor

tdsmith commented Sep 22, 2015

I'll close this for now; thanks for reporting this and getting it fixed!

@tdsmith tdsmith closed this as completed Sep 22, 2015
@zmwangx
Copy link
Contributor Author

zmwangx commented Sep 22, 2015

No problem.

tdsmith pushed a commit that referenced this issue Sep 22, 2015
`make test` fails due to incompatibility with CPython 3.5.

commonmark/cmark#83
#44197

Closes #44246.

Signed-off-by: Tim D. Smith <git@tim-smith.us>
@Homebrew Homebrew locked and limited conversation to collaborators Jul 10, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants