[Bug 522935] Review Request: mod_gnutls - GnuTLS module for the Apache HTTP server

[bugzilla at 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=522935


Erick Calder <e at arix.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|needinfo?(e at arix.com)       |




--- Comment #8 from Erick Calder <e at arix.com>  2009-09-17 15:57:30 EDT ---
(In reply to comment #7)
> I'd suggest using %{name}-%{version}.tar.bz2 here.

I should have thought of that :)

> There is missing 'Requires: httpd%{?_isa}' otherwise the package could be
> installed into unowned directories and it also obviously would not be working
> without httpd.  

well... I didn't add a Requires field because the manuals say not to; that one
should rely on the dependency generator, although it seems that is a little
lacking since it missed httpd (it did pick up on the need for GnuTLS):

config(mod_gnutls) = 0.5.5-3.fc11
libc.so.6  
libc.so.6(GLIBC_2.0)  
libc.so.6(GLIBC_2.1)  
libc.so.6(GLIBC_2.1.3)  
libc.so.6(GLIBC_2.4)  
libgnutls.so.26  
libgnutls.so.26(GNUTLS_1_4)  
libpthread.so.0  
libpthread.so.0(GLIBC_2.0)  
rpmlib(CompressedFileNames) <= 3.0.4-1
rpmlib(FileDigests) <= 4.6.0-1
rpmlib(PayloadFilesHavePrefix) <= 4.0-1
rtld(GNU_HASH) 

so I've added it.  thanks for picking up on that.  latest version is -4 (see
usual place for it)

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