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

conditional build for addons #2516

Merged
merged 6 commits into from
Oct 26, 2019
Merged

conditional build for addons #2516

merged 6 commits into from
Oct 26, 2019

Conversation

jerch
Copy link
Member

@jerch jerch commented Oct 25, 2019

Coming from #2503 (comment).

This uses a custom addon-install script, that is only present for repo forks (not contained in the npm package). The conditional execution is simply done at package.json level as a try { require(...) } ... call.

@Tyriar:
This works for me, the conditional stuff is merely a hack but should work on all platforms. Did not like the idea to provide different shell scripts here.

@jerch jerch added the area/build Regarding the build process label Oct 25, 2019
@jerch jerch mentioned this pull request Oct 25, 2019
package.json Outdated Show resolved Hide resolved
bin/install-addons.js Show resolved Hide resolved
bin/install-addons.js Outdated Show resolved Hide resolved
bin/install-addons.js Outdated Show resolved Hide resolved
@leomoty
Copy link
Contributor

leomoty commented Oct 25, 2019

Just for reference, an alternative would be lerna:

https://github.com/lerna/lerna/tree/master/commands/bootstrap

Not sure if it is worth the trouble.

@jerch
Copy link
Member Author

jerch commented Oct 25, 2019

@leomoty Thx for the suggestion. I think we should keep it as simple as possible, to me lerna seems to be overkill here.

@leomoty
Copy link
Contributor

leomoty commented Oct 25, 2019

@jerch, yeah agreed.

package.json Outdated Show resolved Hide resolved
@Tyriar Tyriar added this to the 4.3.0 milestone Oct 25, 2019
@jerch jerch merged commit edc0957 into xtermjs:master Oct 26, 2019
@jerch jerch deleted the addon_build branch October 26, 2019 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/build Regarding the build process
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants