[Bug 226468] Merge Review: system-config-samba

[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-samba


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





------- Additional Comments From nphilipp at redhat.com  2007-10-15 05:14 EST -------
(In reply to comment #4)
> 2) Is there any need to use following in SPEC?
>    ExclusiveOS: Linux

Removed.

> 3) I am bit confused whether we really need 
> Requires(post): hicolor-icon-theme
> Requires(postun): hicolor-icon-theme
> 
> because
>
http://fedoraproject.org/wiki/Packaging/ScriptletSnippets#head-7103f6c38d1b5735e8477bdd569ad73ea2c49bda
> suggests no such usage of Requires needed.

removed hicolor-icon-theme requirement, calling gtk-update-icon-cache with full path

system-config-samba-1.2.56-1.fc8 is building with these changes right now


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.