Bug 241834
Summary: | Review Request: online-desktop - Desktop built around web sites and online services | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Colin Walters <walters> |
Component: | Package Review | Assignee: | Matthias Clasen <mclasen> |
Status: | CLOSED RAWHIDE | QA Contact: | Fedora Package Reviews List <fedora-package-review> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | mclasen, petersen |
Target Milestone: | --- | Flags: | mclasen:
fedora-review+
tcallawa: fedora-cvs+ |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2007-07-31 14:04:29 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Colin Walters
2007-05-30 23:30:45 UTC
First impressions: - I question the wisdom of calling the script "godmode", but thats up to you. - Package can't be noarch if it installs stuff in libdir. If it is pure python, it should probably live in datadir - Is somebody packaging python-dbus ? I don't have that on my system, and yum seems unable to find it. On the other hand, I do have dbus-python... - Is the split into tons of subpackages beneficial here ? What is gained ? Also, it seems to counteract the idea of making online-desktop a meta-package that pulls in everything thats needed for GOD. - Is there a bigboard dependency missing ? See previous point godmode: well it will be invoked from a menu entry in the next version, I don't really see it as the primary interface. noarch: under discussion python-dbus: hm, not sure how that typo slipped by; fixed subpackages: the core package is everything needed, but you can install the extra packages for the individual sites you use like gmail/google-reader to add desktop entries for them. We may actually just punt on the whole desktop file thing and generate menu entries inside the bigboard panel. bigboard: fixed New files: http://download.mugshot.org/online-desktop/sources/online-desktop-0.2.1.tar.gz http://download.mugshot.org/online-desktop/sources/online-desktop.spec <mclasen> one thing that the online desktop package needs to do is update the icon cache in %post and %postun Doing the checklist: package builds fine in mock rpmlint has no complaints about any of the packages package name: ok spec file name: ok packaging guidelines: - need to update the icon cache in %post and %postun for each subpackage that installs an icon, see http://fedoraproject.org/wiki/Packaging/ScriptletSnippets - need to install desktop files with desktop-file-install, which will expose that they contain invalid categories, like Application; also categories like X-RedHat-... are not useful anymore and should be removed - I believe the "Online Desktop" menu item should go into Preferences > Look and Feel, next to "Desktop Effects" - The guidelines generally recommend that subpackages should require their mother package. Not sure if that is really necessary here; another idea I had is that you could maybe reduce the overhead of all those subpackages by having a single online-desktop-applications subpackage. To cite the README: "This is all in one big tarball since it would just be annoying to have a little tiny tarball for every web app" - since the yahoo and picasa icons are not used yet, it might be better to %exclude them. license: ok license field: ok license file: ok spec file language: American English spec file legibility: excellent src url: typo, "source" should be "sources" source tarball: ok buildability: ok complete BRs: ok locales: n/a, no translations --> not a review issue, but we need to tackle that for F8 shared libs: n/a relocatable: n/a directory ownership: packages that install icons must require hicolor-icon-theme duplicate files: ok file permissions: ok %clean section: ok macro use: mixture of %{buildroot} and $RPM_BUILD_ROOT - please pick one content: ok large docs: n/a headers: n/a static libs: n/a .pc files: n/a shared libs again: still ok subpackage requires base: mentioned above. any reason not to do this ? .desktop files: need to be installed and made valid, see above file ownership: ok %install target: ok filenames utf8: ok New files uploaded that address all of these issues I believe: http://download.mugshot.org/online-desktop/sources/online-desktop-0.2.4.tar.gz http://download.mugshot.org/online-desktop/sources/online-desktop.spec - You need those %post and %postun for each icon-carrying subpackage :-( - Please add either PersonalSettings or DesktopSettings (which is the "Look and Feel" submenu) to the online-desktop.desktop categories, to keep everything in a submenu there Ok, both of those issues should be fixed: http://download.mugshot.org/online-desktop/sources/online-desktop-0.2.5.tar.gz http://download.mugshot.org/online-desktop/sources/online-desktop.spec Looking good now. Approved New Package CVS Request ======================= Package Name: online-desktop Short Description: Desktop built around web sites and online services Owners: walters Branches: F7 cvs done. |