Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: EvtxParser - Windows Event Log parsing tools
https://bugzilla.redhat.com/show_bug.cgi?id=705799
Summary: Review Request: EvtxParser - Windows Event Log parsing tools Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: rjones@redhat.com QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Story Points: ---
Spec URL: http://oirase.annexia.org/reviews/evtx/EvtxParser.spec SRPM URL: http://oirase.annexia.org/reviews/evtx/EvtxParser-1.0.7-1.fc15.src.rpm Description: Windows Event Log parsing tools
For an example of using these tools see: https://rwmj.wordpress.com/2011/04/17/decoding-the-windows-event-log-using-g...
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=705799
Richard W.M. Jones rjones@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Depends on| |705798
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=705799
--- Comment #1 from Richard W.M. Jones rjones@redhat.com 2011-05-18 10:08:58 EDT --- Note that it's not possible to do a scratch build in Koji because this package requires perl-Data-Hexify (bug 705798).
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=705799
--- Comment #2 from Richard W.M. Jones rjones@redhat.com 2011-05-18 10:10:14 EDT --- rpmlint output is:
EvtxParser.src: W: spelling-error %description -l en_US evtx -> Evert EvtxParser.src: W: spelling-error %description -l en_US evtxdump -> Dumpster EvtxParser.noarch: W: spelling-error %description -l en_US evtx -> Evert EvtxParser.noarch: W: spelling-error %description -l en_US evtxdump -> Dumpster EvtxParser.noarch: E: non-standard-executable-perm /usr/bin/evtxinfo.pl 0555L EvtxParser.noarch: E: non-standard-executable-perm /usr/bin/evtxdump.pl 0555L EvtxParser.noarch: E: non-standard-executable-perm /usr/bin/evtxtemplates.pl 0555L 2 packages and 0 specfiles checked; 3 errors, 4 warnings.
I don't understand what "non-standard-executable-perm" is complaining about. It seems perfectly reasonable to use 0555 to me.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=705799
--- Comment #3 from Richard W.M. Jones rjones@redhat.com 2011-05-18 10:24:09 EDT --- I sent the new Makefile.PL and the patch upstream.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=705799
--- Comment #4 from Adam Huffman bloch@verdurin.com 2011-05-31 18:15:21 EDT --- Tried to build this in Mock and saw the following error:
Executing(%build): /bin/sh -e /var/tmp/rpm-tmp.vvj1p5 + umask 022 + cd /builddir/build/BUILD + cd evtxtools-1.0.7 + LANG=C + export LANG + unset DISPLAY + perl Makefile.PL INSTALLDIRS=vendor Can't locate ExtUtils/MakeMaker.pm in @INC (@INC contains: /usr/local/lib/perl5 /usr/local/share/perl5 /usr/lib/perl5/vendor_perl /usr/share/perl5/vendor_perl /usr/lib/perl5 /usr/share/perl5 .) at Makefile.PL line 1. BEGIN failed--compilation aborted at Makefile.PL line 1. RPM build errors: error: Bad exit status from /var/tmp/rpm-tmp.vvj1p5 (%build) Bad exit status from /var/tmp/rpm-tmp.vvj1p5 (%build) Child returncode was: 1
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=705799
--- Comment #5 from Richard W.M. Jones rjones@redhat.com 2011-05-31 18:23:01 EDT --- Strange. perl(ExtUtils::MakeMaker) is a build requirement, so mock ought to install it.
Is it expected that the Perl path should contain /usr/local?
package-review@lists.fedoraproject.org