From Fedora Project Wiki

m (add to category Package Maintainers/Review Template)
 
Line 37: Line 37:
- post/postun ldconfig ok <BR>
- post/postun ldconfig ok <BR>
- devel requires base package n-v-r <BR>
- devel requires base package n-v-r <BR>
[[Category:Package Maintainers/Review Template]]

Latest revision as of 17:52, 9 July 2010

This is spot's Review cheat sheet. Basically, its something I can cut and paste, then edit as needed.

What can I say? I'm a lazy bum.

NO DEVEL

=

Good:

- rpmlint checks return:

- package meets naming guidelines
- package meets packaging guidelines
- license ( ) OK, text in %doc, matches source
- spec file legible, in am. english
- source matches upstream
- package compiles on devel (x86)
- no missing BR
- no unnecessary BR
- no locales
- not relocatable
- owns all directories that it creates
- no duplicate files
- permissions ok
- %clean ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc affects runtime
- no need for .desktop file

EXTRA STUFF FOR PACKAGES WITH DEVEL

==========================

- devel package ok
- no .la files
- post/postun ldconfig ok
- devel requires base package n-v-r