[Bug 167888] Review Request: polyxmass-common - Contains all textual/graphical data required for polyxmass

[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 report.

Summary: Review Request: polyxmass-common - Contains all textual/graphical data required for polyxmass


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=167888


jpmahowald at gmail.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
OtherBugsDependingO|163778                      |163779
              nThis|                            |




------- Additional Comments From jpmahowald at gmail.com  2005-10-29 21:05 EST -------
With the ownership issue resolved, rpmlint remains:

W: polyxmass-common no-version-in-last-changelog
W: polyxmass-common non-conffile-in-etc /etc/polyxmass/chempad.conf
W: polyxmass-common non-conffile-in-etc
/etc/polyxmass/polchem-defs/polyxmass-common-polchem-defs-atom-defs-dic
W: polyxmass-common non-conffile-in-etc
/etc/polyxmass/polchem-defs/polyxmass-common-polchem-defs-cat
W: polyxmass-common non-conffile-in-etc
/etc/polyxmass/atom-defs/polyxmass-common-atom-defs-cat


chempad.conf is definatly a config file and should be marked %config in the spec.

All the other ones contain pointers to xml data located elsewhere. These can
also be labeled config. However, these file locations aren't likely to change.
I'll let you make the call on this one.

Change at least chempad.conf to %config, and this is APPROVED

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.