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

Bikeshedding question: new home for V8's GYP files? #18262

Closed
hashseed opened this issue Jan 19, 2018 · 6 comments
Closed

Bikeshedding question: new home for V8's GYP files? #18262

hashseed opened this issue Jan 19, 2018 · 6 comments
Labels
discuss Issues opened for discussions and feedbacks. v8 engine Issues and PRs related to the V8 dependency.

Comments

@hashseed
Copy link
Member

As you all might know, Chrome has moved off GYP in favor of GN as build system. V8 has followed that move, but kept maintaining gyp configs for longer than Chrome requires, mostly for Node.js.

In the past weeks I have, as planned, implemented a GYP/GN bridge so that V8 as Node.js' dependency can be built with GN. This means that in order to test V8 against Node.js, the V8 team no longer has to maintain GYP.

We will probably be getting rid of GYP configs in upstream V8 in this quarter. Node.js will need to take ownership over these files, at least before it has found an alternative to GYP.

Question is, where should these files reside?

  • Should they stay in deps/v8/ even though they no longer exist in upstream V8? I'm sure we can make this work without affecting updating V8 too much by tweaking .gitignore files.
  • Or should they live in node core, for example tools or a new build subfolder?

I don't have any strong opinion on this, but would like to experiment with this to prepare for handover.

@hashseed hashseed added discuss Issues opened for discussions and feedbacks. v8 engine Issues and PRs related to the V8 dependency. labels Jan 19, 2018
@addaleax
Copy link
Member

/cc @nodejs/v8

Afaik we do maintain the other gypfiles (for e.g. zlib, c-ares, nghttp2) through the first way, i.e. in deps/foo/ even though they don’t exist upstream.

@ofrobots
Copy link
Contributor

+1 for first option.

@targos
Copy link
Member

targos commented Jan 21, 2018

V8 gypfiles are currently scattered in many directories. If we go with option 1, is it possible to put them all in the same place?

@hashseed
Copy link
Member Author

Okay. I have begun moving gyp files into v8/gypfiles. At some point I'll simply delete them from V8.

@mhdawson
Copy link
Member

mhdawson commented Feb 1, 2018

Sounds good to me.

@targos
Copy link
Member

targos commented Feb 23, 2018

GYP files are now out of V8 master. I added the necessary steps to handle this in update-v8: https://github.com/targos/update-v8/blob/master/src/gypfiles.js

@targos targos closed this as completed Feb 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discuss Issues opened for discussions and feedbacks. v8 engine Issues and PRs related to the V8 dependency.
Projects
None yet
Development

No branches or pull requests

5 participants