Fix from https://github.com/buildpacks/pack/pull/1672 is breaking change #1800
Labels
good first issue
A good first issue to get started with.
status/ready
Issue ready to be worked on.
type/bug
Issue that reports an unexpected behaviour.
Milestone
Summary
This was discussed in Working Group and summarized in Slack here and here
At build time, we copy the application (source) directory into the build container using
archive.WriteDirToTar
. #1672 fixes weird permissions on this directory, but it's also a breaking change:0755
permissions for newly created directories, andapplications can fail to start. This can be remedied (e.g., by setting the
umask
at build time to something like0002
) but this could be a breaking change for some users ofpack
.Proposed fix:
includeRoot
is provided toarchive.WriteDirToTar
, but instead of using default permissions for newly created directories on the host, simply use0777
for the application directory as the least breaking change. See linked threads for other fixes discussed and why we feel that this one is the "best"The text was updated successfully, but these errors were encountered: