Merge ~os369510/fisterra/+git/nitrogen6x-gadget:18-classic-devel into ~fisterra-team/fisterra/+git/nitrogen6x-gadget:18-classic

Proposed by jeremyszu
Status: Merged
Merged at revision: 1c8a7372554423467b956060fd28bce01488dec8
Proposed branch: ~os369510/fisterra/+git/nitrogen6x-gadget:18-classic-devel
Merge into: ~fisterra-team/fisterra/+git/nitrogen6x-gadget:18-classic
Diff against target: 62 lines (+14/-6)
2 files modified
build.sh (+11/-5)
snapcraft.yaml (+3/-1)
Reviewer Review Type Date Requested Status
ethan.hsieh Approve
Soar Huang (community) Approve
The Fisterra team Pending
Review via email: mp+385796@code.launchpad.net

Commit message

$ git log
commit 1c8a7372554423467b956060fd28bce01488dec8 (HEAD -> 18-classic, myppa/18-classic-devel)
Author: Jeremy Su <email address hidden>
Date: Tue Jun 16 16:04:48 2020 +0800

    fix warning from shellcheck

commit 9a1c073eb7502dae4aea73981a7848ba7e4e294d
Author: Jeremy Su <email address hidden>
Date: Tue Jun 16 16:04:27 2020 +0800

    fix build fail when using snapcraft from snap repo

Description of the change

The generic build script changed to use snapcraft from snap store.
After that, the gadget will get build fail due to the latest snapcraft changes.

To post a comment you must log in.
Revision history for this message
Soar Huang (soarhuang) :
review: Approve
Revision history for this message
ethan.hsieh (ethan.hsieh) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/build.sh b/build.sh
index c52ae23..4e1cc0b 100644
--- a/build.sh
+++ b/build.sh
@@ -17,7 +17,7 @@
17usage()17usage()
18{18{
19 echo "Please use either --core or --classic"19 echo "Please use either --core or --classic"
20 exit -120 exit 255
21}21}
2222
23core=""23core=""
@@ -41,11 +41,17 @@ done
4141
42set -x42set -x
4343
44if [ "$(systemd-detect-virt)" != "none" ]; then
45 destructive_mode="--destructive-mode"
46else
47 destructive_mode=""
48fi
49
44if [ -z "${core}" ] && [ -z "${classic}" ]; then50if [ -z "${core}" ] && [ -z "${classic}" ]; then
45 usage51 usage
46elif [ ! -z "${core}" ] && [ ! -z "${classic}" ]; then52elif [ -n "${core}" ] && [ -n "${classic}" ]; then
47 usage53 usage
48fi54fi
49sudo snapcraft clean55sudo snapcraft clean "${destructive_mode}"
50sudo snapcraft ${args} prime ${parts[@]}56sudo snapcraft "${args}" prime "${parts[@]}" "${destructive_mode}"
51sudo snapcraft ${args} pack prime57sudo snapcraft "${args}" pack prime
diff --git a/snapcraft.yaml b/snapcraft.yaml
index 86faf9b..16bd3cf 100644
--- a/snapcraft.yaml
+++ b/snapcraft.yaml
@@ -1,5 +1,5 @@
1name: nitrogen6x1name: nitrogen6x
2version: 182version: "18"
3summary: Nitrogen6x support package3summary: Nitrogen6x support package
4description: |4description: |
5 Support files for booting Boundary devices Nitrogen6x5 Support files for booting Boundary devices Nitrogen6x
@@ -12,6 +12,7 @@ grade: stable
12parts:12parts:
13 core-meta:13 core-meta:
14 plugin: nil14 plugin: nil
15 source: .
15 after:16 after:
16 - uboot-fw17 - uboot-fw
17 - uboot-env18 - uboot-env
@@ -24,6 +25,7 @@ parts:
24 cp snap.yaml $SNAPCRAFT_PART_INSTALL/meta25 cp snap.yaml $SNAPCRAFT_PART_INSTALL/meta
25 classic-meta:26 classic-meta:
26 plugin: nil27 plugin: nil
28 source: .
27 after:29 after:
28 - uboot-fw30 - uboot-fw
29 - uboot-bootscript31 - uboot-bootscript

Subscribers

People subscribed via source and target branches

to all changes: