Skip to contentSkip to conversation

Manifest inclusion on image fix.

PATCH Applied Beth Flanagan 2 participants 3 messages Dec 19, 2011 – Dec 21, 2011

lore
Share & tools
lore
#0 lore raw

cover Manifest inclusion on image fix.

The manifest conditionals COPY_LIC_MANIFEST and COPY_LIC_DIRS were returning true due to missing quotes in the bash test. This made images larger than expected.

The following changes since commit 45987c5135d22757ed2eb06b37e192d73309f59f:

eglibc: Fixes for running regression tests (2011-12-19 09:29:18 +0000)

are available in the git repository at:
git://git.yoctoproject.org/poky-contrib eflanagan/manifest
http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=eflanagan/manifest

Beth Flanagan (1):
license: Fix manifest conditional

 meta/classes/license.bbclass |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)
#1 lore raw

PATCH 1/1 license: Fix manifest conditional

The manifest conditionals COPY_LIC_MANIFEST and COPY_LIC_DIRS were returning true due to missing quotes in the bash test. This made images larger than expected.

---
 meta/classes/license.bbclass |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)
meta/classes/license.bbclass +2 -2
index 6e82bd9..d351b5a 100644
@@ -110,10 +110,10 @@ license_create_manifest() {
     # if you are concerned about license compliance 
     # and delivery this should cover all your bases
 
-    if [ -n ${COPY_LIC_MANIFEST} ]; then
+    if [ -n "${COPY_LIC_MANIFEST}" ]; then
         mkdir -p ${IMAGE_ROOTFS}/usr/share/common-licenses/
         cp ${LICENSE_DIRECTORY}/${IMAGE_NAME}/license.manifest ${IMAGE_ROOTFS}/usr/share/common-licenses/license.manifest
-        if [ -n ${COPY_LIC_DIRS} ]; then
+        if [ -n "${COPY_LIC_DIRS}" ]; then
             for pkg in ${INSTALLED_PKGS}; do
                 mkdir -p ${IMAGE_ROOTFS}/usr/share/common-licenses/${pkg}
                 for lic in `ls ${LICENSE_DIRECTORY}/${pkged_pn}`; do
Signature · 1 line
1.7.1
#2 lore raw

Re: [PATCH 1/1] license: Fix manifest conditional

Quoted 8 lines · Beth Flanagan

The manifest conditionals COPY_LIC_MANIFEST and COPY_LIC_DIRS were returning true due to missing quotes in the bash test. This made images larger than expected.

---
 meta/classes/license.bbclass |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

Merged to master, thanks.

Richard