Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at
Bug 606071 (vinetto) - Review Request: vinetto - A forensics tool to examine Thumbs.db files
Summary: Review Request: vinetto - A forensics tool to examine Thumbs.db files
Alias: vinetto
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
Whiteboard: NotReady
Depends On:
TreeView+ depends on / blocked
Reported: 2010-06-20 11:18 UTC by Fabian Affolter
Modified: 2011-01-03 20:39 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2011-01-03 20:39:13 UTC
Type: ---

Attachments (Terms of Use)

Description Fabian Affolter 2010-06-20 11:18:05 UTC
Spec URL:

Project URL:

Vinetto is a forensics tool to examine Thumbs.db files.  It is
a console program to extract thumbnail images and their metadata
from those thumbs.db files generated under Microsoft Windows

Koji scratch build:

rpmlint output:
[fab@laptop011 SRPMS]$ rpmlint vinetto-0.07-1.beta.fc13.src.rpm 
vinetto.src: W: spelling-error %description -l en_US metadata -> meta data, meta-data, metatarsus
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

[fab@laptop011 noarch]$ rpmlint vinetto-0.07-1.beta.fc13.noarch.rpm 
vinetto.noarch: W: spelling-error %description -l en_US metadata -> meta data, meta-data, metatarsus
vinetto.noarch: W: no-manual-page-for-binary vinetto
1 packages and 0 specfiles checked; 0 errors, 2 warnings.

Comment 1 Michael Schwendt 2010-06-22 10:34:01 UTC
* The --version arg and program output report 0.06 (r59) which doesn't match RPM package version. // egg-info even refers to 0.06alpha

* The --help arg and Usage output claim that the  -o DIR  arg is optional, but when omitting -o DIR, the program crashes:

$ vinetto Thumbs.db 
/usr/bin/vinetto:35: DeprecationWarning: the md5 module is deprecated; use hashlib instead
  import md5
Traceback (most recent call last):
  File "/usr/bin/vinetto", line 114, in <module>
    tDBfname, outputdir, htmlrep, utf8encoding, symlinks = getargs()
  File "/usr/bin/vinetto", line 77, in getargs
    os.system("mkdir " + opts.outputdir + "/" + NUMBERED_THUMBS_DIR)
TypeError: cannot concatenate 'str' and 'NoneType' objects

* Something else is wrong with the -o DIR arg, too. If the target directory doesn't exist, vinetto creates only half of the target dirs and then fails. os.system("mkdir",...) is insufficient, and it should either bail out sooner or use os.makedirs(...) properly.

* For such a small program with no manual, no examples in the README, and a warning that only one type of Thumbs.db files is understood, I think the software isn't ready yet.

| SHOULD: The reviewer should test that the package functions as
| described. A package should not segfault instead of running,
| for example.

* The good news: The packaging is fine.

Comment 2 Fabian Affolter 2010-06-22 18:41:05 UTC
My bad.  Next time I should check the upstream bug tracker first and not let the reviewer find bugs. Sorry for that.

Comment 3 Fabian Affolter 2011-01-03 20:39:13 UTC
Upstream seems to be dead.

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