[Bug 668243] Review Request: libqb - An IPC library for high performance servers.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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=668243

--- Comment #8 from Fabio Massimo Di Nitto <fdinitto@xxxxxxxxxx> 2011-01-19 03:00:36 EST ---
(In reply to comment #7)
> 2 MUSTFIXES:
> 
> * From my build.log
> ...
> ./check_resources.sh: line 11: ps: command not found
> ...
> => missing BR: /bin/ps

Yes ACK, I think something has been fluxing in the chroot because in my
previous logs I didnÂt have this issue at all. Now it is showing up. Probably
an update or something did change, anyway not worth investigating IMHO. The fix
is simple.

> 
> 
> * Package doesn't honor RPM_OPT_FLAGS.
> 
> The configure script plays nasty games with *FLAGS in a way they overwrite
> Fedora's *FLAGS.
> 
> Excerpt from my build.log:
> ... gcc -DHAVE_CONFIG_H -I. -I../include -I../include -I../include -O2 -g -pipe
> -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector
> --param=ssp-buffer-size=4 -m64 -mtune=generic -O3 -ggdb3 -Wall ....
> 
> Note: ... "RPM_OPT_FLAGS"... -O3 -ggdb3.
> 
> The later overwrite flags from RPM_OPT_FLAGS.
> 
> One way to fix this is to sed out the stuff which is responsible for this from
> configure.ac: e.g. by adding this before autogen.sh:
> 
> sed -i -e 's,OPT_CFLAGS="-O3",OPT_CFLAGS=,' \
>  -e 's,GDB_FLAGS="-ggdb3",GDB_FLAGS=,' configure.ac

I disagree with this approach as policy allow flags override.

http://fedoraproject.org/wiki/Packaging/Guidelines#Compiler_flags

If upstream considers -O3 -ggdb3 better suitable for his code, they should
simply document that.

According to the build.log only "-O2 -g" are overridden.

> Should one of you be upstream, I'd seriously advise you to rework the
> configure.ac and to start making "make dist" working to ship proerly packaged
> tar-balls instead of .git snapshots.

make dist works just fine upstream, what problem are you experiencing exactly?
Random comments without some background information are not very useful.

I think what you miss is that the release script for libqb is capable of
handling both real release and snapshots to guarantee that the correct version
information are propagated across the board.

The srpm/rpm have been checked against a release tarball and they match. As
long as upstream releases and what is shipped in Fedora do match, I doubt it is
FedoraÂs job to enforce any kind of release policy to upstream.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review



[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]