Bug 483277
Summary: | Review Request: nautilus-gdu - Nautilus extension for disk formatting | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Tomáš Bžatek <tbzatek> |
Component: | Package Review | Assignee: | Tomáš Bžatek <tbzatek> |
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | fedora-package-review, mclasen, notting, tsmetana |
Target Milestone: | --- | Flags: | mclasen:
fedora-review+
huzaifas: 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: | 2009-02-04 13:15:16 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
Tomáš Bžatek
2009-01-30 17:07:29 UTC
Package builds fine in mock. rpmlint output: 3 packages and 0 specfiles checked; 0 errors, 0 warnings. requires/provides/file list look sane package name: ok spec file name: ok packaging guidelines: ok license: ok license field: ok license file: ok spec file language: ok spec file legible: ok sources: you should probably add a comment explaining how to get the tarball from git. longer-term, we need to sort out hosting buildable: ok excludearch: ok build deps: ok locale handling: ok, no translations right now. sorting that out ties in to the hosting question... ldconfig: I don't think you need to call ldconfig in %post/%postun, since you are not installing a shared library in the linker path, just a loadable module relocatable: n/a directory ownership: ok duplicate files: ok permissions: ok %clean: ok macro use: ok permissible content: ok large docs: n/a %doc content: ok headers: n/a static libs: n/a pc files: ok shared libs: ok devel subpackage: n/a libtool archives: ok gui apps: ok file ownership: ok %install: ok utf8 filenames: ok Summary: - add source comment - remove unnecessary %post/%postun Thanks for the review. I've added a comment how to get sources from git (by cloning the repo). Fixed files: Spec URL: http://tbzatek.fedorapeople.org/nautilus-gdu/nautilus-gdu.spec SRPM URL: http://tbzatek.fedorapeople.org/nautilus-gdu/nautilus-gdu-0.1-git20090130.1.fc11.src.rpm looks good now. approved New Package CVS Request ======================= Package Name: nautilus-gdu Short Description: Nautilus extension for disk formatting Owners: tbzatek Branches: devel InitialCC: cvs done Successfully built in rawhide: http://koji.fedoraproject.org/koji/buildinfo?buildID=81493 |