Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226008
Summary: Merge Review: libgcrypt Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: nobody@fedoraproject.org QAContact: fedora-package-review@redhat.com CC: nalin@redhat.com
Fedora Merge Review: libgcrypt
http://cvs.fedora.redhat.com/viewcvs/devel/libgcrypt/ Initial Owner: nalin@redhat.com
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=226008
Miloslav Trmač mitr@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |mitr@redhat.com AssignedTo|nobody@fedoraproject.org |mitr@redhat.com Flag| |fedora-review?
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=226008
Miloslav Trmač mitr@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED
--- Comment #1 from Miloslav Trmač mitr@redhat.com 2010-02-02 19:08:33 EST --- rpmlint:
libgcrypt.src: W: strange-permission hobble-libgcrypt 0755 libgcrypt.x86_64: W: no-documentation
Include license text at the very minimum - see below; including AUTHORS ChangeLog NEWS README (?) THANKS TODO wouldn't hurt either.
libgcrypt.x86_64: W: hidden-file-or-dir /lib64/.libgcrypt.so.11.hmac
See recent discussion on fedora-devel
* libgcrypt-devel contains files under GPLv2+: gcrypt.{texi,info},dumpsexp.c => use "License: GPLv2+", or perhaps better "License: GPLv2+ and LGPLv2+", or split the package
* libgcrypt should contain %doc COPYING.LIB, libgcrypt-devel %doc COPYING
* "The BuildRoot value MUST be below %{_tmppath}/ and MUST contain at least %{name}, %{version} and %{release}:" - %release is not used
* Can you avoid %makeinstall? src/Makefile.in seems to support DESTDIR.
* Use %global instead of %define
* https://fedoraproject.org/wiki/Packaging/Guidelines#All_patches_should_have_...
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=226008
--- Comment #2 from Tomas Mraz tmraz@redhat.com 2010-02-03 03:36:45 EST --- (In reply to comment #1)
rpmlint:
libgcrypt.src: W: strange-permission hobble-libgcrypt 0755 libgcrypt.x86_64: W: no-documentation
Include license text at the very minimum - see below; including AUTHORS ChangeLog NEWS README (?) THANKS TODO wouldn't hurt either.
Fixed. (I did not include ChangeLog and README as they are not much useful.)
libgcrypt.x86_64: W: hidden-file-or-dir /lib64/.libgcrypt.so.11.hmac
See recent discussion on fedora-devel
Technically this file (in contrary to the .hmac files in bin dirs) does not break the FHS. It is still not completely decided where this file should live in case of libgcrypt as the library is not in %{_libdir} but in /%{_lib}
- libgcrypt-devel contains files under GPLv2+: gcrypt.{texi,info},dumpsexp.c => use "License: GPLv2+", or perhaps better "License: GPLv2+ and LGPLv2+", or split the package
Fixed.
- libgcrypt should contain %doc COPYING.LIB,
Fixed.
libgcrypt-devel %doc COPYING
Fixed.
- "The BuildRoot value MUST be below %{_tmppath}/ and MUST contain at least
%{name}, %{version} and %{release}:" - %release is not used
Fixed.
- Can you avoid %makeinstall? src/Makefile.in seems to support DESTDIR.
Fixed.
- Use %global instead of %define
Not fixed. This is not a MUST and here the %define works fine and will work fine in future rpm versions as well.
https://fedoraproject.org/wiki/Packaging/Guidelines#All_patches_should_have_...
I've added a comment.
Here is a new build with the problems fixed. http://koji.fedoraproject.org/koji/taskinfo?taskID=1960493
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=226008
Miloslav Trmač mitr@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+
--- Comment #3 from Miloslav Trmač mitr@redhat.com 2010-02-03 08:17:05 EST --- Thanks, approved; I expect the *.hmac file location will eventually be updated after the final location is clear.
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=226008
Tomas Mraz tmraz@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |RAWHIDE
--- Comment #4 from Tomas Mraz tmraz@redhat.com 2010-02-03 08:55:49 EST --- Yes, thanks.
package-review@lists.fedoraproject.org