[Bug 659756] Review Request: libnfsidmap - Library that handles mapping between names and ids for NFSv4.

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

--- Comment #2 from Steve Whitehouse <swhiteho@xxxxxxxxxx> 2010-12-03 11:35:43 EST ---
MUST items:

Package name: ok
Spec file name: ok
Package Guidelines:
 - Spec is readable
 - Arch support (>1 arch)
 - Meets FHS
 - Change log format is correct
 - Tags ok
 - Requires/deps ok
 - Build requires ok
 - Summary & Description ok
 - Docs ok
 - Devel pkg content ok
 - post/postun ok
 - Uses mix of ${RPM_BUILD_ROOT} and %{buildroot}
   * Should use one of the other style consistently as per:
     https://fedoraproject.org/wiki/Packaging/Guidelines
     "Using %{buildroot} and %{optflags} vs $RPM_BUILD_ROOT and $RPM_OPT_FLAGS"
 - Package appears to use gettext, but there is no find_lang macro call
     https://fedoraproject.org/wiki/Packaging/Guidelines
     "Handling Locale Files"
 - Is is possible to use parallel make?
     https://fedoraproject.org/wiki/Packaging/Guidelines
     "Parallel make"
License: ok
Spec file in english: ok
Spec file legible: ok
Spec file successfully compiles
Locale handling: see comment above
System lib check: ok
Relocation: N/A
Owns all directories: ok
Lists files only once in %files: ok
Consistent use of macros: see above comment on buildroot
Must contain code or permissible content: ok
%doc doesn't affect runtime: ok
Header files must be in -devel: ok
.so in devel: ok
Versioned dep for -devel: ok
Must not contain .la archives: ok
Must not own files/dirs owned by other packages: ok
All filenames must be UTF-8: ok


Thats all the must items, should items coming up shortly.

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