Re: [PATCH 1/2] Support python 3 in utils/key2pub.py.

[Stefan Lippers-Hollmann <s.l-h@gmx.de>]

Hi

On 2015-07-21, Taahir Ahmed wrote:
> On Wednesday 22 July 2015 04:50:45 Stefan Lippers-Hollmann wrote:
> > You omit $(pwd) from ./utils/key2pub.py, while /utils/key2pub.py
> > won't exist.
> 
> Wow, that's a really bad mistake on my part.  It should indeed probably
> be './utils/key2pub.py'.  I made this change after my general testing,
> simply because emacs was undoing the execute bit every time I saved
> key2pub.py.
> 
> It's not really a material change, so I'll probably just put it back the
> way it was.
> 
> The rest of the patch is tested, I pinky-swear :)
>  
> > As little as I know about python packaging policies in Debian
> > (and probably Fedora), /usr/bin/python is never supposed to point
> > to python3 - afaik the interpreter should always be called python3
> > there, so I don't really see how that's going to work there.
> 
> I'm not assuming that the system interpreter is any particular version:
> key2pub.py has been modified so it runs under either 2.7 or 3.x.
> 
> Some more simplification might be possible if 2.7 support is dropped,
> but not much, and I didn't want to rock the boat.

The problem, as I understand it, is that the python3 interpreter will
never be available as (/usr/bin/)python on Debian (or Fedora; even if
python2.x is not installed on the system), but always be called python3. 
So your new python3 compatibility is never actually used, neither on a 
python3-only system.

Therefore I'd suggest this approach instead, either make the python
interpreter user configurable, e.g.:

PYTHON ?= python

so the user can specify the interpreter when invoking make (as in
make PYTHON=python3), xor trying to auto-detect it (untested):

ifeq ($(shell which python3),)
  PYTHON = python
else
  PYTHON = python3
endif

and then using 

$(Q) $(PYTHON) ./utils/key2pub.py --$* $(wildcard $(PUBKEY_DIR)/*.pem) $@

wherever python is needed.

Regards
	Stefan Lippers-Hollmann