[Bug 1086790] Review Request: gnudos - A GNU library to help new users of the GNU system

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1086790



--- Comment #1 from Michael Schwendt <bugs.michael@xxxxxxx> ---
The packaging looks sloppy and half-hearted. :-(

The following does not cover all issues, since there are too many, but one
needs to start somewhere:


> Summary:	The GnuDOS library for GNU/Linux
> Group:		Applications/Productivity

> %description
> GnuDOS is a library of functions [...]
> [...]
> GnuDOS is a group of utilities [...] The tools included in the library
> [...]

The Group tag for library base packages is "System Environment/Libraries".
Either that, or omit the tag, since it's optional nowadays:

https://fedoraproject.org/wiki/Packaging:Guidelines#Group_tag


> %files
> %{_libdir}/*
> %{_bindir}/*
> %{_includedir}/console/*
> %{_mandir}/man1/*
> %{_infodir}/*
> %{_docdir}/*

Library and headers in a single package? That's not how it is done with Red Hat
and Fedora distributions:

  https://fedoraproject.org/wiki/Packaging:Guidelines#Devel_Packages


> %{_includedir}/console/*

  $ repoquery --whatprovides /usr/include/console
  $

https://fedoraproject.org/wiki/Packaging:Guidelines#File_and_Directory_Ownership
https://fedoraproject.org/wiki/Packaging:UnownedDirectories


> %{_infodir}/*

https://fedoraproject.org/wiki/Packaging:ScriptletSnippets#Texinfo


> $ rpmls -p gnudos-1.0-1.fc20.x86_64.rpm |grep ^d
> drwxr-xr-x  /usr/share/doc/fog
> drwxr-xr-x  /usr/share/doc/gnudos
> drwxr-xr-x  /usr/share/doc/mino
> drwxr-xr-x  /usr/share/doc/prime

Not directly a blocker, but it's not nice of an RPM package called "gnudos" to
occupy the doc homedirs "fog", "mino" and "prime" just because it includes
executables with the same names. Typically, the package documentation is
included below %_docdir/%name (i.e. %_pkgdocdir for F20 and newer).


> $ rpmls -p gnudos-1.0-1.fc20.x86_64.rpm |grep lib
> -rw-r--r--  /usr/lib64/libgnudos-1.0.a
> -rwxr-xr-x  /usr/lib64/libgnudos-1.0.la
> lrwxrwxrwx  /usr/lib64/libgnudos-1.0.so
> lrwxrwxrwx  /usr/lib64/libgnudos-1.0.so.1
> -rwxr-xr-x  /usr/lib64/libgnudos-1.0.so.1.0.0

https://fedoraproject.org/wiki/Packaging:Guidelines#Packaging_Static_Libraries


If you don't know the "fedora-review" tool yet, now would be a good opportunity
to use it. Run "fedora-review -b 1086790" to point it at this review ticket. It
evaluates the "SRPM URL:" and "Spec URL:" lines and performs many helpful
checks. The tool also runs rpmlint on all packages, which you should have run
before presenting this package for review.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
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]