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=759057 --- Comment #6 from Julian Sikorski <belegdol@xxxxxxxxx> 2012-01-03 13:43:51 EST --- I am not sure how you got these warnings: [julas@snowball2 SPECS]$ rpm -q SOIL-devel SOIL-devel-1.07-3.20080706.fc16.x86_64 [julas@snowball2 SPECS]$ rpmlint SOIL-devel SOIL-devel.x86_64: W: no-documentation 1 packages and 0 specfiles checked; 0 errors, 1 warnings. [julas@snowball2 SPECS]$ ls -l /usr/include/SOIL razem 64 -rw-r--r--. 1 root root 3212 2008-07-07 image_DXT.h -rw-r--r--. 1 root root 2287 2008-07-07 image_helper.h -rw-r--r--. 1 root root 15545 2008-07-07 SOIL.h -rw-r--r--. 1 root root 15447 2008-07-07 stbi_DDS_aug_c.h -rw-r--r--. 1 root root 797 2008-07-07 stbi_DDS_aug.h -rw-r--r--. 1 root root 16945 2008-07-07 stb_image_aug.h I think it is a side-effect of your sed replacement, -m 644 got missing somehow. Are you 100 % positive that overriding the makefile variables is better than sed? A proper patch would be the best, but it is impossible for macros encoding arch-dependent paths. To me it seems to be the matter of personal preference - I find the sed overrides more legible since they change one thing at a time. -- 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