Skip to content

build: fix version checks in configure.py - #29965

Closed
targos wants to merge 1 commit into
nodejs:masterfrom
targos:py-strictversion
Closed

build: fix version checks in configure.py#29965
targos wants to merge 1 commit into
nodejs:masterfrom
targos:py-strictversion

Conversation

@targos

Copy link
Copy Markdown
Member

Fixes: #29927
Refs: #29931

@nodejs-github-botnodejs-github-bot added the build Issues and PRs related to build files or the CI. label Oct 14, 2019
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@bnoordhuisbnoordhuis left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The long lines are kind of regrettable but I'll get over it.

targos added a commit that referenced this pull request Oct 18, 2019
Fixes: #29927
Refs: #29931
PR-URL: #29965
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
@targos

Copy link
Copy Markdown
MemberAuthor

Landed in d769ebc

@targostargos closed this Oct 18, 2019
@targos
targos deleted the py-strictversion branch October 18, 2019 07:29
MylesBorins pushed a commit that referenced this pull request Oct 23, 2019
Fixes: #29927
Refs: #29931
PR-URL: #29965
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
@MylesBorinsMylesBorins mentioned this pull request Oct 23, 2019
targos added a commit that referenced this pull request Nov 8, 2019
Fixes: #29927
Refs: #29931
PR-URL: #29965
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
targos added a commit that referenced this pull request Nov 10, 2019
Fixes: #29927
Refs: #29931
PR-URL: #29965
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
@targostargos mentioned this pull request Nov 10, 2019
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

buildIssues and PRs related to build files or the CI.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Version number comparisons in Python and GYP files are not safe

6 participants

@targos@nodejs-github-bot@bnoordhuis@lpinca@devnexen@richardlau