On Thu, Nov 15, 2012 at 12:30 AM, Martin Jansa <[email protected]>wrote:

> * it's not very universal, but works with default oe-core setup and
>   shows basic HOW-TO. It can be improved later.
>
> Signed-off-by: Martin Jansa <[email protected]>
> ---
>  V2: added .pyo to WHITELIST
>      shorter filenames
>      TMPDIR
>      added duplicates but not shown
>
>  V3: use also populate-sysroot.MACHINE, manifest name for populate-sysroot
>      was changed in febeaf3d1b8917b660c7279b008d8b03337568e9
>
>  V4: dropped eglibc-initial work around, it was fixed in oe-core
>
>  scripts/sstate-sysroot-cruft.sh | 34 ++++++++++++++++++++++++++++++++++
>  1 file changed, 34 insertions(+)
>  create mode 100755 scripts/sstate-sysroot-cruft.sh
>
> diff --git a/scripts/sstate-sysroot-cruft.sh
> b/scripts/sstate-sysroot-cruft.sh
> new file mode 100755
> index 0000000..ca23dcf
> --- /dev/null
> +++ b/scripts/sstate-sysroot-cruft.sh
> @@ -0,0 +1,34 @@
> +#!/bin/sh
> +
> +# Used to find files installed in sysroot which are not tracked by sstate
> manifest
> +# Update BASE
> +
> +BASE="/OE/oe-core"
>

This seems interesting, but I have a few comments/concerns.

1) don't hardcode BASE, figure out the path relative to the script's
location, e.g. BASE="$(cd $(dirname $(dirname $0)) && pwd)"
2) output files shouldn't go into oe-core directly, as oe-core isn't
guaranteed to be writable, and it's more common to expect output from a
script like this to go relative to the current directory, or a temp
directory
3) extract TMPDIR from bitbake -e, rather than hardcoding that, as that
breaks for any distros or users which separate their tmpdirs by distro, or
set TCLIBCAPPEND = ""
-- 
Christopher Larson
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

Reply via email to