This service will be undergoing maintenance at 00:00 UTC, 2016-09-28. It is expected to last about 1 hours
Bug 227752 - kinput2: $RPM_OPT_FLAGS not used
kinput2: $RPM_OPT_FLAGS not used
Status: CLOSED RAWHIDE
Product: Fedora
Classification: Fedora
Component: kinput2 (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Akira TAGOH
Fedora Extras Quality Assurance
: Patch
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2007-02-07 17:11 EST by Ville Skyttä
Modified: 2007-11-30 17:11 EST (History)
1 user (show)

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


Attachments (Terms of Use)
Use $RPM_OPT_FLAGS (1.30 KB, patch)
2007-02-07 17:11 EST, Ville Skyttä
no flags Details | Diff

  None (edit)
Description Ville Skyttä 2007-02-07 17:11:00 EST
http://www.redhat.com/archives/fedora-maintainers/2007-January/msg00339.html

kinput2 is built without $RPM_OPT_FLAGS (no compiler security features, more or
less useless debuginfo).  Fix attached.
Comment 1 Ville Skyttä 2007-02-07 17:11:00 EST
Created attachment 147606 [details]
Use $RPM_OPT_FLAGS
Comment 2 Akira TAGOH 2007-02-07 23:04:02 EST
Why don't you just patch out imake to generate proper Makefile that contains
$RPM_OPT_FLAGS?
Comment 3 Ville Skyttä 2007-02-08 02:48:08 EST
Other tools such as autofoo aren't patched to do that either.  Granted, they
respect CFLAGS and friends from environment, so it's not quite the same thing.

imake maintainers, what do you think about the change proposed in comment 2?
Comment 4 Matěj Cepl 2007-02-08 03:48:13 EST
Kristian, could please answer this question. Isn't imake legacy stuff anyway?
Comment 5 Kristian Høgsberg 2007-02-08 12:21:50 EST
I agree with comment #3, we're not patching imake (which very much is dying) to
add RPM specific flags.  What we can do is to fix the Imakefiles or the
generated makefiles as the patch does, and longer term work with the upstream
projects to allow custom compiler flags, or even better move away from imake.
Comment 6 Akira TAGOH 2007-02-09 08:10:14 EST
I meant to have a custom compiler flags rather than just having a special hack
since we could usually easily modify it in autotools by giving it before running
configure.
I don't like to move away from imake since this software is really a dead
project in upstream.
Ok, thank you for your info. will apply comment #1 then.
Comment 7 Akira TAGOH 2007-02-13 01:01:22 EST
Fixed in v3.1-31.fc7.

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