-
Notifications
You must be signed in to change notification settings - Fork 645
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
Changing order of RUN vs COPY? #377
Comments
this might be better suited to an external |
Probably. I read in another issue that the assembly stuff may be working with a custom |
could you be a little more specific or point me to that specific issue? i'm not sure what you're referencing. |
The specific comment I was thinking of was https://github.com/rhuss/docker-maven-plugin/issues/327#issuecomment-160878652. It looks like they were referring to https://github.com/rhuss/docker-maven-plugin/issues/205 and https://github.com/rhuss/docker-maven-plugin/issues/203. |
i don't think so - if you have questions on what you need to do to make the assembly work, feel free to ask. |
@andyshinn adding the possibility to refer to an assembly from within Dockerfile is indeed planned for one of the next versions. I'm just thinking about how this could be done best without breaking the Dockerfile syntax (to much). |
OK great. I guess the combination of that + using the alternative |
Its tracked via #205, so I'll close this one for now. |
I don't think it is currently possible to do so. But I'd like to have some of my
RUN
commands happen before theCOPY
. My use case is for package management and dependencies that don't change that often. Right now, theCOPY
happens first and invalidates the cache of thoseRUN
, which causes them to be done again. Those are costly steps in terms of bandwidth and time.I wonder if it makes sense to allow multiple
runCmds
sections and move the optimize to it as well. It could look something like:The text was updated successfully, but these errors were encountered: