Skip to contentSkip to conversation

Small fixes from bitbake world

PATCH In review Martin Jansa 2 participants 14 messages Dec 5, 2012 – Dec 6, 2012

lore
Share & tools
lore
#0 lore raw

cover Small fixes from bitbake world

The following changes since commit d84f7d7a12b4271f7b2bfde9fb356d750abff15d:

sstate.bbclass: fix detection of owners matching sstate files (2012-12-05 15:34:01 +0000)

are available in the git repository at:

git://git.openembedded.org/openembedded-core-contrib jansa/pull
http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=jansa/pull

Martin Jansa (5):
at: change initscript dependency from build time to runtime scripts/sstate-diff-machines.sh: add simple script to compare sstate
checksums between MACHINEs
scripts/sstate-sysroot-cruft.sh: add simple script to find files in
sysroots not tracked by sstate
git: fix unpackaged python files
python-smartpm: don't try to remove /usr/share/share twice

 meta/recipes-devtools/git/git.inc                  |   4 +-
 meta/recipes-devtools/git/git_1.7.7.bb             |   2 +-
 .../python/python-smartpm_1.4.1.bb                 |   5 +-
 meta/recipes-extended/at/at_3.1.13.bb              |  13 ++-
 scripts/sstate-diff-machines.sh                    | 107 +++++++++++++++++++++
 scripts/sstate-sysroot-cruft.sh                    |  78 +++++++++++++++
 6 files changed, 198 insertions(+), 11 deletions(-)
 create mode 100755 scripts/sstate-diff-machines.sh
 create mode 100755 scripts/sstate-sysroot-cruft.sh
Signature · 1 line
1.8.0
#1 lore raw

PATCH 1/5 at: change initscript dependency from build time to runtime

* doesn't look like build dependency

---
 meta/recipes-extended/at/at_3.1.13.bb | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)
meta/recipes-extended/at/at_3.1.13.bb +9 -4
index 3580e09..198b46e 100644
@@ -4,14 +4,19 @@ the system load levels drop to a particular level."
 SECTION = "base"
 LICENSE="GPLv2+"
 LIC_FILES_CHKSUM = "file://COPYING;md5=4325afd396febcb659c36b49533135d4"
-DEPENDS = "flex flex-native initscripts \
-	${@base_contains('DISTRO_FEATURES', 'pam', 'libpam', '', d)}"
-RDEPENDS_${PN} = "${@base_contains('DISTRO_FEATURES', 'pam', '${PAM_DEPS}', '', d)}"
+DEPENDS = "flex flex-native \
+           ${@base_contains('DISTRO_FEATURES', 'pam', 'libpam', '', d)}"
+
+VIRTUAL-RUNTIME_initscripts ?= "initscripts"                                                                                                                 
+RDEPENDS_${PN} = "${@base_contains('DISTRO_FEATURES', 'pam', '${PAM_DEPS}', '', d)} \
+                  ${VIRTUAL-RUNTIME_initscripts} \
+"
+
 PAM_DEPS = "libpam libpam-runtime pam-plugin-env pam-plugin-limits"
 
 RCONFLICTS_${PN} = "atd"
 RREPLACES_${PN} = "atd"
-PR = "r4"
+PR = "r5"
 
 SRC_URI = "${DEBIAN_MIRROR}/main/a/at/at_${PV}.orig.tar.gz \
     file://configure.patch \
Signature · 1 line
1.8.0
#2 lore raw

PATCH 2/5 scripts/sstate-diff-machines.sh: add simple script to compare sstate checksums between MACHINEs

In reply to Martin Jansa

* takes tmpdir, machines and targets from command arguments or env variables

---
 scripts/sstate-diff-machines.sh | 107 ++++++++++++++++++++++++++++++++++++++++
 1 file changed, 107 insertions(+)
 create mode 100755 scripts/sstate-diff-machines.sh
scripts/sstate-diff-machines.sh new file +107 -0
new file mode 100755
index 0000000..f2c151a
@@ -0,0 +1,107 @@
+#!/bin/sh
+
+# Used to compare sstate checksums between MACHINES
+# Execute script and compare generated list.M files
+
+# It's also usefull to keep older sstate checksums
+# to be able to find out why something is rebuilding 
+# after updating metadata
+
+# $ diff \
+#     sstate-diff/1349348392/fake-cortexa8/list.M \
+#     sstate-diff/1349348392/fake-cortexa9/list.M \
+#     | wc -l
+# 538
+
+# Then to compare sigdata use something like:
+# $ ls sstate-diff/1349348392/*/armv7a-vfp-neon*/linux-libc-headers/*do_configure*sigdata*
+#   sstate-diff/1349348392/fake-cortexa8/armv7a-vfp-neon-oe-linux-gnueabi/linux-libc-headers/3.4.3-r0.do_configure.sigdata.cb73b3630a7b8191e72fc469c5137025
+#   sstate-diff/1349348392/fake-cortexa9/armv7a-vfp-neon-oe-linux-gnueabi/linux-libc-headers/3.4.3-r0.do_configure.sigdata.f37ada177bf99ce8af85914df22b5a0b
+# $ bitbake-diffsigs stamps.1349348392/*/armv7a-vfp-neon*/linux-libc-headers/*do_configure*sigdata*
+#   basehash changed from 8d0bd67bb1da6f68717760fc3ef43171 to e869fa61426e88e9c30726ba88a1216a
+#   Variable TUNE_CCARGS value changed from  -march=armv7-a     -mthumb-interwork -mfloat-abi=softfp -mfpu=neon -mtune=cortex-a8 to  -march=armv7-a     -mthumb-interwork -mfloat-abi=softfp -mfpu=neon -mtune=cortex-a9
+
+# Global vars
+tmpdir=
+machines=
+targets=
+default_machines="qemuarm qemux86 qemux86-64"
+default_targets="core-image-base"
+
+usage () {
+  cat << EOF
+Welcome to utility to compare sstate checksums between different MACHINEs.
+$0 <OPTION>
+
+Options:
+  -h, --help
+        Display this help and exit.
+
+  --tmpdir=<tmpdir>
+        Specify tmpdir, will use the environment variable TMPDIR if it is not specified.
+        Something like /OE/oe-core/tmp-eglibc (no / at the end).
+  
+  --machines=<machines>
+        List of MACHINEs separated by space, will use the environment variable MACHINES if it is not specified.
+        Default value is "qemuarm qemux86 qemux86-64".
+  
+  --targets=<targets>
+        List of targets separated by space, will use the environment variable TARGETS if it is not specified.
+        Default value is "core-image-base".
+EOF
+}
+
+# Print error information and exit.
+echo_error () {
+  echo "ERROR: $1" >&2
+  exit 1
+}
+
+while [ -n "$1" ]; do
+  case $1 in
+    --tmpdir=*)
+      tmpdir=`echo $1 | sed -e 's#^--tmpdir=##' | xargs readlink -e`
+      [ -d "$tmpdir" ] || echo_error "Invalid argument to --tmpdir"
+      shift
+        ;;
+    --machines=*)
+      machines=`echo $1 | sed -e 's#^--machines="*\([^"]*\)"*#\1#'`
+      shift
+        ;;
+    --targets=*)
+      targets=`echo $1 | sed -e 's#^--targets="*\([^"]*\)"*#\1#'`
+      shift
+        ;;
+    --help|-h)
+      usage
+      exit 0
+        ;;
+    *)
+      echo "Invalid arguments $*"
+      echo_error "Try '$0 -h' for more information."
+        ;;
+  esac
+done
+
+# tmpdir directory, use environment variable TMPDIR
+# if it was not specified, otherwise, error.
+[ -n "$tmpdir" ] || tmpdir=$TMPDIR
+[ -n "$tmpdir" ] || echo_error "No tmpdir found!"
+[ -d "$tmpdir" ] || echo_error "Invalid tmpdir \"$tmpdir\""
+[ -n "$machines" ] || machines=$MACHINES
+[ -n "$machines" ] || machines=$default_machines
+[ -n "$targets" ] || targets=$TARGETS
+[ -n "$targets" ] || targets=$default_targets
+
+OUTPUT=${tmpdir}/sstate-diff/`date "+%s"`
+
+for M in ${machines}; do
+  find ${tmpdir}/stamps/ -name \*sigdata\* | xargs rm -f
+  mkdir -p ${OUTPUT}/${M}
+  export MACHINE=${M}; bitbake -S ${targets} | tee -a ${OUTPUT}/${M}/log;
+  cp -ra ${tmpdir}/stamps/* ${OUTPUT}/${M}
+  find ${OUTPUT}/${M} -name \*sigdata\* | sed "s#${OUTPUT}/${M}/##g" | sort > ${OUTPUT}/${M}/list
+  M_UNDERSCORE=`echo ${M} | sed 's/-/_/g'`
+  sed "s/${M_UNDERSCORE}/MACHINE/g; s/${M}/MACHINE/g" ${OUTPUT}/${M}/list | sort > ${OUTPUT}/${M}/list.M
+  find ${tmpdir}/stamps/ -name \*sigdata\* | xargs rm -f
+done
Signature · 1 line
1.8.0
#3 lore raw

PATCH 3/5 scripts/sstate-sysroot-cruft.sh: add simple script to find files in sysroots not tracked by sstate

In reply to Martin Jansa
---
 scripts/sstate-sysroot-cruft.sh | 78 +++++++++++++++++++++++++++++++++++++++++
 1 file changed, 78 insertions(+)
 create mode 100755 scripts/sstate-sysroot-cruft.sh
scripts/sstate-sysroot-cruft.sh new file +78 -0
new file mode 100755
index 0000000..6caa252
@@ -0,0 +1,78 @@
+#!/bin/sh
+
+# Used to find files installed in sysroot which are not tracked by sstate manifest
+
+# Global vars
+tmpdir=
+
+usage () {
+  cat << EOF
+Welcome to sysroot cruft finding utility.
+$0 <OPTION>
+
+Options:
+  -h, --help
+        Display this help and exit.
+
+  --tmpdir=<tmpdir>
+        Specify tmpdir, will use the environment variable TMPDIR if it is not specified.
+	Something like /OE/oe-core/tmp-eglibc (no / at the end).
+EOF
+}
+
+# Print error information and exit.
+echo_error () {
+  echo "ERROR: $1" >&2
+  exit 1
+}
+
+while [ -n "$1" ]; do
+  case $1 in
+    --tmpdir=*)
+      tmpdir=`echo $1 | sed -e 's#^--tmpdir=##' | xargs readlink -e`
+      [ -d "$tmpdir" ] || echo_error "Invalid argument to --tmpdir"
+      shift
+        ;;
+    --help|-h)
+      usage
+      exit 0
+        ;;
+    *)
+      echo "Invalid arguments $*"
+      echo_error "Try '$0 -h' for more information."
+        ;;
+  esac
+done
+
+# sstate cache directory, use environment variable TMPDIR
+# if it was not specified, otherwise, error.
+[ -n "$tmpdir" ] || tmpdir=$TMPDIR
+[ -n "$tmpdir" ] || echo_error "No tmpdir found!"
+[ -d "$tmpdir" ] || echo_error "Invalid tmpdir \"$tmpdir\""
+
+OUTPUT=${tmpdir}/sysroot.cruft.`date "+%s"`
+WHITELIST="\/var\/pseudo\/*[^\/]*$ \/shlibs$ \.pyc$ \.pyo$"
+
+mkdir ${OUTPUT}
+find ${tmpdir}/sstate-control -name \*.populate-sysroot\* -o -name \*.package\* | xargs cat | grep sysroots | \
+  sed 's#/$##g; s#///*#/#g' | \
+  # work around for paths ending with / for directories and multiplied // (e.g. paths to native sysroot)
+  sort > ${OUTPUT}/master.list.all
+sort -u ${OUTPUT}/master.list.all > ${OUTPUT}/master.list # -u because some directories are listed for more recipes
+find ${tmpdir}/sysroots/ | \
+  sort > ${OUTPUT}/sysroot.list
+
+diff ${OUTPUT}/master.list.all ${OUTPUT}/master.list > ${OUTPUT}/duplicates
+diff ${OUTPUT}/master.list ${OUTPUT}/sysroot.list > ${OUTPUT}/diff.all
+
+cp ${OUTPUT}/diff.all ${OUTPUT}/diff
+for item in ${WHITELIST}; do 
+  sed -i "/${item}/d" ${OUTPUT}/diff;
+done
+
+# too many false positives for directories
+# echo "Following files are installed in sysroot at least twice"
+# cat ${OUTPUT}/duplicates
+
+echo "Following files are installed in sysroot, but not tracked by sstate"
+cat ${OUTPUT}/diff
Signature · 1 line
1.8.0
#4 lore raw

Re: [PATCH 3/5] scripts/sstate-sysroot-cruft.sh: add simple script to find files in sysroots not tracked by sstate

Martin Jansa <[email protected]>

Quoted 1 line · writes:

+WHITELIST="\/var\/pseudo\/*[^\/]*$ \/shlibs$ \.pyc$ \.pyo$"

Is it really wanted that this matches paths like '/var/pseudosomepath'?

Enrico

#5 lore raw

Re: [PATCH 3/5] scripts/sstate-sysroot-cruft.sh: add simple script to find files in sysroots not tracked by sstate

Quoted 6 lines · Enrico Scholz

Martin Jansa <[email protected]>

Quoted 1 line · writes:

+WHITELIST="\/var\/pseudo\/*[^\/]*$ \/shlibs$ \.pyc$ \.pyo$"

Is it really wanted that this matches paths like '/var/pseudosomepath'?

.*/var/pseudo
and
.*/var/pseudo/somefile$

which matches also /var/pseudosomepath, I can split it in 2 regexps if that's likely to show in sysroot, or use "\/var\/pseudo\($\|\/[^\/]*$\)"

to match:
/OE/jansa-test/shr-core/tmp-eglibc/sysroots/om-gta02/var/pseudo /OE/jansa-test/shr-core/tmp-eglibc/sysroots/om-gta02/var/pseudo/files.db /OE/jansa-test/shr-core/tmp-eglibc/sysroots/om-gta02/var/pseudo/logs.db /OE/jansa-test/shr-core/tmp-eglibc/sysroots/om-gta02/var/pseudo/pseudo.lock /OE/jansa-test/shr-core/tmp-eglibc/sysroots/om-gta02/var/pseudo/pseudo.log /OE/jansa-test/shr-core/tmp-eglibc/sysroots/om-gta02/var/pseudo/pseudo.pid /OE/jansa-test/shr-core/tmp-eglibc/sysroots/om-gta02/var/pseudo/pseudo.socket

Cheers,

Signature · 1 line
Martin 'JaMa' Jansa     jabber: [email protected]
#6 lore raw

Re: [PATCH 3/5] scripts/sstate-sysroot-cruft.sh: add simple script to find files in sysroots not tracked by sstate

Quoted 6 lines · Martin Jansa

.*/var/pseudo
and
.*/var/pseudo/somefile$

which matches also /var/pseudosomepath, I can split it in 2 regexps if that's likely to show in sysroot, or use "\/var\/pseudo\($\|\/[^\/]*$\)"

A basic regexp '/var/pseudo\(/.*\)\?$' should suffice. Btw, to avoid escaping of '/', you can write

  sed -i "\!${var}!d"

Enrico

#7 lore raw

Re: [PATCH 3/5] scripts/sstate-sysroot-cruft.sh: add simple script to find files in sysroots not tracked by sstate

That would match .*/var/pseudo/too/bar,wouldn't it? but true that making that group optional is probably easier to read.. But who doesn't read regexps fluently nowadays? :)

Quoted 17 lines · Enrico Scholz
Quoted 6 lines · Martin Jansa

.*/var/pseudo
and
.*/var/pseudo/somefile$

which matches also /var/pseudosomepath, I can split it in 2 regexps if that's likely to show in sysroot, or use "\/var\/pseudo\($\|\/[^\/]*$\)"

A basic regexp '/var/pseudo\(/.*\)\?$' should suffice. Btw, to avoid escaping of '/', you can write

  sed -i "\!${var}!d"

Enrico

#8 lore raw

Re: [PATCH 3/5] scripts/sstate-sysroot-cruft.sh: add simple script to find files in sysroots not tracked by sstate

Quoted 1 line · Martin Jansa

That would match .*/var/pseudo/too/bar,wouldn't it?

yes; when you really want to exclude such subpaths, you can match against '/var/pseudo\(/[^/]*\)\?$'.

Enrico

#9 lore raw

PATCH v2 3/5 scripts/sstate-sysroot-cruft.sh: add simple script to find files in sysroots not tracked by sstate

In reply to Martin Jansa

---

v2: fixed WHITELIST for var/pseudo* as Enrico reported

 scripts/sstate-sysroot-cruft.sh | 78 +++++++++++++++++++++++++++++++++++++++++
 1 file changed, 78 insertions(+)
 create mode 100755 scripts/sstate-sysroot-cruft.sh
scripts/sstate-sysroot-cruft.sh new file +78 -0
new file mode 100755
index 0000000..530b20d
@@ -0,0 +1,78 @@
+#!/bin/sh
+
+# Used to find files installed in sysroot which are not tracked by sstate manifest
+
+# Global vars
+tmpdir=
+
+usage () {
+  cat << EOF
+Welcome to sysroot cruft finding utility.
+$0 <OPTION>
+
+Options:
+  -h, --help
+        Display this help and exit.
+
+  --tmpdir=<tmpdir>
+        Specify tmpdir, will use the environment variable TMPDIR if it is not specified.
+	Something like /OE/oe-core/tmp-eglibc (no / at the end).
+EOF
+}
+
+# Print error information and exit.
+echo_error () {
+  echo "ERROR: $1" >&2
+  exit 1
+}
+
+while [ -n "$1" ]; do
+  case $1 in
+    --tmpdir=*)
+      tmpdir=`echo $1 | sed -e 's#^--tmpdir=##' | xargs readlink -e`
+      [ -d "$tmpdir" ] || echo_error "Invalid argument to --tmpdir"
+      shift
+        ;;
+    --help|-h)
+      usage
+      exit 0
+        ;;
+    *)
+      echo "Invalid arguments $*"
+      echo_error "Try '$0 -h' for more information."
+        ;;
+  esac
+done
+
+# sstate cache directory, use environment variable TMPDIR
+# if it was not specified, otherwise, error.
+[ -n "$tmpdir" ] || tmpdir=$TMPDIR
+[ -n "$tmpdir" ] || echo_error "No tmpdir found!"
+[ -d "$tmpdir" ] || echo_error "Invalid tmpdir \"$tmpdir\""
+
+OUTPUT=${tmpdir}/sysroot.cruft.`date "+%s"`
+WHITELIST="\/var\/pseudo\($\|\/[^\/]*$\) \/shlibs$ \.pyc$ \.pyo$"
+
+mkdir ${OUTPUT}
+find ${tmpdir}/sstate-control -name \*.populate-sysroot\* -o -name \*.package\* | xargs cat | grep sysroots | \
+  sed 's#/$##g; s#///*#/#g' | \
+  # work around for paths ending with / for directories and multiplied // (e.g. paths to native sysroot)
+  sort > ${OUTPUT}/master.list.all
+sort -u ${OUTPUT}/master.list.all > ${OUTPUT}/master.list # -u because some directories are listed for more recipes
+find ${tmpdir}/sysroots/ | \
+  sort > ${OUTPUT}/sysroot.list
+
+diff ${OUTPUT}/master.list.all ${OUTPUT}/master.list > ${OUTPUT}/duplicates
+diff ${OUTPUT}/master.list ${OUTPUT}/sysroot.list > ${OUTPUT}/diff.all
+
+cp ${OUTPUT}/diff.all ${OUTPUT}/diff
+for item in ${WHITELIST}; do 
+  sed -i "/${item}/d" ${OUTPUT}/diff;
+done
+
+# too many false positives for directories
+# echo "Following files are installed in sysroot at least twice"
+# cat ${OUTPUT}/duplicates
+
+echo "Following files are installed in sysroot, but not tracked by sstate"
+cat ${OUTPUT}/diff
Signature · 1 line
1.8.0
#10 lore raw

PATCH 4/5 git: fix unpackaged python files

In reply to Martin Jansa

* fixes:
git-1.7.7: git: Files/directories were installed but not shipped
/usr/lib64
/usr/lib64/python2.7
/usr/lib64/python2.7/site-packages /usr/lib64/python2.7/site-packages/git_remote_helpers-0.1.0-py2.7.egg-info /usr/lib64/python2.7/site-packages/git_remote_helpers /usr/lib64/python2.7/site-packages/git_remote_helpers/__init__.py /usr/lib64/python2.7/site-packages/git_remote_helpers/util.pyc /usr/lib64/python2.7/site-packages/git_remote_helpers/__init__.pyc /usr/lib64/python2.7/site-packages/git_remote_helpers/util.py /usr/lib64/python2.7/site-packages/git_remote_helpers/git /usr/lib64/python2.7/site-packages/git_remote_helpers/git/__init__.py /usr/lib64/python2.7/site-packages/git_remote_helpers/git/importer.pyc /usr/lib64/python2.7/site-packages/git_remote_helpers/git/non_local.pyc /usr/lib64/python2.7/site-packages/git_remote_helpers/git/exporter.py /usr/lib64/python2.7/site-packages/git_remote_helpers/git/git.py /usr/lib64/python2.7/site-packages/git_remote_helpers/git/repo.py /usr/lib64/python2.7/site-packages/git_remote_helpers/git/__init__.pyc /usr/lib64/python2.7/site-packages/git_remote_helpers/git/importer.py /usr/lib64/python2.7/site-packages/git_remote_helpers/git/exporter.pyc /usr/lib64/python2.7/site-packages/git_remote_helpers/git/repo.pyc /usr/lib64/python2.7/site-packages/git_remote_helpers/git/git.pyc /usr/lib64/python2.7/site-packages/git_remote_helpers/git/non_local.py

* fixes [YOCTO #3344]

---
 meta/recipes-devtools/git/git.inc      | 4 ++--
 meta/recipes-devtools/git/git_1.7.7.bb | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)
meta/recipes-devtools/git/git.inc +2 -2
index 55e1482..80a88c2 100644
@@ -16,7 +16,7 @@ EXTRA_OECONF = "--with-perl=${STAGING_BINDIR_NATIVE}/perl-native/perl \
 
 EXTRA_OEMAKE = "NO_PYTHON=1"
 
-inherit autotools perlnative
+inherit autotools perlnative python-dir
 
 do_compile_prepend () {
 	# Remove perl/perl.mak to fix the out-of-date perl.mak error
@@ -58,7 +58,7 @@ FILES_${PN}-perltools += " \
 RDEPENDS_${PN}-perltools = "${PN} perl perl-module-file-path findutils"
 
 PACKAGES =+ "${PN}-python"
-FILES_${PN}-python = "${libdir}/python*"
+FILES_${PN}-python = "${libdir}/python* ${PYTHON_SITEPACKAGES_DIR}"
 
 
 # git-tk package with gitk and git-gui
meta/recipes-devtools/git/git_1.7.7.bb +1 -1
index 9d24911..6b6e04b 100644
@@ -1,6 +1,6 @@
 require git.inc
 
-PR = "r2"
+PR = "r3"
 
 SRC_URI = "http://git-core.googlecode.com/files/git-${PV}.tar.gz"
 SRC_URI[md5sum] = "5d645884e688921e773186783b65ce33"
Signature · 1 line
1.8.0
#11 lore raw

Re: [PATCH 4/5] git: fix unpackaged python files

Quoted 60 lines · Martin Jansa

* fixes:
git-1.7.7: git: Files/directories were installed but not shipped
/usr/lib64
/usr/lib64/python2.7
/usr/lib64/python2.7/site-packages /usr/lib64/python2.7/site-packages/git_remote_helpers-0.1.0-py2.7.egg-info /usr/lib64/python2.7/site-packages/git_remote_helpers /usr/lib64/python2.7/site-packages/git_remote_helpers/__init__.py /usr/lib64/python2.7/site-packages/git_remote_helpers/util.pyc /usr/lib64/python2.7/site-packages/git_remote_helpers/__init__.pyc /usr/lib64/python2.7/site-packages/git_remote_helpers/util.py /usr/lib64/python2.7/site-packages/git_remote_helpers/git /usr/lib64/python2.7/site-packages/git_remote_helpers/git/__init__.py /usr/lib64/python2.7/site-packages/git_remote_helpers/git/importer.pyc /usr/lib64/python2.7/site-packages/git_remote_helpers/git/non_local.pyc /usr/lib64/python2.7/site-packages/git_remote_helpers/git/exporter.py /usr/lib64/python2.7/site-packages/git_remote_helpers/git/git.py /usr/lib64/python2.7/site-packages/git_remote_helpers/git/repo.py /usr/lib64/python2.7/site-packages/git_remote_helpers/git/__init__.pyc /usr/lib64/python2.7/site-packages/git_remote_helpers/git/importer.py /usr/lib64/python2.7/site-packages/git_remote_helpers/git/exporter.pyc /usr/lib64/python2.7/site-packages/git_remote_helpers/git/repo.pyc /usr/lib64/python2.7/site-packages/git_remote_helpers/git/git.pyc /usr/lib64/python2.7/site-packages/git_remote_helpers/git/non_local.py

* fixes [YOCTO #3344]

---
 meta/recipes-devtools/git/git.inc      | 4 ++--
 meta/recipes-devtools/git/git_1.7.7.bb | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)
meta/recipes-devtools/git/git.inc +2 -2
index 55e1482..80a88c2 100644
@@ -16,7 +16,7 @@ EXTRA_OECONF = "--with-perl=${STAGING_BINDIR_NATIVE}/perl-native/perl \
 
 EXTRA_OEMAKE = "NO_PYTHON=1"
 
-inherit autotools perlnative
+inherit autotools perlnative python-dir
 
 do_compile_prepend () {
 	# Remove perl/perl.mak to fix the out-of-date perl.mak error
@@ -58,7 +58,7 @@ FILES_${PN}-perltools += " \
 RDEPENDS_${PN}-perltools = "${PN} perl perl-module-file-path findutils"
 
 PACKAGES =+ "${PN}-python"
-FILES_${PN}-python = "${libdir}/python*"
+FILES_${PN}-python = "${libdir}/python* ${PYTHON_SITEPACKAGES_DIR}"
 
 
 # git-tk package with gitk and git-gui
meta/recipes-devtools/git/git_1.7.7.bb +0 -0
index 9d24911..6b6e04b 100644

meta/recipes-devtools/git/git_1.7.7.bb line 6

Quoted 7 lines
meta/recipes-devtools/git/git_1.7.7.bb +1 -1
@@ -1,6 +1,6 @@
 require git.inc
 
-PR = "r2"
+PR = "r3"
 
 SRC_URI = "http://git-core.googlecode.com/files/git-${PV}.tar.gz"
 SRC_URI[md5sum] = "5d645884e688921e773186783b65ce33"

Oh, this isn't sometimes enough.. in some cases like commit message it's using wrong python-config and installing to paths like on host (lib64).

Sending v2 soon

Cheers,

Signature · 1 line
Martin 'JaMa' Jansa     jabber: [email protected]
#12 lore raw

PATCH v2 4/5 git: fix unpackaged python files

In reply to Martin Jansa

* EXTRA_OEMAKE is needed after inheriting autotools which sets it to empty
* fixes:
git-1.7.7: git: Files/directories were installed but not shipped
/usr/lib64
/usr/lib64/python2.7
/usr/lib64/python2.7/site-packages /usr/lib64/python2.7/site-packages/git_remote_helpers-0.1.0-py2.7.egg-info /usr/lib64/python2.7/site-packages/git_remote_helpers /usr/lib64/python2.7/site-packages/git_remote_helpers/__init__.py /usr/lib64/python2.7/site-packages/git_remote_helpers/util.pyc /usr/lib64/python2.7/site-packages/git_remote_helpers/__init__.pyc /usr/lib64/python2.7/site-packages/git_remote_helpers/util.py /usr/lib64/python2.7/site-packages/git_remote_helpers/git /usr/lib64/python2.7/site-packages/git_remote_helpers/git/__init__.py /usr/lib64/python2.7/site-packages/git_remote_helpers/git/importer.pyc /usr/lib64/python2.7/site-packages/git_remote_helpers/git/non_local.pyc /usr/lib64/python2.7/site-packages/git_remote_helpers/git/exporter.py /usr/lib64/python2.7/site-packages/git_remote_helpers/git/git.py /usr/lib64/python2.7/site-packages/git_remote_helpers/git/repo.py /usr/lib64/python2.7/site-packages/git_remote_helpers/git/__init__.pyc /usr/lib64/python2.7/site-packages/git_remote_helpers/git/importer.py /usr/lib64/python2.7/site-packages/git_remote_helpers/git/exporter.pyc /usr/lib64/python2.7/site-packages/git_remote_helpers/git/repo.pyc /usr/lib64/python2.7/site-packages/git_remote_helpers/git/git.pyc /usr/lib64/python2.7/site-packages/git_remote_helpers/git/non_local.py

* fixes [YOCTO #3344]

---

v2: should be disabled correctly now, instead of packaging them

 meta/recipes-devtools/git/git.inc      | 4 ++--
 meta/recipes-devtools/git/git_1.7.7.bb | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)
meta/recipes-devtools/git/git.inc +2 -2
index 55e1482..82df6e3 100644
@@ -14,10 +14,10 @@ EXTRA_OECONF = "--with-perl=${STAGING_BINDIR_NATIVE}/perl-native/perl \
 		--without-tcltk \
 "
 
-EXTRA_OEMAKE = "NO_PYTHON=1"
-
 inherit autotools perlnative
 
+EXTRA_OEMAKE = "NO_PYTHON=1"
+
 do_compile_prepend () {
 	# Remove perl/perl.mak to fix the out-of-date perl.mak error
 	# during rebuild
meta/recipes-devtools/git/git_1.7.7.bb +1 -1
index 9d24911..6b6e04b 100644
@@ -1,6 +1,6 @@
 require git.inc
 
-PR = "r2"
+PR = "r3"
 
 SRC_URI = "http://git-core.googlecode.com/files/git-${PV}.tar.gz"
 SRC_URI[md5sum] = "5d645884e688921e773186783b65ce33"
Signature · 1 line
1.8.0
#13 lore raw

PATCH 5/5 python-smartpm: don't try to remove /usr/share/share twice

In reply to Martin Jansa

* distutils.bbclass now removes that directory

---
 meta/recipes-devtools/python/python-smartpm_1.4.1.bb | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)
meta/recipes-devtools/python/python-smartpm_1.4.1.bb +1 -4
index 254318c..b896c89 100644
@@ -11,7 +11,7 @@ LICENSE = "GPLv2"
 LIC_FILES_CHKSUM = "file://LICENSE;md5=393a5ca445f6965873eca0259a17f833"
 
 DEPENDS = "python rpm"
-PR = "r1"
+PR = "r2"
 SRCNAME = "smart"
 
 SRC_URI = "\
@@ -43,9 +43,6 @@ PACKAGECONFIG[gtk] = ",,gtk+,${GTK_RDEP}"
 inherit distutils
 
 do_install_append() {
-   # Cleanup unused item...
-   rmdir ${D}${datadir}/share
-
    # We don't support the following items
    rm -rf ${D}${libdir}/python*/site-packages/smart/backends/slack
    rm -rf ${D}${libdir}/python*/site-packages/smart/backends/arch
Signature · 1 line
1.8.0