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

Support for go mod vendor #60

Merged
merged 4 commits into from
Dec 24, 2020
Merged

Support for go mod vendor #60

merged 4 commits into from
Dec 24, 2020

Conversation

rogchap
Copy link
Owner

@rogchap rogchap commented Dec 23, 2020

When using go mod vendor, the command only copies folders that contain imported Go packages.
By adding a Go packages to all the folders that contain V8 libs and header files will insure all these files are copied
to the vendor directory when running go mod vendor.

Fixes #34
Ref #53

@rogchap rogchap requested a review from tmc December 23, 2020 04:57
Copy link
Collaborator

@tmc tmc left a comment

Choose a reason for hiding this comment

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

LGTM. It may make sense to add a test to prevent regression (if someone introduces a new directory down the line).

@rogchap rogchap merged commit bcfd2d6 into master Dec 24, 2020
@rogchap rogchap deleted the mod-vendor branch December 24, 2020 07:44
genevieve pushed a commit to Shopify/v8go that referenced this pull request Oct 28, 2021
genevieve pushed a commit to Shopify/v8go that referenced this pull request Oct 28, 2021
genevieve pushed a commit to Shopify/v8go that referenced this pull request Nov 16, 2021
* Add intl support with test

* Update V8 static library for macos-latest

* Update V8 static library for ubuntu-18.04

* Temporarily disable build for mac/ubuntu, try windows with python2

* Change import statement for python2

* Force __init__.py to be created for python2

* Try rogchap#60

* Create __init__.py all the way down

* Update intl_test.go
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

'v8.h' file not found
2 participants