[Bug 755890] Review Request: Snap A modular cross-platform system backup/restore utility

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #9 from Richard Shaw <hobbes1069@xxxxxxxxx> 2011-11-23 09:23:50 EST ---
(In reply to comment #8)
> > I don't think I'd hold up a review for it, but it's not much work to export a
> > 48x48 icon (png) just to be sure :)
> 
> Done, added the 48x48 icon to the project / spec
> 
> 
> As a side note, unless I'm missing something it doesn't seem like the icons
> macros are defined in F16 and up. I had to declare them myself to get it
> building against rawhide on Koji. Regardless, updated:

Interesting. I never knew there were macros for icons :) Learn something new
every day! 

Though from a readability standpoint, I don't think it's any better than using
%{_datadir}/icons...

Richard

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review



[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]