Skip to content
This repository was archived by the owner on Apr 29, 2021. It is now read-only.

Workaround fix for removing BR python2#2

Closed
stratakis wants to merge 1 commit intofedora-python:masterfrom
stratakis:master
Closed

Workaround fix for removing BR python2#2
stratakis wants to merge 1 commit intofedora-python:masterfrom
stratakis:master

Conversation

@stratakis
Copy link
Copy Markdown
Member

No description provided.

@hroncok
Copy link
Copy Markdown
Member

hroncok commented Oct 4, 2016

How does it work?

@stratakis
Copy link
Copy Markdown
Member Author

Not sure of the exact magic behind it, it seems that these files are supposed to be generated, but they need to exist as well.

Solution taken from https://mail.python.org/pipermail/python-dev/2010-December/106425.html

@encukou
Copy link
Copy Markdown
Member

encukou commented Oct 4, 2016

These files are generated using Python – basically, you need to have Python installed to fully compile Python.
For this reason, CPython release tarballs (and even the VCS tree) includes such generated files. But, make still re-makes them according to the usual rules, i.e. if they're old enough.

According to the Fedora packaging guidelines, it would be preferred to BuildRequire python2 here, and re-build these files. But I don't think it really matters; these are hardly the only files that are regenerated like this.
The compliant thing to do would involve bootstrapping, but if we want to go that way, let's do it for current python3 only.

@stratakis
Copy link
Copy Markdown
Member Author

Also for some reason this happens only at F23 and F24, not at F25 or F26

@torsava
Copy link
Copy Markdown
Member

torsava commented Oct 5, 2016

The compliant thing to do would involve bootstrapping, but if we want to go that way, let's do it for current python3 only.

python3 in Fedora is already bootstrapped.

@hroncok hroncok closed this Apr 29, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants