AVSystem / Anjay

C implementation of the client-side OMA LwM2M protocol
Other
188 stars 68 forks source link

Fix hard to find bug with apt-get install change not refreshing apt index #54

Closed cerkiewny closed 3 years ago

cerkiewny commented 3 years ago

In case of apt-get install list change, the apt-get update won't be run, which means that potentially outdated list of apt-get packages will be used in the build. This PR solves that by executing the layers as one command.

kFYatek commented 3 years ago

Thanks for this fix! We generally dealt with this problem by always using --no-cache (as mentioned in the readme), but this is definitely an improvement :)

cerkiewny commented 3 years ago

I would also suggest just mapping directory instead of copying files and having some sort of build.sh that cmake .. makes as an entrypoint, than the docker can be executed in the cloud and keeps the code / configuration in isolation