Bug 225811
Summary: | Merge Review: gnome-applet-vm | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Nobody's working on this, feel free to take it <nobody> | ||||||
Component: | Package Review | Assignee: | Richard W.M. Jones <rjones> | ||||||
Status: | CLOSED RAWHIDE | QA Contact: | Fedora Package Reviews List <fedora-package-review> | ||||||
Severity: | medium | Docs Contact: | |||||||
Priority: | medium | ||||||||
Version: | rawhide | CC: | berrange, crobinso, kzak, rjones, virt-maint | ||||||
Target Milestone: | --- | Flags: | rjones:
fedora-review+
|
||||||
Target Release: | --- | ||||||||
Hardware: | All | ||||||||
OS: | Linux | ||||||||
Whiteboard: | |||||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||||
Doc Text: | Story Points: | --- | |||||||
Clone Of: | Environment: | ||||||||
Last Closed: | 2009-02-12 12:41:09 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: | |||||||||
Attachments: |
|
Description
Nobody's working on this, feel free to take it
2007-01-31 18:50:26 UTC
Taking for review. gnome-applet-vm.x86_64: W: non-conffile-in-etc /etc/gconf/schemas/vm-applet.schemas gnome-applet-vm.x86_64: E: explicit-lib-dependency libvirt gnome-applet-vm.x86_64: W: conffile-without-noreplace-flag /etc/pam.d/vm_applet_wrapper gnome-applet-vm.x86_64: W: conffile-without-noreplace-flag /etc/security/console.apps/vm_applet_wrapper gnome-applet-vm.x86_64: W: one-line-command-in-%postun /sbin/ldconfig 3 packages and 0 specfiles checked; 1 errors, 4 warnings. Frankly, I want to orphan this package... it means that we need upstream maintainer too. I have already asked on many places, but without any result ;-( Daniel, any idea? Maybe someone around virtualization in RH... kzak: If you want, orphan this & I will take it. Created attachment 331595 [details] Proposed patch to gnome-applet-vm.spec With the attached patch, we are left with this single rpmlint warning: > gnome-applet-vm.x86_64: W: non-conffile-in-etc /etc/gconf/schemas/vm-applet.schemas I think this is one we can ignore, since this looks like the place where gconf expects these schemata to be installed. + rpmlint output See comment 5. + package name satisfies the packaging naming guidelines + specfile name matches the package base name + package should satisfy packaging guidelines + license meets guidelines and is acceptable to Fedora "GPL+" is an acceptable license, and was set by spot, so assume this is correct. + license matches the actual package license + %doc includes license file + spec file written in American English + spec file is legible + upstream sources match sources in the srpm + package successfully builds on at least one architecture + ExcludeArch bugs filed Originally had ExclusiveArch i386, x86-64, but should build everywhere so I removed this. + BuildRequires list all build dependencies + %find_lang instead of %{_datadir}/locale/* + binary RPM with shared library files must call ldconfig in %post and %postun Package called ldconfig, but contains no libraries, so I removed that. + does not use Prefix: /usr + package owns all directories it creates Originally tried to own /usr/share/pixmaps, but I fixed that. + no duplicate files in %files + %defattr line + %clean contains rm -rf $RPM_BUILD_ROOT + consistent use of macros + package must contain code or permissible content n/a large documentation files should go in -doc subpackage + files marked %doc should not affect package n/a header files should be in -devel n/a static libraries should be in -static n/a packages containing pkgconfig (.pc) files need 'Requires: pkgconfig' n/a libfoo.so must go in -devel n/a -devel must require the fully versioned base n/a packages should not contain libtool .la files n/a packages containing GUI apps must include %{name}.desktop file I think this is not applicable because although this is a "GUI app", it is a Gnome panel plugin so doesn't need a desktop file. + packages must not own files or directories owned by other packages + %install must start with rm -rf %{buildroot} etc. + filenames must be valid UTF-8 Optional: + if there is no license file, packager should query upstream n/a translations of description and summary for non-English languages, if available + reviewer should build the package in mock + the package should build into binary RPMs on all supported architectures - review should test the package functions as described + scriptlets should be sane n/a pkgconfig files should go in -devel + shouldn't have file dependencies outside /etc /bin /sbin /usr/bin or /usr/sbin Created attachment 331596 [details]
Proposed patch to gnome-applet-vm.spec
Updated specfile patch.
Cole, can you take a quick look at the proposed changes to the specfile in comment 7, and if you approve, say yea. All the changes look good. What is the remaining process here? Do I need to apply the patch for all supported branches or is rawhide sufficient? Can the bug be closed after the patch is applied? (In reply to comment #9) > All the changes look good. > > What is the remaining process here? Do I need to apply the patch for all only in rawhide > supported branches or is rawhide sufficient? Can the bug be closed after the > patch is applied? depends on who is the owner and who is the reviewer, after reviewer will say "APPROVED" then the owner can close OK I've applied the patch in comment 7. ---------------------------------- This package is APPROVED by rjones ---------------------------------- |