h2def configure test should consider installs that have .pyc without .py

[Arkadiusz Miśkiewicz]

  if test -x /usr/share/pygobject/2.0/codegen/h2def.py; then
    H2DEF=/usr/share/pygobject/2.0/codegen/h2def.py
    PYG_CODEGEN=pygobject-codegen-2.0

Note that this test doesn't seem good.

First py files could not be there. It's perfectly fine to have only *.pyc or *.pyo files (just like on my system).

Also here that file is not executable and can be called only by "python file.pyc" instead of just file.pyc.

[Alexander Sack (asac)]

> First py files could not be there. It's perfectly fine to have only *.pyc or *.pyo files (just like on my system).

what do you have in /usr/bin/pygobject-codegen-2.0 ?

I see ... exec /usr/bin/python $codegendir/codegen.py "$@" here

do you have a .pyc there?

[Arkadiusz Miśkiewicz (arekm)]

$ cat /usr/bin/pygobject-codegen-2.0
#!/bin/sh

prefix=/usr
datarootdir=${prefix}/share
datadir=/usr/share
codegendir=${datadir}/pygobject/2.0/codegen

PYTHONPATH=$codegendir
export PYTHONPATH

if [ -f $codegendir/codegen.pyc ]; then
        codegenexe="$codegendir/codegen.pyc"
else
        codegenexe="$codegendir/codegen.py"
fi

exec /usr/bin/python $codegenexe "$@"

[Alexander Sack (asac)]

please checkout bzr branch:
 lp:~asac/ntrack/lp743881

patch attampt is: http://bazaar.launchpad.net/~asac/ntrack/lp743881/revision/285

let me know if thats good enough for your purpose.

[Alexander Sack (asac)]

oh, also: what version of pygobject are you running?

[Arkadiusz Miśkiewicz (arekm)]

2.28 but that above comes from local distro patch.

Could you use
PYTHON=python
and then call like $PYTHON something in configure.in ? That will allow to easier pointing PYTHON to specified binary. Like ./configure PYTHON=/usr/bin/python2 etc.

[Alexander Sack (asac)]

yeah, I can try to make PYTHON configurable.

BTW, our ubuntu python guru told me that no .py files is not really something supported for all corner cases; so basically it's expected that folks have to patch stuff if they opt-in to remove all .py ;).

[Alexander Sack (asac)]

for commit log: what distro are you running?

[Arkadiusz Miśkiewicz (arekm)]

Kind of "own" - www.pld-linux.org

[Alexander Sack (asac)]

------------------------------------------------------------
revno: 285 [merge]
fixes bug(s): https://launchpad.net/bugs/743881
committer: Alexander Sack <email address hidden>
branch nick: ntrack
timestamp: Sun 2011-04-03 12:49:45 +0200
message:
  merge::build: improve H2DEF detect; recognize .pyc and .py extensions - lp:743881
  + merged branch lp:~asac/ntrack/lp743881
  + reported by arekm as needed for http://www.pld-linux.org
------------------------------------------------------------

[Alexander Sack (asac)]

Fixed: https://launchpad.net/ntrack/main/014