[Bug 483615] Review Request: CodeAnalyst - Performance Analysis Suite for AMD-based System (based on Oprofile)

[bugzilla at redhat.com]

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.