[Bug 245120] Review Request: libgsasl - includes support for the SASL framework

bugzilla at redhat.com bugzilla at redhat.com
Fri Jun 22 12:23:48 UTC 2007


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libgsasl - includes support for the SASL framework


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


pertusus at free.fr changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Review Request: libgsasl -  |Review Request: libgsasl -
                   |includes support for the    |includes support for the
                   |SASL framework              |SASL framework
                 CC|                            |pertusus at free.fr




------- Additional Comments From pertusus at free.fr  2007-06-22 08:23 EST -------
The summary could be better in my opinion, something like

Summary: GNU Simple Authentication and Security Layer (SASL) library

or simply

Summary: GNU SASL library

rpmlint gives:
W: libgsasl mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 7)

I see only one real issue, the timestamp for the header files
are no kept. I think it should be solved by doing something along:
make install DESTDIR=$RPM_BUILD_ROOT INSTALL='install -p'


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.




More information about the Fedora-package-review mailing list