Red Hat Bugzilla – Bug 167381
Review request: perl-File-NCopy - Copy file, file Copy file[s] | dir[s], dir
Last modified: 2007-11-30 17:11:12 EST
Spec Name or Url:
SRPM Name or Url:
File::NCopy copies files to directories, or a single file to another
file. The functionality is very similar to cp.
- rpmlint clean
- package and spec file name OK
- package meets guidelines
- license is same as perl, matches upstream
- spec file written in English and is legible
- sources match upstream
- builds OK in mock on FC4 (i386)
- BRs OK
- no locales, libraries, subpackages or pkgconfigs to worry about
- not relocatable
- no directory ownership or permissions problems
- no duplicate files
- %clean section present and correct
- code, not content
- no large docs
- docs don't affect runtime
- no scriptlets
- URL for Source0 is 404-compliant
- license text not included
- Summary could be clearer I think; how about:
Copy files to directories, or a single file to another file
Created attachment 118390 [details]
Patch addressing review issues
(In reply to comment #1)
> - URL for Source0 is 404-compliant
What kind of superfluous kind of convention are you trying to invent now?
> - Summary could be clearer I think; how about:
> Copy files to directories, or a single file to another file
This is the original dist description from
(In reply to comment #3)
> (In reply to comment #1)
> > - URL for Source0 is 404-compliant
> What kind of superfluous kind of convention are you trying to invent now?
$ spectool --gf perl-File-NCopy.spec
Resolving www.cpan.org... 22.214.171.124
Connecting to www.cpan.org[126.96.36.199]:80... connected.
HTTP request sent, awaiting response... 404 Not Found
13:44:12 ERROR 404: Not Found.
> > - Summary could be clearer I think; how about:
> > Copy files to directories, or a single file to another file
> This is the original dist description from
I know it is. That doesn't make it a clear description of what the package is
for. I believe my suggestion is much clearer, but it's only a suggestion; feel
free to use the original text if you prefer.
This link works
It appears to be a missing or a broken symbolic link. The tarballs in CPAN
are all located under the authors directory tree. All others references
(e.g. from the modules directory) are symbolic links.
Updates to appear soon at:
License text still missing, no changelog entry.
To get the package approved just add following line
"# For license text(s), see the perl package."
to the %files section (and the changelog entry).
--- perl-File-NCopy.spec.2 2005-09-02 14:50:15.000000000 +0100
+++ perl-File-NCopy.spec 2005-09-10 02:38:20.000000000 +0100
@@ -37,6 +37,7 @@
+# For license text(s), see the perl package.
Since the license text policy has now changed, this package meets the guidelines
and is Approved.
Please fix the changelog entry though to make the release consistent, and
s/www/search/ on the Source0 URL.
Aargh. Forget the bit about Source0; the current URL works fine.