On Thu, 21 Mar 2013, Togan Muftuoglu wrote:
On 03/21/2013 09:49 AM, Richard Biener wrote:
On Wed, 20 Mar 2013, Togan Muftuoglu wrote:
The reason is I had a declination with a comment why I have not explained use of %ghost macro in a spec file, but a person who would have reviewed the rpm buildlogs for the package would realize the fact that rpmlint gives the following error.
non-ghost-in-var-run (Badness: 1000)
And the mentioning %ghost in changes file in IMHO is nonsense as the issue has nothing to the with the package but packaging itself and from the end users point of is just cosmetic.
Eh? The .changes file is first and foremost about changes in _packaging_!
Yes and no. If the changes affect the way the program behaves it should be included in the changes, but cosmetic things like fixing rpmlint errors or warnings (not mentioning the rpmlintrc file) that is cosmetic should not clutter the changes file.
Well, the .changes file is the _only_ place where changes in packaging can be documented. Changes in program behavior are documented in its own documentation, on web-pages in packaged README files, in the package description, etc. It's important to document packaging changes. And the .changes file is the place to do that!
Though on a second thought maybe I should have mentioned it in the #sr message yet my expactation was /is one looks the rpmlint logs and understands the logic behind it. But it was pointed out not many people read the logs.
Note that sr messages are not an appropriate way of documenting changes. Eventually the commit log history would be, but that get's seriously messed up and shortened with submits to factory (and of course with the default to just include .changes changes it's redundant as well). Richard. -- Richard Biener <rguenther@suse.de> SUSE / SUSE Labs SUSE LINUX Products GmbH - Nuernberg - AG Nuernberg - HRB 16746 GF: Jeff Hawn, Jennifer Guild, Felix Imend -- To unsubscribe, e-mail: opensuse-packaging+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse-packaging+owner@opensuse.org