Bug 227597 - jogl: $RPM_OPT_FLAGS not used, other issues
jogl: $RPM_OPT_FLAGS not used, other issues
Status: CLOSED WONTFIX
Product: Fedora
Classification: Fedora
Component: jogl (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Anthony Green
Fedora Extras Quality Assurance
: Patch
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2007-02-06 18:21 EST by Ville Skyttä
Modified: 2007-11-30 17:11 EST (History)
0 users

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2007-11-11 16:33:04 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)
Use $RPM_OPT_FLAGS, fix file permissions and build directly from CVS (1.70 KB, patch)
2007-02-06 18:21 EST, Ville Skyttä
no flags Details | Diff

  None (edit)
Description Ville Skyttä 2007-02-06 18:21:45 EST
http://www.redhat.com/archives/fedora-maintainers/2007-January/msg00339.html

jogl's native parts are built without $RPM_OPT_FLAGS, resulting in no compiler
security features and more or less useless debuginfo.  Crude fix attached.

I couldn't find a way to pass $RPM_OPT_FLAGS sanely to the cpptasks stuff, so
this approach uses a wrapper script.

In addition to fixing optflags, the patch fixes the build for me directly from
CVS (the FE CVS setup doesn't override %{_topdir}, but $RPM_SOURCE_DIR should
work everywhere), as well as fixes file permissions of *.so which should be
usually executable, 755.  At least they must not be 766 (world-writable) in
%install, - that'll result in world writable corresponding *.debug in the
-debuginfo subpackage as well even if fixed in the %files section for the main
package.
Comment 1 Ville Skyttä 2007-02-06 18:21:46 EST
Created attachment 147525 [details]
Use $RPM_OPT_FLAGS, fix file permissions and build directly from CVS
Comment 2 Ville Skyttä 2007-07-28 16:11:52 EDT
Ping?
Comment 3 Ville Skyttä 2007-10-12 18:22:48 EDT
Ping #2?  Current devel needs some further fixes for the build to succeed,
updated patch at http://scop.fedorapeople.org/patches/debuginfo/jogl.patch
Comment 4 Anthony Green 2007-10-12 19:05:29 EDT
Thanks for pinging me.  I'll look at this over the weekend.
Comment 5 Anthony Green 2007-11-11 16:33:04 EST
jogl is no longer part of Fedora due to licensing issues.

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