Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225658
Summary: Merge Review: cpuspeed Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: nobody@fedoraproject.org QAContact: fedora-package-review@redhat.com CC: jwilson@redhat.com
Fedora Merge Review: cpuspeed
http://cvs.fedora.redhat.com/viewcvs/devel/cpuspeed/ Initial Owner: jwilson@redhat.com
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Merge Review: cpuspeed
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225658
------- Additional Comments From ville.skytta@iki.fi 2007-02-01 14:49 EST ------- http://www.redhat.com/archives/fedora-maintainers/2007-January/msg00339.html
"make CFLAGS=..." is a no-op; the Makefile uses COPTS, thus the build ends up doing just "gcc -Wall -fno-exceptions -c -O2 cpuspeed.cc". Just changing CFLAGS to COPTS makes the build fail here on FC6 i386, though.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Merge Review: cpuspeed
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225658
wtogami@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|nobody@fedoraproject.org |jwilson@redhat.com CC| |wtogami@redhat.com Flag| |fedora-review-
------- Additional Comments From wtogami@redhat.com 2007-02-01 15:11 EST ------- Needs work then, assigning to owner before the review can go further. After you fix this issue, please set fedora-review back to BLANK and reassign back to the reviewer.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Merge Review: cpuspeed
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225658
jwilson@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review- |
------- Additional Comments From jwilson@redhat.com 2007-02-04 12:13 EST ------- Okay, I think I *almost* have this one sorted out... I've got the package building on rawhide x86_64 using most of our stock optflags, save -fexceptions, which is the one that was causing the build to fail when switching from CFLAGS= to COPTS= (note that that stock Makefile is set up to pass -fno- exceptions). I've tweaked things a bit further so the -z bits are only passed during linking, so as to suppress some warning messages.
----8<---- + make 'CFLAGS=-O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fno-exceptions -fstack-protector --param=ssp-buffer-size=4 -m64 -mtune=generic -fpie -pie' LDFLAGS=-Wl,-z,relro,-z,now gcc -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fno-exceptions -fstack-protector --param=ssp- buffer-size=4 -m64 -mtune=generic -fpie -pie -c -O2 cpuspeed.cc cpuspeed.cc:121: warning: non-local variable '<anonymous struct> speeds [16]' uses anonymous type cpuspeed.cc: In function 'int main(unsigned int, char**)': cpuspeed.cc:810: warning: ignoring return value of 'int daemon(int, int)', declared with attribute warn_unused_result gcc -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fno-exceptions -fstack-protector --param=ssp- buffer-size=4 -m64 -mtune=generic -fpie -pie -Wl,-z,relro,-z,now cpuspeed.o -o cpuspeed + exit 0 ----8<----
This results in built packages that, at a glance, seem to be functioning correctly. Unfortunately, I'm still winding up with no source in the debuginfo package for some reason that is beyond me at the moment...
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Merge Review: cpuspeed
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225658
jwilson@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|jwilson@redhat.com |nobody@fedoraproject.org
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Merge Review: cpuspeed
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225658
------- Additional Comments From jwilson@redhat.com 2007-02-04 14:22 EST ------- Turns out the lack of source files in my debuginfo is due to my rpm build root's $RPM_BUILD_DIR having a shorter path name than /usr/src/debug, which debugedit (called from find-debuginfo.sh) pukes on. With a suitably longer build root path, source files are indeed being properly included in -debuginfo now.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=225658
Ondrej Vasik ovasik@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |CLOSED CC| |ovasik@redhat.com Resolution| |WONTFIX Last Closed| |2011-08-18 08:10:53
--- Comment #5 from Ondrej Vasik ovasik@redhat.com 2011-08-18 08:10:53 EDT --- as cpuspeed is dead package now, closing WONTFIX...
package-review@lists.fedoraproject.org