Bug 1714255 - Perl packaging mistakes
Summary: Perl packaging mistakes
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: eiskaltdcpp
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Vasiliy Glazov
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2019-05-27 13:59 UTC by Petr Pisar
Modified: 2019-05-30 06:38 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2019-05-28 07:06:51 UTC
Type: Bug
Embargoed:

Attachments (Terms of Use)

Description Petr Pisar 2019-05-27 13:59:16 UTC 
eiskaltdcpp-0:2.2.11-20.20181221gita1fdeb8.fc30 delivers /usr/bin/eiskaltdcpp-cli-xmlrpc that loads various Perl modules at run time. These modules must be declared as dependencies on an RPM level.

Please add:

BuildRequires: perl-generators

to eiskaltdcpp.spec. That will ensure the dependencies are scanned and added when building the package.


Also the package provides "perl(cli-xmlrpc-config.pl) = 0.1". Please remove the provide. perl() name space is reserved for Perl modules installed into standard Perl search paths (/usr/{share,lib*}/perl5). This is not the case.
Comment 1 Vasiliy Glazov 2019-05-28 07:06:51 UTC 
Done in rawhide.
Comment 2 nucleo 2019-05-29 07:12:02 UTC 
Maybe new subpackage with perl deps can be created?
Comment 3 Vasiliy Glazov 2019-05-29 07:15:53 UTC 
I can create eiskaltdcpp xmlrpc with files:

/usr/bin/eiskaltdcpp-cli-xmlrpc
/usr/share/man/man1/eiskaltdcpp-cli-xmlrpc.1.gz
Comment 4 nucleo 2019-05-29 07:31:21 UTC 
Is it optional (required by main program)?
Comment 5 Vasiliy Glazov 2019-05-29 07:34:29 UTC 
I don't know.
You can check it.
Comment 6 nucleo 2019-05-29 20:22:22 UTC 
Looks like eiskaltdcpp-cli-xmlrpc is daemon related application, eiskaltdcpp-qt works if I remove both eiskaltdcpp-cli-xmlrpc and eiskaltdcpp-daemon.
Comment 7 Vasiliy Glazov 2019-05-30 06:38:08 UTC 
xmlrpc with perl deps moved to subpackage.
Note You need to log in before you can comment on or make changes to this bug.