Red Hat Bugzilla – Bug 469673
Review request: perl-File-ShareDir-PAR - same functionality as File::ShareDir with PAR support
Last modified: 2008-11-24 03:14:20 EST
File::ShareDir::PAR provides the same functionality as File::ShareDir but tries hard to be compatible with PAR packaged applications. The problem is, that the concept of having a distribution or module specific share directory becomes a little hazy when you're loading everything from a single file. PAR uses an @INC hook to intercept any attempt to load a module. File::ShareDir uses the directory structure that is typically found in the directories that are listed in @INC for storing the shared data. In a PAR enviroment, this is not necessarily possible.
We have scripts that parse these review requests. I ask again: please use the template or at least use the same format as other review tickets.
Jason, I suppose you mean
Yes, or simply use the same format. Isn't the goal to have the packages reviewed quickly? I can't imagine why someone would want to differ needlessly, since it only complicates things for any automated processes we might have and thus slows things down.
OK source files match upstream:
OK package meets naming and versioning guidelines.
OK specfile is properly named, is cleanly written and uses macros consistently.
OK dist tag is present.
OK build root is correct.
OK license field matches the actual license.
OK license is open source-compatible.
OK license text included in package.
OK latest version is being packaged.
OK BuildRequires are proper.
OK %clean is present.
OK package builds in mock.
OK package installs properly.
OK no debuginfo package
OK rpmlint is silent.
BAD final provides and requires are sane -- see below
OK %check is present and all tests pass:
All tests successful.
Files=3, Tests=60, 2 wallclock secs
OK no shared libraries
OK owns the directories it creates.
BAD owns directories created by perl-File-ShareDir; please replace
OK no duplicates in %files.
OK file permissions are appropriate.
OK no scriptlets present.
OK code, not content.
OK documentation is small, so no -docs subpackage is necessary.
OK %docs are not necessary for the proper functioning of the package.
OK no headers. no pkgconfig files. no libtool .la droppings.
OK no desktop files
There are three redundant manual requires:
Requires: perl(Class::Inspector) >= 1.12
Requires: perl(Params::Util) >= 0.07
But all there requires are also detected automatically.
To add the version requirements, I suggest to patch PAR.rpm this way:
sed -i "
s/^use Params::Util *'/use Params::Util 0.07 '/
s/^use Class::Inspector *()/use Class::Inspector 1.12 ()/
Please remember to set the fedora-review flag when you're reviewing a package.
Please see new version
I apologize for wrong format. I was using what firefox offers in history :(
The above changes has been incorporated, except the sed command patching
File/ShareDir/PAR.pm, which really belongs upstream, not to the patch.
New Package CVS Request
Package Name: perl-File-ShareDir-PAR
Short Description: same functionality as File::ShareDir (compatible with PAR)
Branches: devel, F-10