]> code.ossystems Code Review - openembedded-core.git/commitdiff
classes/image: implement generic locale package installation
authorPaul Eggleton <paul.eggleton@linux.intel.com>
Thu, 22 Dec 2011 14:46:32 +0000 (14:46 +0000)
committerSaul Wold <sgw@linux.intel.com>
Tue, 3 Jan 2012 04:26:25 +0000 (20:26 -0800)
Let each package-specific rootfs implementation provide basic functions
to query the existence of a package and install a list of packages and
then have a generic install function so this logic is in one place.

Note: unlike previous versions of this code in OE-Core this uses the
IMAGE_LINGUAS variable and not IMAGE_LOCALES - note that IMAGE_LINGUAS
was what was used in OE-Classic and it is already used in OE-Core in
order to install locale-base-*. This will mean that if IMAGE_LINGUAS is
left at the default you will now get more packages installed. If you
don't want these language support packages then you should set
IMAGE_LINGUAS explicitly.

This restores locale installation to the same state as OE-Classic, only
we now support all the packaging backends.

Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
meta/classes/image.bbclass
meta/classes/rootfs_deb.bbclass
meta/classes/rootfs_ipk.bbclass
meta/classes/rootfs_rpm.bbclass

index 865d430121d2f2e695d0e41ab81c6b8d63d0ac60..275b28f4ac798ec15ee4b4285da1524a63fbf0e4 100644 (file)
@@ -255,6 +255,43 @@ multilib_sanity_check() {
   echo $@ | python ${MULTILIB_CHECK_FILE}
 }
 
+get_split_linguas() {
+    for translation in ${IMAGE_LINGUAS}; do
+        translation_split=$(echo ${translation} | awk -F '-' '{print $1}')
+        echo ${translation}
+        echo ${translation_split}
+    done | sort | uniq
+}
+
+rootfs_install_all_locales() {
+    # Generate list of installed packages for which additional locale packages might be available
+    INSTALLED_PACKAGES=`list_installed_packages | egrep -v -- "(-locale-|^locale-base-|-dev$|-doc$|^kernel|^glibc|^ttf|^task|^perl|^python)"`
+
+    # Generate a list of locale packages that exist
+    SPLIT_LINGUAS=`get_split_linguas`
+    PACKAGES_TO_INSTALL=""
+    for lang in $SPLIT_LINGUAS; do
+        for pkg in $INSTALLED_PACKAGES; do
+            existing_pkg=`rootfs_check_package_exists $pkg-locale-$lang`
+            if [ "$existing_pkg" != "" ]; then
+                PACKAGES_TO_INSTALL="$PACKAGES_TO_INSTALL $existing_pkg"
+            fi
+        done
+    done
+
+    # Install the packages, if any
+    if [ "$PACKAGES_TO_INSTALL" != "" ]; then
+        rootfs_install_packages $PACKAGES_TO_INSTALL
+    fi
+
+    # Workaround for broken shell function dependencies
+    if false ; then
+        get_split_linguas
+        list_installed_packages
+        rootfs_check_package_exists
+    fi
+}
+
 # set '*' as the root password so the images
 # can decide if they want it or not
 zap_root_password () {
index bef055c6f5f1f832201d83154756b2475faad88a..b6c706c580e6cbe0af399c8e29872b708d1fff6f 100644 (file)
@@ -8,8 +8,18 @@ ROOTFS_PKGMANAGE_BOOTSTRAP  = "run-postinsts"
 do_rootfs[depends] += "dpkg-native:do_populate_sysroot apt-native:do_populate_sysroot"
 do_rootfs[recrdeptask] += "do_package_write_deb"
 
+DEB_POSTPROCESS_COMMANDS = "rootfs_install_all_locales; "
+
 opkglibdir = "${localstatedir}/lib/opkg"
 
+deb_package_setflag() {
+       sed -i -e "/^Package: $2\$/{n; s/Status: install ok .*/Status: install ok $1/;}" ${IMAGE_ROOTFS}/var/lib/dpkg/status
+}
+
+deb_package_getflag() {
+       cat ${IMAGE_ROOTFS}/var/lib/dpkg/status | sed -n -e "/^Package: $2\$/{n; s/Status: install ok .*/$1/; p}"
+}
+
 fakeroot rootfs_deb_do_rootfs () {
        set +e
 
@@ -28,25 +38,18 @@ fakeroot rootfs_deb_do_rootfs () {
        export INSTALL_TASK_DEB="rootfs"
 
        package_install_internal_deb
-
+       ${DEB_POSTPROCESS_COMMANDS}
 
        export D=${IMAGE_ROOTFS}
        export OFFLINE_ROOT=${IMAGE_ROOTFS}
        export IPKG_OFFLINE_ROOT=${IMAGE_ROOTFS}
        export OPKG_OFFLINE_ROOT=${IMAGE_ROOTFS}
 
-       _flag () {
-               sed -i -e "/^Package: $2\$/{n; s/Status: install ok .*/Status: install ok $1/;}" ${IMAGE_ROOTFS}/var/lib/dpkg/status
-       }
-       _getflag () {
-               cat ${IMAGE_ROOTFS}/var/lib/dpkg/status | sed -n -e "/^Package: $2\$/{n; s/Status: install ok .*/$1/; p}"
-       }
-
        # Attempt to run preinsts
        # Mark packages with preinst failures as unpacked
        for i in ${IMAGE_ROOTFS}/var/lib/dpkg/info/*.preinst; do
                if [ -f $i ] && ! sh $i; then
-                       _flag unpacked `basename $i .preinst`
+                       deb_package_setflag unpacked `basename $i .preinst`
                fi
        done
 
@@ -54,7 +57,7 @@ fakeroot rootfs_deb_do_rootfs () {
        # Mark packages with postinst failures as unpacked
        for i in ${IMAGE_ROOTFS}/var/lib/dpkg/info/*.postinst; do
                if [ -f $i ] && ! sh $i configure; then
-                       _flag unpacked `basename $i .postinst`
+                       deb_package_setflag unpacked `basename $i .postinst`
                fi
        done
 
@@ -104,3 +107,17 @@ list_package_depends() {
 list_package_recommends() {
        ${DPKG_QUERY_COMMAND} -s $1 | grep ^Recommends | sed -e 's/^Recommends: //' -e 's/,//g' -e 's:([=<>]* [0-9a-zA-Z.~\-]*)::g'
 }
+
+rootfs_check_package_exists() {
+       if [ `apt-cache showpkg $1 | wc -l` -gt 2 ]; then
+               echo $1
+       fi
+}
+
+rootfs_install_packages() {
+       ${STAGING_BINDIR_NATIVE}/apt-get install $@ --force-yes --allow-unauthenticated
+
+       for pkg in $@ ; do
+               deb_package_setflag installed $pkg
+       done
+}
index b5556fa254474fb25d1340814bdc218c2f5d5999..48fb2fbd1889999e44c44750458cfa1a4cb78eef 100644 (file)
@@ -16,7 +16,7 @@ IPKG_ARGS = "-f ${IPKGCONF_TARGET} -o ${IMAGE_ROOTFS} --force-overwrite"
 
 OPKG_PREPROCESS_COMMANDS = "package_update_index_ipk; package_generate_ipkg_conf"
 
-OPKG_POSTPROCESS_COMMANDS = "ipk_insert_feed_uris"
+OPKG_POSTPROCESS_COMMANDS = "ipk_insert_feed_uris; rootfs_install_all_locales; "
 
 opkglibdir = "${localstatedir}/lib/opkg"
 
@@ -148,26 +148,14 @@ list_package_recommends() {
        opkg-cl ${IPKG_ARGS} info $1 | grep ^Recommends | sed -e 's/^Recommends: //' -e 's/,//g' -e 's:([=<>]* [0-9a-zA-Z.~\-]*)::g'
 }
 
-install_all_locales() {
-
-    PACKAGES_TO_INSTALL=""
-
-    INSTALLED_PACKAGES=`list_installed_packages | egrep -v -- "(-locale-|-dev$|-doc$|^kernel|^glibc|^ttf|^task|^perl|^python)"`
-
-    for pkg in $INSTALLED_PACKAGES
-    do
-        for lang in ${IMAGE_LOCALES}
-        do
-            if [ `opkg-cl ${IPKG_ARGS} info $pkg-locale-$lang | wc -l` -gt 2 ]
-            then
-                    PACKAGES_TO_INSTALL="$PACKAGES_TO_INSTALL $pkg-locale-$lang"
-            fi
-        done
-    done
-    if [ "$PACKAGES_TO_INSTALL" != "" ]
-    then
-        opkg-cl ${IPKG_ARGS} install $PACKAGES_TO_INSTALL
-    fi
+rootfs_check_package_exists() {
+       if [ `opkg-cl ${IPKG_ARGS} info $1 | wc -l` -gt 2 ]; then
+               echo $1
+       fi
+}
+
+rootfs_install_packages() {
+       opkg-cl ${IPKG_ARGS} install $PACKAGES_TO_INSTALL
 }
 
 ipk_insert_feed_uris () {
index 966b074181cf790ff4e64176a8dc8098e56277cd..30f3783e3c6d0501207596fc9d36a79dd2efe02a 100644 (file)
@@ -21,12 +21,7 @@ do_rootfs[depends] += "opkg-native:do_populate_sysroot"
 do_rootfs[recrdeptask] += "do_package_write_rpm"
 
 RPM_PREPROCESS_COMMANDS = "package_update_index_rpm; package_generate_rpm_conf; "
-RPM_POSTPROCESS_COMMANDS = ""
-
-# To test the install_all_locales.. enable the following...
-#RPM_POSTPROCESS_COMMANDS = "install_all_locales; "
-#
-#IMAGE_LOCALES="en-gb"
+RPM_POSTPROCESS_COMMANDS = "rootfs_install_all_locales; "
 
 # 
 # Allow distributions to alter when [postponed] package install scripts are run
@@ -198,24 +193,16 @@ list_package_recommends() {
        ${RPM_QUERY_CMD} -q --suggests $1
 }
 
-install_all_locales() {
-       PACKAGES_TO_INSTALL=""
-
-       # Generate list of installed packages...
-       INSTALLED_PACKAGES=`list_installed_packages | egrep -v -- "(-locale-|-dev$|-doc$|^kernel|^glibc|^ttf|^task|^perl|^python)"`
-
-       # This would likely be faster if we did it in one transaction
-       # but this should be good enough for the few users of this function...
-       for pkg in $INSTALLED_PACKAGES; do
-               for lang in ${IMAGE_LOCALES}; do
-                       pkg_name=$(resolve_package_rpm $pkg-locale-$lang ${RPMCONF_TARGET_BASE}.conf)
-                       if [ -n "$pkg_name" ]; then
-                               ${RPM} --root ${IMAGE_ROOTFS} -D "_dbpath ${rpmlibdir}" \
-                                       -D "__dbi_txn create nofsync private" \
-                                       --noscripts --notriggers --noparentdirs --nolinktos \
-                                       -Uhv $pkg_name || true
-                       fi
-               done
+rootfs_check_package_exists() {
+       resolve_package_rpm ${RPMCONF_TARGET_BASE}-base_archs.conf $1
+}
+
+rootfs_install_packages() {
+       for pkg in $@; do
+               ${RPM} --root ${IMAGE_ROOTFS} -D "_dbpath ${rpmlibdir}" \
+                       -D "__dbi_txn create nofsync private" \
+                       --noscripts --notriggers --noparentdirs --nolinktos \
+                       -Uhv $pkg || true
        done
 }