Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tools: update gyp-next to v0.7.0 #36580

Closed
wants to merge 1 commit into from
Closed

Conversation

targos
Copy link
Member

@targos targos commented Dec 19, 2020

Refs: https://github.com/nodejs/gyp-next/releases/tag/v0.7.0

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

@targos targos added the gyp Issues and PRs related to the GYP tool and .gyp build files label Dec 19, 2020
@nodejs-github-bot nodejs-github-bot added build Issues and PRs related to build files or the CI. tools Issues and PRs related to the tools directory. labels Dec 19, 2020
@targos targos added the request-ci Add this label to start a Jenkins CI on a PR. label Dec 19, 2020
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Dec 19, 2020
@nodejs-github-bot
Copy link
Collaborator

@targos
Copy link
Member Author

targos commented Dec 21, 2020

Would love to have some reviews :)
@nodejs/gyp @nodejs/python

Copy link
Member

@richardlau richardlau left a comment

Choose a reason for hiding this comment

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

Is the mode change (100755 → 100644) for tools/gyp/setup.py intentional?

@targos
Copy link
Member Author

targos commented Dec 21, 2020

I wonder if it was unintentionally set to 755 before? I did the update by removing the directory and cloning a fresh one from https://github.com/nodejs/gyp-next

@ryzokuken
Copy link
Contributor

@targos I suppose those could be landed...

targos added a commit that referenced this pull request Dec 26, 2020
Refs: https://github.com/nodejs/gyp-next/releases/tag/v0.7.0

PR-URL: #36580
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Ujjwal Sharma <ryzokuken@disroot.org>
@targos
Copy link
Member Author

targos commented Dec 26, 2020

Landed in 8c94aea

@targos targos closed this Dec 26, 2020
@targos targos deleted the gyp-0.7.0 branch December 26, 2020 11:35
danielleadams pushed a commit that referenced this pull request Jan 12, 2021
Refs: https://github.com/nodejs/gyp-next/releases/tag/v0.7.0

PR-URL: #36580
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Ujjwal Sharma <ryzokuken@disroot.org>
@danielleadams danielleadams mentioned this pull request Jan 12, 2021
codebytere added a commit to electron/electron that referenced this pull request May 19, 2021
codebytere added a commit to electron/electron that referenced this pull request May 19, 2021
codebytere added a commit to electron/electron that referenced this pull request May 20, 2021
targos added a commit that referenced this pull request May 25, 2021
Refs: https://github.com/nodejs/gyp-next/releases/tag/v0.7.0

PR-URL: #36580
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Ujjwal Sharma <ryzokuken@disroot.org>
codebytere added a commit to electron/electron that referenced this pull request May 31, 2021
codebytere added a commit to electron/electron that referenced this pull request May 31, 2021
targos added a commit that referenced this pull request Jun 5, 2021
Refs: https://github.com/nodejs/gyp-next/releases/tag/v0.7.0

PR-URL: #36580
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Ujjwal Sharma <ryzokuken@disroot.org>
codebytere added a commit to electron/electron that referenced this pull request Jun 8, 2021
codebytere added a commit to electron/electron that referenced this pull request Jun 9, 2021
codebytere added a commit to electron/electron that referenced this pull request Jun 10, 2021
targos added a commit that referenced this pull request Jun 11, 2021
Refs: https://github.com/nodejs/gyp-next/releases/tag/v0.7.0

PR-URL: #36580
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Ujjwal Sharma <ryzokuken@disroot.org>
richardlau pushed a commit to richardlau/node-1 that referenced this pull request Aug 18, 2021
Refs: https://github.com/nodejs/gyp-next/releases/tag/v0.7.0

PR-URL: nodejs#36580
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Ujjwal Sharma <ryzokuken@disroot.org>
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. gyp Issues and PRs related to the GYP tool and .gyp build files tools Issues and PRs related to the tools directory.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants