Hello FESCo.
When https://fedoraproject.org/wiki/Changes/Python_built_with_gcc_O3 was originally proposed, the expectation was that the C flags for building Python extension modules other than modules from the Python standard library would not be changed.
-O2
-O
However, after implementing the change proposal, it was accidentally changed so the Python extension modules built outside of Fedora RPM packages are built with the -O3 flag. This was not originally intended, yet the change owners believe we should keep it that way because it makes Fedora's Python closer to upstream Python and because it makes Fedora more competitive with other platforms on CIs and similar systems.
-O3
For more details, see https://lists.fedoraproject.org/archives/list/python-devel@lists.fedoraproject.org/thread/2IXTGNASAVR3NNHKFCOIC27CMFA6RJRH/
I've put that to the change proposal as https://fedoraproject.org/wiki/Changes/Python_built_with_gcc_O3#Amendment:_Flag_for_user-built_extension_modules and https://discussion.fedoraproject.org/t/f41-change-proposal-python-built-with-gcc-03-self-contained/112743/17
I'd like to ask FESCo to approve this amendment. If you think this should wait a week on discussion.fedoraproject.org, I am OK with that. Thanks
+1, assuming no gotchas come up in discussion -- and I do think we should give that time.
Yes, I think we should wait a week for comment on the discussion thread. Please also mention it on the fedora-devel list, as not everyone is subscribed to the original Change on Discussion.
That said, if it's been working this far, I'm leaning +1 to letting it continue.
Fine with me +1
I've posted this to devel-announce as a reply to the original change announcement, pending approval.
+1 here too
It's now been a week with 4 +1s, no -1s and no responses on either the discussion or devel threads. I'm calling this approved.
APPROVED (+4, 0, -0)
Metadata Update from @sgallagh: - Issue tagged with: pending announcement
This was announced last week, but I forgot to close the ticket.
Metadata Update from @sgallagh: - Issue close_status updated to: Accepted - Issue status updated to: Closed (was: Open)
Log in to comment on this ticket.