[Bug 478852] Review Request: lpairs - Classical memory game with cards
bugzilla at redhat.com
bugzilla at redhat.com
Tue Mar 3 19:30:17 UTC 2009
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=478852
--- Comment #3 from Marcin Zajaczkowski <mszpak at wp.pl> 2009-03-03 14:30:16 EDT ---
Thanks guys for your suggestion.
I made new SPEC with following changes:
- replaced %makeinstall
- added desktop-file-utils to BuildRequires
- removed "--vendor fedora"
- removed update-desktop-database
- removed SDL from Requires
http://timeoff.wsisiz.edu.pl/rpms/lpairs/lpairs.spec
http://timeoff.wsisiz.edu.pl/rpms/lpairs/lpairs-1.0.4-3.src.rpm
> %defattr(-,root,root,-) won't work? Why?
Probably works. I have never used them. I started using RPMs when original
packages were made only by RedHat and there were many user pages with packages.
I was safer in my opinion to force mentioned rights (especially for suided
files).
Are there situations where (-,root,root,-) is better? Automatic builds uses
(0644,root,root,0755) anyway.
> * desktop-file-install warns about the Icon= entry
Probably I have too old desktop-file-install (Fedora 8), but I didn't see any
warning. Is it about .png extension?
If yes I would have to patch original file or create a new one. Is it worth to
do that?
--
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.
More information about the package-review
mailing list