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

--- Comment #16 from Michal Schmidt <[email protected]> 2011-03-31 14:01:20 
EDT ---
Well, I don't think you need to copy Kevin's fragment exactly. I linked it to
provide inspiration. For instance, I don't think you really need to add
-Wno-non-virtual-dtor. And there's no pkg-config usage in this project, so I
don't see any value in PKGCONFIG_CFLAGS either.

And did you really need to switch to calling qmake-qt4 instead of ./configure?
I believe that when the patch is applied, this should work fine:
CXXFLAGS="%{optflags}" ./configure --prefix...
But I do not really understand what difference it makes, so if you think
calling qmake-qt4 is better, that's fine with me.

-- 
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
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to