-
Notifications
You must be signed in to change notification settings - Fork 88
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
CLI / Dev image #25
Comments
Hiya! This issue has gone quiet. Spooky quiet. 👻 We get a lot of issues, so we currently close issues after 30 days of inactivity. It’s been at least 20 days since the last update here. If we missed this issue or if you want to keep it open, please reply here. You can also add the label "not stale" to keep this issue open! Thanks for being a part of the Gatsby community! 💪💜 |
@polarathene I opened a PR #26 , maybe you could check it and help me improve the current Docker Image |
agreed with the above comments. the current doesnt build. the first step in the instructions conflictingly say "build your assets" then copy 'em in |
@cdaringe #26 was merged which updates the onbuild image to build the project and uses Debian Buster, so that should handle that purpose better. A development image still needs to be supplied, although I imagine there is less demand for that, will keep this issue open if anyone wants to tackle that. |
agree with OP, this image is not usable for any of the things everyone will expect it to be useful for.
|
Came here looking for a dev image. So, a bump for this issue. :) |
Technically, you could probably just use a nodejs docker image and mount a local data volume that has gatsby installed? Might want another volume for the package cache or configure npm/yarn to use a dir in the mounted volume. Wouldn't be too different to what you'd want from a dev image? Buster is available now, so the outdated packages for things like webp issue isn't an issue anymore. Not sure if npm package linux support for image libs improved to handle Alpine or not, I'm assuming not. |
I was looking for this after having problems getting Gatsby to install and work on a Ubuntu 20.04 distribution (Pop). I ended up using this and modifying the Dockerfile to use it to create a new site as well. https://www.stoutlabs.com/blog/2019-02-05-my-docker-setup-gatsby-next/ |
I built docker image for Gatsby development and wrote procedure manual: If anyone wants to fork to customize, Dockerfile is here: I could develop Gatsby site with this image. |
I am not aware of official gatsby devs caring too much for the maintenance of Docker images here. They probably have no plans for such but wouldn't be against one being contributed here.
This is the correct repo to submit a PR to, however it would need to pass review. I am a bit busy until sometime next month to properly evaluate your Docker image, I do recall alpine linux having a problem with some packages Gatsby uses, you have added You seem to add a bunch of packages for mozjpeg which is an optional package and shouldn't matter too much for a development image, what I do remember is those dependencies are for an older version of mozjpeg, if they're still relevant then the package providing mozjpeg is using an old version and not been updated, here's a docker image with alpine that builds mozjpeg. There is an existing PR in this repo that had a good approach to a dev image, the entrypoint script made it useful. I believe back then |
Image doesn't actually provide Gatsby in a way that can be used for development via Docker.
@gatsbybot has closed two similar issues. It's a bit broken as it closes issues regardless of activity and states you can re-open the issue, but the issue author is not able to actually do so.
#11
#24
Should be noted that alpine isn't a good candidate for dev image at present. Numerous issues detailed in the related PR. Official node images have debian stretch slim available, it should work alright despite having packages a fair bit outdated(2015), Buster will arrive later this year. Official node images are also considering adding Ubuntu as a base option too.
The text was updated successfully, but these errors were encountered: