Skip to content

build: Fix python path resolution in configure#16460

Merged
refack merged 1 commit intonodejs:masterfrom
digitalinfinity:pypath_fix
Oct 26, 2017
Merged

build: Fix python path resolution in configure#16460
refack merged 1 commit intonodejs:masterfrom
digitalinfinity:pypath_fix

Conversation

@digitalinfinity
Copy link
Contributor

The check to validate whether the current python process is the same as
the one resolved from the current path fails if the paths differ by case
which can happen on an operating system with case insensitive file
system behavior like Windows. Canonicalize it by converting both to
lower case if running on Windows.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

build

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Issues and PRs related to build files or the CI.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants