[Bug 226453] Merge Review: system-config-boot

[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: Merge Review: system-config-boot


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





------- Additional Comments From kevin at tummy.com  2007-03-23 23:12 EST -------
>> 3. I assume the reason it only builds on ix86/x86_64 is that it only understands
>> lilo/grub? Might be worth filing a bug and noting it in the spec and see if
>> some ppc folk are interested in contributing yaboot support.
>
>what about sparc, s390 and the others?

Sure. I suppose there could be one bug about "non x86/x86_64" support?

>> 6. The desktop file is missing a valid Main Category, see:
>> http://standards.freedesktop.org/menu-spec/latest/apa.html
>> Suggest: System or Settings be added.
>> Without this, this tool shows up under a "Other" menu in Xfce.
>
>Categories=System;Application;SystemSetup;X-Red-Hat-Base;

Humm...my mistake. I thought it didn't have "System" in there.

>> g)
>> W: system-config-boot macro-in-%changelog dist
>> W: system-config-boot macro-in-%changelog dist
>>
>> Suggest: Change occurances of %dist in the changelog with %%dist
>
>done

I still see dist in the changelog. Looking further at it, I think
they shouldn't be there at all. You don't need to have them in the
version string per release.

Thanks for the quick fixes!


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