[Bug 483615] Review Request: CodeAnalyst - Performance Analysis Suite for AMD-based System (based on Oprofile)
bugzilla at redhat.com
bugzilla at redhat.com
Wed Mar 4 09:22:24 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=483615
--- Comment #18 from Parag AN(पराग) <panemade at gmail.com> 2009-03-04 04:22:22 EDT ---
* When using "cp" or "install" commands, add "-p" option
to keep timestamps on installed files:
https://fedoraproject.org/wiki/Packaging/Guidelines#Timestamps
* Any reason for not using service command to %preun?
see
https://fedoraproject.org/wiki/Packaging/SysVInitScripts#Initscripts_in_spec_file_scriptlets
* I see many files installed in /usr/share/codeanalyst/doc/cadoc
how about creating new subpackage as given at
http://fedoraproject.org/wiki/PackagingGuidelines#Documentation
--
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 Fedora-package-review
mailing list