FS#33847 - [git] namcap errors (python version) and warnings

Attached to Project: Arch Linux
Opened by Alain Kalker (ackalker) - Wednesday, 13 February 2013, 21:51 GMT
Last edited by Dan McGee (toofishes) - Thursday, 14 March 2013, 01:44 GMT
Task Type Bug Report
Category Packages: Extra
Status Closed
Assigned To Dan McGee (toofishes)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:

Running namcap on git package shows several warnings and an error. The error indicates a Python version mismatch in several scripts.
The scripts contain 'print "..."' statements so are definitely not compatible with Python 3.

Additional info:
* package version(s): git1.8.1.3-1
* config and/or log files etc.: no change from default


Steps to reproduce:

$ namcap /var/cache/pacman/pkg/git-1.8.1.3-1-x86_64.pkg.tar.xz
This task depends upon

Closed by  Dan McGee (toofishes)
Thursday, 14 March 2013, 01:44 GMT
Reason for closing:  Fixed
Additional comments about closing:  1.8.2
Comment by Dave Reisner (falconindy) - Saturday, 16 February 2013, 15:04 GMT
I don't see what the problem is here. git only references python2 as an optional dependency and never python(3), so the scripts are right to be python2 compliant.
Comment by Alain Kalker (ackalker) - Tuesday, 19 February 2013, 13:25 GMT
  • Field changed: Percent Complete (100% → 0%)
The problem isn't with the optional python2 dependency, it is with the hashbang in 2 python scripts:
./usr/share/git/remote-helpers/git-remote-hg:#!/usr/bin/env python
./usr/share/git/svn-fe/svnrdump_sim.py:#!/usr/bin/python
These will still be executed by python(3), whether python2 is installed or not. (These files aren't touched by the sed substitution in the PKGBUILD.)
Anyway, I'm working on a patch for upstream to fix the substitution of $PYTHON_PATH, so please see this as more like a heads-up.

Loading...