Note: This is a beta release of Red Hat Bugzilla 5.0. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Also email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback here.
Bug 610073 - Review Request: flyback - time machine for linux
Summary: Review Request: flyback - time machine for linux
Alias: None
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
Depends On:
TreeView+ depends on / blocked
Reported: 2010-07-01 14:00 UTC by Sascha Thomas Spreitzer
Modified: 2010-11-24 15:28 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2010-11-24 15:28:39 UTC

Attachments (Terms of Use)

Description Sascha Thomas Spreitzer 2010-07-01 14:00:00 UTC
Spec URL:
! this is my first package, and I am seeking a sponsor !
A time machine for linux, a clone of Apples Time Machine backup and archive utility.

Comment 1 Sascha Thomas Spreitzer 2010-07-01 14:02:11 UTC
Checked the spec and the srpm with rpmlint. No errors or warnings.

Comment 2 Sascha Thomas Spreitzer 2010-08-04 22:53:48 UTC
David Woodhouse is sponsoring me, removing FE-NEEDSPONSOR blockage.

Comment 3 Mohammed Safwat 2010-08-06 23:43:12 UTC
I amn't a sponsor; this's just a casual review.

- The Version field should reflect the corresponding software version as specified on the project website. I can't find a version matching 20100629 at

- There's neither a separate license(COPYING) file nor a license header notice in the source files. The specified version, GPLv2, correctly matches the one specified at the project page; you should notify the upstream to include a license file and a license notice in the source file headers. This isn't a blocker, however.

- The Source0 URL expands to path pertaining to the packager personal space at fedorapeople; this can't be a valid permanent link to get the original software source from. You should specify a download link as provided by the upstream, usually at the software site.

- You can substitute the project name directly in the Source0 field instead of the macro %{name} just to facilitate tracking the URL for reviewers, but it's a matter of personal prefernce anyway.

- You should uncomment the BuildRequires filed, stating appropriate required python runtime development libraries(python 2 or python 3). See for details. You should also add desktop-file-utils in the BuildRequires field as explained at, since you've a desktop file in your package.

- Under %build section, the instructions section are intended to add a new wrapper shell script to a python script. You should instead create a patch containing this wrapper script and use %patch under %prep section to apply the patch.

- Under %install section, you should use the install command with appropriate command-line switches instead of mkdir and cp commands. Consult the install manual as well as for available install options and examples.

- Under %install section, to install the desktop file use the command desktop-file-install instead of cp. See for more usage scenarios of this command.

- Under %install and %files sections, you should make use of the predefined path macros instead of specifying explicit paths, for example %{_datarootdir} instead of /usr/share, %{_bindir} instead of /usr/bin, and %{_desktopdir} instead of /usr/share/applications, ...etc. The command `rpm --showrc' can help you identify the paths predefined by macros. Check for other useful examples.

- In the desktop file flyback.desktop, it's better to specify the icon with a short name, but the full path is also OK. See for explanation.

- The desktop file contains a deprecated key, FilePattern, as described at It should be removed.

Comment 4 Sascha Thomas Spreitzer 2010-08-07 13:03:28 UTC
(In reply to comment #3)

Wow, thank you very much Mohammed for the detailed review, I will soon start to work through your list. Please stand by. :)

Comment 5 Mohammed Safwat 2010-08-15 07:11:27 UTC
If there's no clear source tarball for the software(as I haven't found one on the website), consider creating the tarball yourself. In this case, just state the tarball file name(with no URL) in the Source0 field, as illustrated at

Comment 6 Mario Ceresa 2010-11-24 09:25:21 UTC
I tried to retrieve the spec file but I cannot access it anymore. Is the review still going on? I'll be interested to help.


Comment 7 Jason Tibbitts 2010-11-24 15:28:39 UTC
I believe the submitter indicated that they were completely leaving the project for whatever reason.  I know they had all content removed from their account.  I'll just close this ticket.

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