Bug 756862

Summary: [PATCH] Build with $RPM_LD_FLAGS, use xz compressed tarball
Product: [Fedora] Fedora Reporter: Ville Skyttä <ville.skytta>
Component: pythonAssignee: Dave Malcolm <dmalcolm>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: dmalcolm, ivazqueznet, jonathansteffan
Target Milestone: ---Keywords: Patch
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 2.7.2-18 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-01-04 21:30:44 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
Build with $RPM_LD_FLAGS, use xz compressed tarball none

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.