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
alpine image should include git #366
Comments
I am inclined to agree! I will open a PR and we can continue the discussion there. |
That was quick, @LaurentGoderre! I am going to close this issue in favour of the PR. Closed in favour of #367 |
Hi! I see that we did add git but not for all versions of node js. Is it possible to do the update so in future versions, git can be available? thank you! |
No, git was not added. |
We are not modifying the base image. Debian includes it alpine doesn't. With the introduction of multistage builds, it doesn't make sense for us to add git to alpine. |
This doesn't work if your package.json has packages that are to be installed directly from git. |
just install it yourself if you need it |
npm has its git repo as a dependency feature (see https://docs.npmjs.com/files/package.json#git-urls-as-dependencies). In order for it to work, of course, the git client must be available.
If you try to do a basic
npm install
in an alpine-based image, and one of the packages uses that feature, it will fail. Similarly, if anything in yourpackage.json
refers to a git URL rather than a semver version, it will fail.Since git, therefore, is a requirement for basic npm usage, I would suggest it should be included in the alpine image via the Dockerfile.
Simple example (from
package.json
in/my/local/path
):and then
The text was updated successfully, but these errors were encountered: