https://bugzilla.redhat.com/show_bug.cgi?id=829865

--- Comment #4 from Dan Scott <[email protected]> ---
Thanks, BuildRequires: perl(ExtUtils::MakeMaker) added.

The permissions problem is indeed weird, and actually matches what's happening
in perl-MARC-Record. If you remove the %attr specification for %{_bindir} here,
you get:

perl-MARC-XML.noarch: E: non-standard-executable-perm /usr/bin/xml2marc 0555L

(This is weird because ExtUtil::MakeMaker says that the default permission for
EXE_FILES is supposed to be 755, and I don't see anything overriding that
default in Makefile.PL for this module).

Over in perl-MARC-Record, the solution I inherited had been to "chmod -R u+w
$RPM_BUILD_ROOT/*". That seemed a bit bunt, which is why I went with the %attr
approach to resolve the problem specifically with the contents of %{_bindir}.

That said, I agree that %attr{755,-,-} is much cleaner than a mix of commas and
spaces, and I've made that change.

Spec URL: http://bzr.coffeecode.net/scratch/perl-MARC-XML/perl-MARC-XML.spec
SRPM URL:
http://bzr.coffeecode.net/scratch/perl-MARC-XML/perl-MARC-XML-0.93-1.fc17.src.rpm

Also, good catch of bug 827801 - that's actually the bug that started my whole
crazy quest to get this set of modules packaged, as I initially simply wanted
to get perl-MARC-Record updated and thought I would help out by offering a spec
file :) I'll comment over there accordingly. Thanks again, Iain!

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to