Bug 245120 - Review Request: libgsasl - includes support for the SASL framework
Review Request: libgsasl - includes support for the SASL framework
Status: CLOSED ERRATA
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
low Severity medium
: ---
: ---
Assigned To: Patrice Dumas
Fedora Extras Quality Assurance
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2007-06-21 05:16 EDT by Nikolay Vladimirov
Modified: 2011-07-11 14:31 EDT (History)
3 users (show)

See Also:
Fixed In Version: 0.2.18-3.fc7
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2007-07-09 11:49:24 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
pertusus: fedora‑review+
limburgher: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Nikolay Vladimirov 2007-06-21 05:16:19 EDT
Spec URL: http://ns.bgtld.net/build/libgsasl.spec
SRPM URL: http://ns.bgtld.net/build/libgsasl-0.2.18-1.fc7.src.rpm

Description:
The library is without NTLM and KERBEROS_V5 support. 
NTLM support can be included when libntlm( bug  245080 ) is in the repos. 

From the README:
  - KERBEROS_V5 (experimental, requires Shishi)

And also I'm making this package for msmtp( bug 243631) which doesn't use kerberos.
Comment 1 Patrice Dumas 2007-06-22 08:23:46 EDT
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'
Comment 2 Nikolay Vladimirov 2007-06-22 09:06:46 EDT
* Fri Jun 22 2007 Nikolay Vladimirov <nikolay@vladimiroff.com> - 0.2.18-2
- fixed mixed-use-of-spaces-and-tabs
- fixed timestamps for header files 
- edited summary

Spec URL: http://ns.bgtld.net/build/libgsasl.spec
SRPM URL: http://ns.bgtld.net/build/libgsasl-0.2.18-2.fc7.src.rpm


Comment 3 Patrice Dumas 2007-06-22 19:01:34 EDT
* rpmlint is silent
* free software, l icense included
* follow packaging and naming guidelines
* match upstream
d118c5a5a9333411ec3d6cb7973b314a  libgsasl-0.2.18.tar.gz
* sane provides
Provides: libgsasl.so.7
* right %files section
* .so in -devel, no .la, no static lib

APPROVED

I will sponsor you, just apply. That way you can rebuild libgsasl
against libntml, and msmtp against libgsasl.
Comment 4 Patrice Dumas 2007-06-23 03:41:57 EDT
You should be sponsored now.
Comment 5 Nikolay Vladimirov 2007-06-24 16:05:36 EDT
New Package CVS Request
=======================
Package Name: libgsasl
Short Description: GNU SASL library 
Owners: nikolay@vladimiroff.com
Branches: FC-6 F-7
InitialCC: 
Comment 6 Kevin Fenzi 2007-06-25 14:57:15 EDT
cvs done.
Comment 7 Nikolay Vladimirov 2007-06-25 21:54:45 EDT
* Tue Jun 26 2007 Nikolay Vladimirov <nikolay@vladimiroff.com> - 0.2.18-3
- added NTLM support


Spec URL: http://ns.bgtld.net/build/libgsasl.spec
SRPM URL: http://ns.bgtld.net/build/libgsasl-0.2.18-3.fc7.src.rpm

building libntlm succeeded.
Comment 8 Fedora Update System 2007-07-05 15:11:23 EDT
libgsasl-0.2.18-3.fc7 has been pushed to the Fedora 7 testing repository.  If problems still persist, please make note of it in this bug report.
Comment 9 Fedora Update System 2007-07-09 11:49:21 EDT
libgsasl-0.2.18-3.fc7 has been pushed to the Fedora 7 stable repository.  If problems still persist, please make note of it in this bug report.
Comment 10 Nikolay Vladimirov 2009-01-19 06:06:39 EST
Package Change Request
======================
Package Name: libgsasl
New Branches: EL-5
Owners: turki
Comment 11 Kevin Fenzi 2009-01-19 18:24:55 EST
cvs done.
Comment 12 Nikolay Vladimirov 2011-07-11 14:22:08 EDT
Package Change Request
======================
Package Name: libgsasl
New Branches: EL-4
Owners: turki
Comment 13 Jon Ciesla 2011-07-11 14:31:31 EDT
Git done (by process-git-requests).

Note You need to log in before you can comment on or make changes to this bug.