ACK: [PATCH] UBUNTU: snapcraft.yaml: stop invoking the obsolete (and non-existing) 'firmware_install' target

[Colin Ian King]

On 17/07/18 15:58, Paolo Pisati wrote:
> BugLink: http://bugs.launchpad.net/bugs/1782166
> 
> The snapcraft.yaml in bionic/master (by default) invokes the now obsolete
> 'firmware_install' target that was removed in
> 5620a0d1aacd554ebebcff373e31107bb1ef7769 ("firmware: delete in-kernel
> firmware"), and by invoking a non existing target, it fails to build.
> 
> Since firmware files were removed from the Linux source code, and we rely on
> external files provided by the linux-firmware package, skip invoking
> 'firmware_install' altogether and fix the build.
> 
> Signed-off-by: Paolo Pisati <paolo.pisati at canonical.com>
> ---
>  snapcraft.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/snapcraft.yaml b/snapcraft.yaml
> index f96d4aa..2452c40 100644
> --- a/snapcraft.yaml
> +++ b/snapcraft.yaml
> @@ -21,6 +21,7 @@ parts:
>        cp debian/scripts/retpoline-extract-one \
>          $SNAPCRAFT_PART_BUILD/scripts/ubuntu-retpoline-extract-one
>        snapcraftctl build
> +    kernel-with-firmware: false
>    firmware:
>      plugin: nil
>      stage-packages:
> 

Makes sense.

Acked-by: Colin Ian King <colin.king at canonical.com>