[Bug 207896] Review Request: astyle - Source code formatter
bugzilla at redhat.com
bugzilla at redhat.com
Mon Jun 18 14:03:57 UTC 2007
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: astyle - Source code formatter
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=207896
------- Additional Comments From gnome at dux-linux.org 2007-06-18 10:03 EST -------
(In reply to comment #18)
> (In reply to comment #15)
> > > *rpmlint - PASS
> > > Umm??? Don't pass (see below)
> >
> > This is very strange indeed b/c the rpmlint passes
> > for me with flying colors.
> >
> > [a at buildbox ~]$ rpmlint /sw/BUILDING/RPMS/i386/astyle-1.20.2-2.fc7.i386.rpm
> > [a at buildbox ~]$
> Well, when you rebuild srpm (by rpmbuild or mock), you should also
> have debuginfo rpm. For this package, there should be
> astyle-debuginfo rpm. Also, we have to check rpmlint error for
> src.rpm and the _installed_ rpm.
So I need to run rpmlint not only on the primary rpm, but also on the debug
rpms and etc. Thank you for clarifying. :-)
> > > *changelog - CHANGE
> > > -- should remove the last changelog comment
> > about a different version
> > > Why?
> >
> > Why? Because it's stated in the guidelines that changelog comments should
> > match the version of the pkg a person is packaging.
> This is for the *newest* (i.e. top) entry of the %changelog.
Good to know. I was under the wrong impression that versioning needed to
be consistent with the pkg: I stand corrected. Thank you.
> > If she wants to pkg a
> > different version, she should have the comments for that version in its' spec
> > file, and not in this one.
> Well, please try "rpm -q --changelog coreutils", for example.
Good to know. Thank you.
> BTW as this is NEEDSPONSOR blocker, will someone going to sponsor
> the reviewer, Ralf, Kevin (and me)?
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.
More information about the package-review
mailing list