Bug 229054 - pgadmin3: $RPM_OPT_FLAGS not used
pgadmin3: $RPM_OPT_FLAGS not used
Product: Fedora
Classification: Fedora
Component: pgadmin3 (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: Gavin Henry
Fedora Extras Quality Assurance
: Patch
Depends On:
  Show dependency treegraph
Reported: 2007-02-16 13:45 EST by Ville Skyttä
Modified: 2008-12-30 08:32 EST (History)
3 users (show)

See Also:
Fixed In Version: 1.8.4-2
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2008-12-30 08:32:51 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)
Use $RPM_OPT_FLAGS (555 bytes, patch)
2007-02-16 13:45 EST, Ville Skyttä
no flags Details | Diff
Specfile part of the fix (885 bytes, patch)
2007-02-16 13:46 EST, Ville Skyttä
no flags Details | Diff

  None (edit)
Description Ville Skyttä 2007-02-16 13:45:47 EST

Fix attached.  devel needs a similar fix, but I don't have a Rawhide box to test
with at the moment.  --disable-dependency-tracking is not really part of the
fix, but beneficial in the cleaner build output (easier to spot problems like
this) sense and may provide some build speedups.
Comment 1 Ville Skyttä 2007-02-16 13:45:47 EST
Created attachment 148224 [details]
Comment 2 Ville Skyttä 2007-02-16 13:46:30 EST
Created attachment 148226 [details]
Specfile part of the fix
Comment 3 Ville Skyttä 2007-07-28 16:09:01 EDT
Ping?  Still affects current releases it seems.
Comment 4 Ville Skyttä 2007-10-12 18:54:01 EDT
Ping?  Devel still affected, updated patch at
Comment 5 Devrim GUNDUZ 2007-11-14 23:03:02 EST

Pushed 1.8.0 to F8 and rawhide. It will appear in F-7 soon. Could you please try
it under F-8? AFAICS this is solved.

Regards, Devrim
Comment 6 Ville Skyttä 2007-11-15 11:21:34 EST
Not fixed, the build log clearly indicates that $RPM_OPT_FLAGS are still not in
use, see eg. http://koji.fedoraproject.org/koji/getfile?taskID=242080&name=build.log
Comment 7 Ville Skyttä 2007-11-15 11:22:24 EST
(Devrim, see comment 6)
Comment 8 Fedora Update System 2007-11-15 19:40:13 EST
pgadmin3-1.8.0-1.fc7 has been pushed to the Fedora 7 stable repository.  If problems still persist, please make note of it in this bug report.
Comment 9 Ville Skyttä 2007-11-18 06:59:39 EST
As noted, the problem does persist, see comment 6.
Comment 10 Bug Zapper 2008-04-03 15:10:09 EDT
Based on the date this bug was created, it appears to have been reported
against rawhide during the development of a Fedora release that is no
longer maintained. In order to refocus our efforts as a project we are
flagging all of the open bugs for releases which are no longer
maintained. If this bug remains in NEEDINFO thirty (30) days from now,
we will automatically close it.

If you can reproduce this bug in a maintained Fedora version (7, 8, or
rawhide), please change this bug to the respective version and change
the status to ASSIGNED. (If you're unable to change the bug's version
or status, add a comment to the bug and someone will change it for you.)

Thanks for your help, and we apologize again that we haven't handled
these issues to this point.

The process we're following is outlined here:

We will be following the process here:
http://fedoraproject.org/wiki/BugZappers/HouseKeeping to ensure this
doesn't happen again.
Comment 11 Ville Skyttä 2008-04-03 16:27:55 EDT
Still an issue with current Rawhide:


For example:
if g++ -DHAVE_CONFIG_H -I. -I. -I..   -I/usr/include -DSSL
-I/usr/lib64/wx/include/gtk2-unicode-release-2.8 -I/usr/include/wx-2.8
-I/usr/include/libxml2 -I/usr/include/libxml2
-DDATA_DIR=\"/usr/share/pgadmin3/\" -Wall -Wno-non-virtual-dtor
-fno-strict-aliasing -I../pgadmin/include   -MT pgAdmin3.o -MD -MP -MF
".deps/pgAdmin3.Tpo" -c -o pgAdmin3.o `test -f './pgAdmin3.cpp' || echo
'./'`./pgAdmin3.cpp; \

No -D_FORTIFY_SOURCE=2 (which can be seen as a security issue) nor other flags
that are in $RPM_OPT_FLAGS.
Comment 12 Bug Zapper 2008-05-13 22:37:30 EDT
Changing version to '9' as part of upcoming Fedora 9 GA.
More information and reason for this action is here:
Comment 13 Ville Skyttä 2008-07-14 00:28:45 EDT
Because there was no progress nor real comments here in about 1.5 years and the
ACLs allow it, I updated the patch for the third time and committed it as
1.8.4-2 to devel, but did not tag or build it yet.  Let me know if you'd like me
to do that.
Comment 14 John Poelstra 2008-07-15 15:25:43 EDT
changing to needinfo re: comment #13
Comment 15 Devrim GUNDUZ 2008-08-11 03:59:06 EDT
(In reply to comment #13)
> Because there was no progress nor real comments here in about 1.5 years and the
> ACLs allow it, I updated the patch for the third time and committed it as
> 1.8.4-2 to devel, but did not tag or build it yet.  Let me know if you'd like me
> to do that.

Pushed to devel.
Comment 16 Robert Scheck 2008-12-30 08:32:51 EST
Fixed in CVS and latest build (1.8.4-2), closing hereby.

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