Bug 1109717 - Review Request: python-evtx - Windows Event Log parsing library and tools
Review Request: python-evtx - Windows Event Log parsing library and tools
Status: NEW
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: Nobody's working on this, feel free to take it
Fedora Extras Quality Assurance
Depends On:
  Show dependency treegraph
Reported: 2014-06-16 04:27 EDT by Richard W.M. Jones
Modified: 2015-11-26 13:57 EST (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed:
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:

Attachments (Terms of Use)

  None (edit)
Description Richard W.M. Jones 2014-06-16 04:27:58 EDT
Spec URL: http://oirase.annexia.org/reviews/python-evtx/python-evtx.spec
SRPM URL: http://oirase.annexia.org/reviews/python-evtx/python-evtx-0-0.1.git8838756b.src.rpm
Description: Windows Event Log parsing library and tools
Fedora Account System Username: rjones
Comment 1 Richard W.M. Jones 2014-06-16 04:29:58 EDT
Upstream URLs:
Comment 2 Richard W.M. Jones 2014-06-16 04:30:14 EDT
rpmlint output:

python-evtx.src: W: invalid-url Source0: python-evtx-master.zip
python-evtx.noarch: W: no-manual-page-for-binary evtxdump.py
python-evtx.noarch: W: no-manual-page-for-binary evtxinfo.py
3 packages and 0 specfiles checked; 0 errors, 3 warnings.
Comment 3 Richard W.M. Jones 2014-06-16 05:08:03 EDT
Scratch build:
Comment 4 Yanko Kaneti 2014-06-18 04:05:47 EDT
Please follow http://fedoraproject.org/wiki/Packaging:SourceURL#Github
Upstream even does releases so its either
Source0: https://github.com/williballenthin/%{name}/archive/v%{version}.tar.gz
for the last 0.3.1 release
or for master
%global commit 8838756be2736695311440073d9f0b498df1fc24
%global shortcommit %(c=%{commit}; echo ${c:0:7})
Source0: https://github.com/williballenthin/%{name}/archive/%{commit}/%{name}-%{commit}.tar.gz
Comment 6 Antonio Trande 2014-08-13 15:03:25 EDT
I think there are some changes to do according to the Packaging Guidelines for Python.
Comment 7 Dmitry Tantsur 2014-10-09 09:35:56 EDT

1. BuildRequires:  python
should be 
BuildRequires:  python2-devel

2. find Evtx -name '*.py' | xargs sed -i '/\/usr\/bin\/python/d'
I think it would be better to check for shebang only, something like:
| xargs sed -i ''/^#!.*\/python/d'

3. Why not use summary and description from package's setup.py?
Also I think in "Programming documentation for %{name}" using full summary of the base package would be more helpful for a user.

Newer build for reference: http://koji.fedoraproject.org/koji/taskinfo?taskID=7811652

Note You need to log in before you can comment on or make changes to this bug.