Project

General

Profile

Bug #212

PYTHON_EXT_SUFFIX in depend.mk

Added by Guilhem Saurel about 1 month ago. Updated about 1 month ago.

Status:
Closed
Priority:
Normal
Assignee:
-

Description

Hi,

So now that wip-quadprog is building, it looks like the SYSTEM_SEARCH is wrong:

╰─>$ make show-var VARNAME=SYSTEM_SEARCH.py-quadprog
'lib/python2.7/site-packages/quadprog'

(from wip/py-crocoddyl)

the PYTHON_EXT_SUFFIX is missing. And if I just copy-paste its two-line definition from mk/sysdep/python.mk to wip/py-quadprog/depend.mk, this situation doesn't improve.

Do you have an idea about what am I missing ?

Associated revisions

Revision 1132180d (diff)
Added by Anthony Mallet about 1 month ago

[mk/sysdep] Make sure PYTHON_EXT_SUFFIX is not referenced too early

PYTHON_EXT_SUFFIX needs a python interpreter to be computed properly, so it
cannot be used before the dependency resolution. In particular, it cannot be
used in SYSTEM_SEARCH.pkg (#212).

A possible solution could be to assign a default value, like '*.so', until the
dependency resolution has completed. This would fix #212 but it is a big magic
and the default value might not fit all contexts well.

This commit instead makes sure that an explicit error is raised if the variable
is used before it can be correctly defined, so that there are no surprise
regarding its value.

Revision e305aade (diff)
Added by Anthony Mallet about 1 month ago

[wip/py-quadprog] Don't reference PYTHON_EXT_SUFFIX in SYSTEM_SEARCH

PYTHON_EXT_SUFFIX cannot be used before dependency resolution.

Fixes #212

History

#1

Updated by Anthony Mallet about 1 month ago

Mmm ... PYTHON_EXT_SUFFIX is only available after the dependency
resolution, once the $WRKDIR/.sysdepends file is created and contains
the SYSTEM_FILES.python27.

In the context of SYSTEM_SEARCH, this is too early, the make process
has not restarted and not read those files yet.

I will think about a possible solution, but I'm not sure if anything
simple can be done at all ...

#2

Updated by Anthony Mallet about 1 month ago

  • Status changed from New to Closed

Also available in: Atom PDF