Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 1075218 - Review Request: pam_ldap2krb - password migration tool ldap to kerberos
Summary: Review Request: pam_ldap2krb - password migration tool ldap to kerberos
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: FE-NEEDSPONSOR
TreeView+ depends on / blocked
 
Reported: 2014-03-11 18:23 UTC by Kazım SARIKAYA
Modified: 2019-11-26 02:21 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2019-11-26 02:21:21 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Kazım SARIKAYA 2014-03-11 18:23:24 UTC
Spec URL: https://git.sanaldiyar.com/gitweb.cgi/pam_ldap2krb.git/blob/HEAD:/pam_ldap2krb.spec
SRPM URL: http://srcrepo.sanaldiyar.com/repoview/pam_ldap2krb.html
Description: Password migration pam module for ldap to ldap+kerberos authentication system.
Passwords will be created in kerberos if user name and password are validated with
existing ldap server. A configuration file is  /etc/pam_ldap2krb.conf

extended description is here: https://git.sanaldiyar.com/gitweb.cgi/pam_ldap2krb.git/blob_plain/HEAD:/README

Fedora Account System Username: kazimsarikaya

Koji Build: https://koji.fedoraproject.org/koji/taskinfo?taskID=6621924

Hi, I am Kazim. It is my first package for contributing fedora. Hence i need a sponsor. 

Thanks for all.

Comment 1 Kazım SARIKAYA 2014-03-11 18:58:50 UTC
A rawhide koji build is also available: http://koji.fedoraproject.org/koji/taskinfo?taskID=6622780

Comment 2 Igor Gnatenko 2014-03-12 14:26:51 UTC
Some quick notes from first glance:

* make install DESTDIR=%{buildroot}
  would be %make_install

* Source0:        %{name}-%{version}.tar.gz
  I'd prefer if you will provide link to upstream releases on ftp or http or git

* A configuration file is  /etc/pam_ldap2krb.conf
  Should not be in description. Should be in README which installing to system.

* Group:          security
  Should not present if you have no plans for supporting you package for EL5 or EL6 (forget)

* %{_sysconfdir}/*
  not good. please provide %{_sysconfdir}/pam_ldap2krb.conf

* %{_libdir}/*
  not good. please provide something like upper.

* Tue Mar 11 2014 Kazım SARIKAYA <kazimsarikaya>
  there should be also version of package.

* Requires:       pam libconfuse krb5-workstation openldap
  I'm not sure that we really want to have this dependencies. Some from this should automatically be picked up by RPM.

Comment 3 Igor Gnatenko 2014-03-12 15:13:41 UTC
Also no ldconfig macroses.
https://fedoraproject.org/wiki/Packaging:ScriptletSnippets#Shared_libraries

Comment 4 Kazım SARIKAYA 2014-03-12 21:09:19 UTC
i fixed my spec file:

* using %make_install
* adding scm information before Source0 described at https://fedoraproject.org/wiki/Packaging:SourceURL#Using_Revision_Control
* comment group (i have my own repo for my servers and i use group keyword, however for fedora i can comment it )
* rewrite description
* %files section is complete file names not wildcard
* %changelog adding version
* commenting requires 
* adding ldconfig macros.

i intend to add a new koji build task link however. the koji is painfully slow and gives timeout error. hence i think i should not wait it for now.

the rpms can be generated with steps:

1) git clone https://git.sanaldiyar.com/pam_ldap2krb.git
2) ./autogen.sh
3) ./configure
4) make rpmbuild

Comment 5 Igor Gnatenko 2014-03-13 02:51:56 UTC
Please provide direct links to spec and srpm here.

Comment 7 Igor Gnatenko 2014-03-13 04:50:07 UTC
(In reply to Kazım SARIKAYA from comment #4)
> i fixed my spec file:
> 
> * comment group (i have my own repo for my servers and i use group keyword,
> however for fedora i can comment it )
> * commenting requires 
Commenting is not good way. please delete this lines.

About requires I'm not sure from start. Feel free re-check it.

Comment 8 Kazım SARIKAYA 2014-03-13 07:48:38 UTC
i remove everything about group and required. 
from the fedora packaging wiki, they mention that using required is necessary if the version of the required package is important. for my package it is not important.

the locations of srcrpm and spec do not changed. 

here the koji builds:
f20: http://koji.fedoraproject.org/koji/taskinfo?taskID=6628211
rawhide: http://koji.fedoraproject.org/koji/taskinfo?taskID=6628236

Comment 9 Igor Gnatenko 2014-03-13 08:16:56 UTC
* Tue Mar 11 2014 Kazım SARIKAYA <kazimsarikaya> - 0.1

There should be %{version}-%{release}, i.e. in you case now it should be 0.1-1

%post
/sbin/ldconfig
%postun
/sbin/ldconfig

Replace with %post -p /sbin/ldconfig
the same with postun.

That's all, how I can help you. Find sponsor :)

Comment 12 Michael Schwendt 2014-03-29 21:59:16 UTC
> License:	GPLv3

https://www.gnu.org/licenses/gpl-faq.html#NoticeInSourceFile


> %post -p /sbin/ldconfig
> %postun -p /sbin/ldconfig

These scriptlets are not needed, because the package does not store any shared libraries in runtime linker's search path. The PAM Modules are located within a private path.


%files
%doc README
%{_sysconfdir}/pam_ldap2krb.conf

These two apply here:

  https://fedoraproject.org/wiki/Packaging:Guidelines#Configuration_files
  https://fedoraproject.org/wiki/Packaging:LicensingGuidelines#License_Text

Comment 13 Kazım SARIKAYA 2014-04-03 11:38:06 UTC
* I added license info to source file
* delete post and postun scripts
* add copying to %doc
* add %config at begining of config file line

spec: http://projects.sanaldiyar.com/data/pam_ldap2krb/pam_ldap2krb.spec
srpm: http://projects.sanaldiyar.com/data/pam_ldap2krb/pam_ldap2krb-0.1-1.src.rpm
koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=6702421

Comment 14 Igor Gnatenko 2016-07-07 09:55:29 UTC
guidelines were updated, so notes:
* Use %license LICENSE
* use %make_build instead of make %{?_smp_mflags}
* Add BuildRequires: gcc

What's your FAS username?

Comment 15 Igor Gnatenko 2016-11-14 17:10:44 UTC
ping?

Comment 16 Kazım SARIKAYA 2019-11-26 02:21:21 UTC
i stopped development of this package.


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