Bug 756862 - [PATCH] Build with $RPM_LD_FLAGS, use xz compressed tarball
Summary: [PATCH] Build with $RPM_LD_FLAGS, use xz compressed tarball
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: python
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Dave Malcolm
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-11-24 22:03 UTC by Ville Skyttä
Modified: 2012-01-04 21:30 UTC (History)
3 users (show)

Fixed In Version: 2.7.2-18
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-01-04 21:30:44 UTC
Type: ---


Attachments (Terms of Use)
Build with $RPM_LD_FLAGS, use xz compressed tarball (2.32 KB, patch)
2011-11-24 22:03 UTC, Ville Skyttä
no flags Details | Diff

Description Ville Skyttä 2011-11-24 22:03:47 UTC
Created attachment 536061 [details]
Build with $RPM_LD_FLAGS, use xz compressed tarball

* Thu Nov 24 2011 Ville Skyttä <ville.skytta> - 2.7.2-18
- Build with $RPM_LD_FLAGS.
- Use xz-compressed source tarball.

Like with perl (see bug 756830), this will propagate to arch dependent python modules "for free" too.  See the perl bug for some more background.

Want me to push and build this for devel?

Comment 1 Dave Malcolm 2011-11-29 17:43:33 UTC
The bz2 -> xz change looks sane, go for it.

I'm wondering what the effects of setting LD_FLAGS to $RPM_LD_FLAGS is (if any).

Comment 2 Dave Malcolm 2011-11-29 17:47:56 UTC
Note to self: bug 728974 added $RPM_LD_FLAGS; see:
  https://fedorahosted.org/fesco/ticket/563
for discussion of what it does

Comment 3 Ville Skyttä 2011-11-29 20:10:07 UTC
(In reply to comment #1)
> The bz2 -> xz change looks sane, go for it.

Just to make sure, do you mean that I should apply both the tarball and the ldflags change?

More information about the ldflags stuff is at http://lists.fedoraproject.org/pipermail/devel/2011-August/155358.html .  Note that my patch here implements the "Partial RELRO" case, not the "Full RELRO" or "hardened" one.

Comment 4 Ville Skyttä 2012-01-04 20:50:05 UTC
I took the liberty to push and build this for rawhide so as many packages as possible get the benefits as part of the ongoing gcc 4.7.0 mass rebuild.

However, the build failed, but I *guess* the failure was not related to this build because the previous build had failed as well (hard to tell because apparently the build logs are no longer available).  Leaving open until a version with this is actually in the repos.

http://koji.fedoraproject.org/koji/buildinfo?buildID=281119

Comment 5 Ville Skyttä 2012-01-04 20:51:20 UTC
(In reply to comment #4)
Please ignore that comment, it was intended for bug 756863.

Comment 6 Ville Skyttä 2012-01-04 21:30:44 UTC
I took the liberty to push and build this for rawhide so as many packages as
possible get the benefits as part of the ongoing gcc 4.7.0 mass rebuild.


Note You need to log in before you can comment on or make changes to this bug.