apple / HomeKitADK

Apache License 2.0
2.56k stars 232 forks source link

Fix builds when using recent Docker installations #81

Closed kevinmarty closed 3 years ago

kevinmarty commented 3 years ago

This fixes issue #65

New installs of Docker Desktop as of version 2.4.0 enable BuildKit by default, which changes the format of the output of the "docker build" command. The Makefile attempts to parse this output to get the ID of the image it builds (line 30 of the Makefile), but fails if BuildKit is enabled because the format is different.

Existing installs of Docker upgraded to this version are not affected, as the configuration is not changed, but this fix makes sure that BuildKit is disabled regardless of the default setting so that the output is in the format that the Makefile expects.

aajain-com commented 3 years ago

Thanks for the fix.