Project

General

Profile

Actions

Bug #5042

closed

Injecting PKG_CONFIG_PATH in os.environ doesn't work

Added by Davide Pesavento about 5 years ago. Updated about 5 years ago.

Status:
Closed
Priority:
Normal
Category:
Build
Target version:
Start date:
Due date:
% Done:

100%

Estimated time:

Description

The trick implemented in #2178 that adds /usr/local/lib/pkgconfig to PKG_CONFIG_PATH doesn't work anymore: the changes to os.environ don't have any effect on pkg-config processes spawned by waf.

I initially noticed this problem several months ago on FreeBSD, but I thought it was a BSD-specific behavior so I didn't give it much attention. However, the same now occurs on CentOS 8 (and likely on RHEL, Fedora, and similar distros). It could be a change in behavior between python2 and python3 (I was using python3 on CentOS), or simply a regression in waf when pre-forked worker processes were introduced. I filed an issue with the waf project about this.

There is a much simpler way to implement #2178 trick though: pass pkg_config_path=... to the check_cfg function.


Related issues 3 (0 open3 closed)

Related to NFD - Task #2178: Set PKG_CONFIG_PATH in wscriptClosedAlex Afanasyev

Actions
Blocks NFD - Task #5040: Deploy CentOS 8 Jenkins agentsClosedDavide Pesavento

Actions
Blocks NFD - Task #5045: Release 0.7.0ClosedAlex Afanasyev

Actions
Actions #1

Updated by Davide Pesavento about 5 years ago

  • Related to Task #2178: Set PKG_CONFIG_PATH in wscript added
Actions #2

Updated by Davide Pesavento about 5 years ago

  • Blocks Task #5040: Deploy CentOS 8 Jenkins agents added
Actions #3

Updated by Davide Pesavento about 5 years ago

  • Status changed from New to In Progress
Actions #4

Updated by Davide Pesavento about 5 years ago

  • % Done changed from 0 to 10
Actions #5

Updated by Davide Pesavento about 5 years ago

  • % Done changed from 10 to 60
Actions #6

Updated by Davide Pesavento about 5 years ago

Actions #7

Updated by Davide Pesavento about 5 years ago

  • Status changed from In Progress to Code review
  • % Done changed from 60 to 100
Actions #8

Updated by Davide Pesavento about 5 years ago

  • Status changed from Code review to Closed
Actions

Also available in: Atom PDF