Bug 1239148 - Review Request: thefuck - App that corrects your previous console command
Summary: Review Request: thefuck - App that corrects your previous console command
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Zbigniew Jędrzejewski-Szmek
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-07-03 18:03 UTC by Matías Kreder
Modified: 2015-11-18 14:41 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-11-18 14:41:10 UTC
Type: ---
Embargoed:
zbyszek: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Matías Kreder 2015-07-03 18:03:21 UTC
Spec URL: https://delete.fedorapeople.org/thefuck/thefuck.spec
SRPM URL: https://delete.fedorapeople.org/thefuck/thefuck-1.46-1.fc22.src.rpm 
Description: This application corrects your previous console command.
Fedora Account System Username: delete


The purpose of the program is to simplify tasks like these.

➜ puthon
No command 'puthon' found, did you mean:
 Command 'python' from package 'python-minimal' (main)
 Command 'python' from package 'python3' (main)
zsh: command not found: puthon

➜ fuck
python
Python 3.4.2 (default, Oct  8 2014, 13:08:17)

➜ git brnch
git: 'brnch' is not a git command. See 'git --help'.

Did you mean this?
    branch

➜ fuck
git branch
* master


This is a successful build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=10284857

Comment 1 Zbigniew Jędrzejewski-Szmek 2015-07-09 15:26:56 UTC
Hm, have you considered using a copr repo for this instead? Adding a package to the distribution is a long-time commitment, and it seems likely that this project will not see any more updates.

Comment 2 Matías Kreder 2015-07-09 16:37:06 UTC
Zbigniew, why do you think that this project wont see any more updates? their GIT repository is pretty active, I have submitted a patch myself to make it run in python 3.4 and they accepted it.

Comment 3 Matías Kreder 2015-07-09 16:55:15 UTC
They actually just made a release, and another one 2 days ago.

Comment 4 Zbigniew Jędrzejewski-Szmek 2015-07-09 17:55:24 UTC
If the upstream is making new releases, then OK.

Please update to the latest version.

Add %license LICENSE.md to %files.

[ ]: Package requires other packages for directories it uses.
     Note: No known owner of /usr/lib/python3.4/site-packages/thefuck,
     /usr/lib/python3.4/site-packages/tests/rules, /usr/lib/python3.4/site-
     packages/thefuck-1.46-py3.4.egg-info
(Basically you can remove "/*" from those lines in %files.)

You cannot install files into %{python3_sitelib}/tests. This is an upstream bug, but it must be fixed in some way. Either %exclude this directory from the package, or move the directory underneath %{python3_sitelib}/thefuck/.

Rpmlint
-------
thefuck.noarch: E: non-executable-script /usr/lib/python3.4/site-packages/thefuck/rules/cd_correction.py 0644L /usr/bin/env

This contains '/usr/bin/env python' which is misleading, even if not actually used. Probably best to do 'sed -i 1d ...'.

Comment 5 Matías Kreder 2015-07-10 15:24:11 UTC
Updated spec and SRPM.
Let me know if I have miss anything.

SPEC: https://delete.fedorapeople.org/thefuck/thefuck.spec
SRPM: https://delete.fedorapeople.org/thefuck/thefuck-1.48-1.fc22.noarch.rpm

scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=10329673

Comment 6 Zbigniew Jędrzejewski-Szmek 2015-07-10 17:41:50 UTC
Please use %license for the license file [https://fedoraproject.org/wiki/Packaging:LicensingGuidelines#License_Text].

%{python3_sitelib}/thefuck-%{version}-py3.4.egg-info is still unowned.

Would adding the aliases to .bash_profile really work? It would only be available in the login shell iiuc.
Also in the %description, please add a mention that this is for bash.

Comment 8 Zbigniew Jędrzejewski-Szmek 2015-07-10 19:33:27 UTC
+ license is OK
+ license file present and installed properly
+ upstream link OK
+ latest version
+ packaging is OK
+ python packaging OK
+ package works as advertised :)

Package is APPROVED.

Comment 9 Matías Kreder 2015-07-11 00:59:55 UTC
New Package SCM Request
=======================
Package Name: thefuck
Short Description: This application corrects your previous console command.
Upstream URL: https://github.com/nvbn/thefuck
Owners: delete
Branches: f21 f22 el6 epel7
InitialCC:

Comment 10 Zbigniew Jędrzejewski-Szmek 2015-07-12 15:21:49 UTC
New Package SCM Request
=======================
Package Name: thefuck
Short Description: This application corrects your previous console command
Upstream URL: https://github.com/nvbn/thefuck
Owners: delete
Branches: f21 f22 el6 epel7
InitialCC:

Comment 11 Gwyn Ciesla 2015-07-15 17:36:40 UTC
Git done (by process-git-requests).

Comment 12 Zbigniew Jędrzejewski-Szmek 2015-08-17 23:40:24 UTC
OK, changing the status so this does not clutter up my search results.

Comment 13 Upstream Release Monitoring 2015-09-24 15:05:14 UTC
jgrulich's scratch build of kdevelop?#c8e2b9bc57f11e41f3dc6612cdbcc591078d9062 for f22-candidate and git://pkgs.fedoraproject.org/kdevelop?#c8e2b9bc57f11e41f3dc6612cdbcc591078d9062 completed http://koji.fedoraproject.org/koji/taskinfo?taskID=11212117


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