https://bugzilla.redhat.com/show_bug.cgi?id=825489

Volker Fröhlich <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]

--- Comment #5 from Volker Fröhlich <[email protected]> ---
Don't use the rm macro. Just use "rm -rf" like you did below.

You don't need to rm the buildroot in the install section and you don't need to
define defattr in the files sections either.

Please ship license.txt via %doc as well.

Use -p on cp to preserve the timestamp of the header file. But you could also
consider to correct src.pro. In this case you should be able to drop most of
your commands from the install section.

It seems to me, libGLEW is not linked to the libraries. Am I wrong?

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