[Bug 546301] Review Request: moblin-app-installer - Moblin Application Installer

[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=546301


Christoph Wickert <cwickert@xxxxxxxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-review?              |fedora-review+




--- Comment #6 from Christoph Wickert <cwickert@xxxxxxxxxxxxxxxxx>  2010-01-06 22:37:16 EDT ---
I agree with you in both points:
1. Lets work with a reduced applist to get this package reviewed. We can still
make a package for the applist later if necessary.
2. For the icons I think a generic icon is best because the current "trick"
with the symling will not work reliable. It will only show icons of installed
apps.

Best would be to link to /usr/share/icons/hicolor/48x48/apps/ (because it
contains the actual application icons) and fall back to a generic icon if an
icon is not found. Not sure how much work this is and I don't consider this a
blocker, because the installer will also work with no icons at all.

The new applist is fine, all blockers are fixed, the package is APPROVED.

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

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

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