ACK/cmnt: [PATCH] UBUNTU: snapcraft.yaml: various improvements

[Stefan Bader]

On 26.06.2017 16:23, Paolo Pisati wrote:
> BugLink: http://bugs.launchpad.net/bugs/1700576
> 
> This patch improves over the original snapcraft.yaml we ship in Xenial,
> and makes the generated kernel snap nearly identical to the one we ship
> in the store.
> Among the different improvements, we have:
> 
> 1) include the linux-firmwar and linux-firmware-snapdraone packages in
>    the final snap
> 2) rename it to snapdragon-kernel to match the snap we have in the store
> 3) dynamic versioning
> 4) autogenerated config from debian.$DEBIAN/config/*
> 
> Signed-off-by: Paolo Pisati <paolo.pisati at canonical.com>
Acked-by: Stefan Bader <stefan.bader at canonical.com>

> ---

When applying we could fix those spelling errors...

>  snapcraft.yaml | 14 ++++++++++----
>  1 file changed, 10 insertions(+), 4 deletions(-)
> 
> diff --git a/snapcraft.yaml b/snapcraft.yaml
> index 9eb3f40..9ceba92 100644
> --- a/snapcraft.yaml
> +++ b/snapcraft.yaml
> @@ -1,10 +1,10 @@
> -name: pc-kernel
> +name: dragonboard-kernel
>  version: null
>  version-script: |
>      . debian/debian.env
>      dpkg-parsechangelog -l $DEBIAN/changelog -S version
> -summary: The Ubuntu generic Linux kernel
> -description: This Ubuntu generic Linux kernel
> +summary: The canonical snapdragon arm64 kernel
> +description: The canonical snapdragon arm64 kernel
>  grade: stable
>  confinement: strict
>  type: kernel
> @@ -14,18 +14,24 @@ parts:
>      plugin: kernel
>      source: .
>      source-type: git
> -    kconfigflavour: generic
> +    kconfigflavour: snapdragon
>      kconfigs:
>        - CONFIG_DEBUG_INFO=n
> +    install: |
> +      cp $SNAPCRAFT_PART_INSTALL/dtbs/qcom/apq8016-sbc-snappy.dtb $SNAPCRAFT_PART_INSTALL/dtbs/qcom/apq8016-sbc.dtb
>    firmware:
>      plugin: nil
>      stage-packages:
>        - linux-firmware
> +      - linux-firmware-snapdragon
>      organize:
>        lib/firmware: firmware
>      prime:
>        - -usr
>        - -lib
> +    install: |
> +      mkdir -p $SNAPCRAFT_PART_INSTALL/firmware/wlan
> +      ln -s /run/macaddr0 $SNAPCRAFT_PART_INSTALL/firmware/wlan/
>      build-packages:
>        - cpio
>        - libssl-dev
> 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20170627/846d2186/attachment.sig>