Bug 639470

Summary: remove samba dependency on perl
Product: [Fedora] Fedora Reporter: Daniel Mach <dmach>
Component: sambaAssignee: Guenther Deschner <gdeschner>
Status: CLOSED CURRENTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: low    
Version: rawhideCC: dpal, gdeschner, jlayton, ssorce
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-02-12 05:05:53 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 471381    

Description Daniel Mach 2010-10-01 20:20:38 UTC
There are perl scripts in /usr/share/doc:
/usr/share/doc/samba-3.5.5/LDAP/convertSambaAccount
/usr/share/doc/samba-3.5.5/LDAP/ol-schema-migrate.pl
/usr/share/doc/samba-3.5.5/misc/adssearch.pl
/usr/share/doc/samba-3.5.5/misc/check_multiple_LDAP_entries.pl
/usr/share/doc/samba-3.5.5/misc/cldap.pl
/usr/share/doc/samba-3.5.5/misc/swat.pl
/usr/share/doc/samba-3.5.5/misc/wall.perl
/usr/share/doc/samba-3.5.5/printer-accounting/acct-sum
/usr/share/doc/samba-3.5.5/printer-accounting/hp5-redir
/usr/share/doc/samba-3.5.5/printer-accounting/lp-acct

These scripts do not work with additional perl modules installed, so I think the perl dependency could be removed completely.

This is probably related to bug 473051.

Comment 1 Fedora Admin XMLRPC Client 2010-10-08 14:43:43 UTC
This package has changed ownership in the Fedora Package Database.  Reassigning to the new owner of this component.

Comment 2 Guenther Deschner 2010-11-01 15:38:21 UTC
Actually we explicitly filter out the dependency to all the other perl modules so that users can easily get the full documentation package (including some perl example scripts) in one package. Once they really have a need for one of the perl scripts figuring out which additional modules are required is easy. I don't think we should change that. Sounds ok ?

Comment 3 Jason Tibbitts 2011-03-21 22:14:25 UTC
I think the point of this ticket is to point out that while the dependencies for the Perl modules are indeed gone, the dependency on /usr/bin/perl is still there.  As far as I can tell, nothing in that package actually needs perl except for those few documentation files, and so as long as you're filtering Perl dependencies, you might as well go all the way and filter /usr/bin/perl too.  

Unfortunately that isn't completely trivial since you can't just add it to the existing filter-requires-samba.sh, though it should be possible using the new RPM dependency filtering mechanism.  I'd provide a patch if only that were documented.

Comment 4 Daniel Mach 2011-03-24 13:39:23 UTC
Sorry for not responding for a while.
I spoke to perl maintainer and she suggested to run 'chmod -x' on all documentation files (which should help) or use following macro:
%filter_provides_in %{_docdir}

see: https://fedoraproject.org/wiki/Packaging:AutoProvidesAndRequiresFiltering

Comment 5 Guenther Deschner 2011-04-06 15:18:08 UTC
We will address this in future fedora

Comment 6 Fedora Update System 2013-01-30 17:30:03 UTC
samba-4.0.2-1.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/samba-4.0.2-1.fc18

Comment 7 Fedora Update System 2013-02-01 16:26:44 UTC
Package samba-4.0.2-1.fc18:
* should fix your issue,
* was pushed to the Fedora 18 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing samba-4.0.2-1.fc18'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2013-1654/samba-4.0.2-1.fc18
then log in and leave karma (feedback).

Comment 8 Fedora Update System 2013-02-12 05:05:59 UTC
samba-4.0.2-1.fc18 has been pushed to the Fedora 18 stable repository.  If problems still persist, please make note of it in this bug report.