From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-ee0-f45.google.com (mail-ee0-f45.google.com [74.125.83.45]) by mail.openembedded.org (Postfix) with ESMTP id 63F966DF23 for ; Thu, 5 Dec 2013 11:49:29 +0000 (UTC) Received: by mail-ee0-f45.google.com with SMTP id d49so3188614eek.4 for ; Thu, 05 Dec 2013 03:49:28 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=date:from:to:cc:subject:message-id:references:mime-version :content-type:content-disposition:in-reply-to:user-agent; bh=IBtIhS6fjT+q2Tc4qqmbvx79bLrFm+vinhXPyiuPNn8=; b=NLAz+7TvqccLYFc3zdttBpHU3NeVTBL6JaY19LdoxHvWmWXuK+YvKdCES/PIAzh4HV QthYvhADsjPG6m89FQ2X8bXxEsOMorRZMrLUOlnYKHfIC3W3kkmhsIYTCrxhQA4TeleK YvQHfcluvY79QIl3otgwuDxlRLkzsHvaoT9FyTX4Y9J1jnoxpEKlxZavVepuq/s5cw47 ZKoROoK2eKwuEGnJSwdqAzyCQkf58myo0zIZJ2f2XLQ+d37eUZW5uBfiJn+JMbDUIhcv AZmdM34YK9QwFkQvBjseUzDpef7yFoN0FEbDBftXK+HVMX8nm1bEB+ut57mHvzm6LUiu I/dQ== X-Received: by 10.15.76.6 with SMTP id m6mr15988549eey.37.1386244168036; Thu, 05 Dec 2013 03:49:28 -0800 (PST) Received: from localhost (ip-89-176-104-107.net.upcbroadband.cz. [89.176.104.107]) by mx.google.com with ESMTPSA id p45sm15084186eeg.1.2013.12.05.03.49.26 for (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 05 Dec 2013 03:49:27 -0800 (PST) Date: Thu, 5 Dec 2013 12:49:39 +0100 From: Martin Jansa To: Iyad Qumei Message-ID: <20131205114939.GD3724@jama> References: <1386218151-16177-1-git-send-email-iyadkq@gmail.com> MIME-Version: 1.0 In-Reply-To: <1386218151-16177-1-git-send-email-iyadkq@gmail.com> User-Agent: Mutt/1.5.22 (2013-10-16) Cc: openembedded-core@lists.openembedded.org Subject: Re: [PATCH v2] icecc-create-env: Upgrade script X-BeenThere: openembedded-core@lists.openembedded.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: Patches and discussions about the oe-core layer List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 05 Dec 2013 11:49:30 -0000 X-Groupsio-MsgNum: 47931 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="llIrKcgUOe3dCx0c" Content-Disposition: inline --llIrKcgUOe3dCx0c Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Dec 04, 2013 at 08:35:51PM -0800, Iyad Qumei wrote: > The original icecc script does not handle toolchain content > properly, resulting in build failures, such as >=20 > gxx: fatal error: -fuse-linker-plugin, but liblto_plugin.so not found >=20 > This patch brings in the latest script form the icecream source, and > modifies it for use in the OE build environment. The change was tested > with OE-Core built toolchain, and external toolchain. >=20 > Signed-off-by: Iyad Qumei > --- > .../icecc-create-env-native/icecc-create-env | 266 ++++++++++++++= +----- > .../icecc-create-env-native_0.1.bb | 26 -- > .../icecc-create-env-native_1.0.1.bb | 26 ++ > 3 files changed, 233 insertions(+), 85 deletions(-) > delete mode 100644 meta/recipes-devtools/icecc-create-env/icecc-create-e= nv-native_0.1.bb > create mode 100644 meta/recipes-devtools/icecc-create-env/icecc-create-e= nv-native_1.0.1.bb >=20 > diff --git a/meta/recipes-devtools/icecc-create-env/icecc-create-env-nati= ve/icecc-create-env b/meta/recipes-devtools/icecc-create-env/icecc-create-e= nv-native/icecc-create-env > index 7e4dbc4..6d2e3c6 100755 > --- a/meta/recipes-devtools/icecc-create-env/icecc-create-env-native/icec= c-create-env > +++ b/meta/recipes-devtools/icecc-create-env/icecc-create-env-native/icec= c-create-env > @@ -29,33 +29,95 @@ add_file () > toadd=3D"$name=3D$path" > is_contained "$toadd" && return > if test -z "$silent"; then > - echo "adding file $toadd" > + echo "adding file $toadd" > fi > target_files=3D"$target_files $toadd" > if test -x "$path"; then > # Only call ldd when it makes sense > if file -L "$path" | grep 'ELF' > /dev/null 2>&1; then > - if ! file -L "$path" | grep 'static' > /dev/null 2>&1; then > - # ldd now outputs ld as /lib/ld-linux.so.xx on current nptl based gl= ibc > - # this regexp parse the outputs like: > - # ldd /usr/bin/gcc > - # linux-gate.so.1 =3D> (0xffffe000) > - # libc.so.6 =3D> /lib/tls/libc.so.6 (0xb7e81000) > - # /lib/ld-linux.so.2 (0xb7fe8000) > - # covering both situations ( with =3D> and without ) > - for lib in `ldd "$path" | sed -n 's,^[^/]*\(/[^ ]*\).*,\1,p'`;= do > - test -f "$lib" || continue > - # Check wether the same library also exists in the parent directory, > - # and prefer that on the assumption that it is a more generic one. > - local baselib=3D`echo "$lib" | sed 's,\(/[^/]*\)/.*\(/[^/]*\)$,\1\2= ,'` > - test -f "$baselib" && lib=3D$baselib > - add_file "$lib" > - done > + if ! file -L "$path" | grep 'static' > /dev/null 2>&1; then > + # ldd now outputs ld as /lib/ld-linux.so.xx on current nptl b= ased glibc > + # this regexp parse the outputs like: > + # ldd /usr/bin/gcc > + # linux-gate.so.1 =3D> (0xffffe000) > + # libc.so.6 =3D> /lib/tls/libc.so.6 (0xb7e81000) > + # /lib/ld-linux.so.2 (0xb7fe8000) > + # covering both situations ( with =3D> and without ) > + for lib in `ldd "$path" | sed -n 's,^[^/]*\(/[^ ]*\).*,\1,p'`= ; do > + test -f "$lib" || continue > + # Check wether the same library also exists in the parent d= irectory, > + # and prefer that on the assumption that it is a more gener= ic one. > + local baselib=3D`echo "$lib" | sed 's,\(/[^/]*\)/.*\(/[^/]*= \)$,\1\2,'` > + test -f "$baselib" && lib=3D$baselib > + add_file "$lib" > + done > fi > fi > fi > } > =20 > +# returns abs path to filedir > +abs_path() > +{ > + local path=3D$1 > + if test -f "$path"; then > + pushd $(dirname $path) > /dev/null 2>&1 > + dir_path=3D`pwd -P` > + path=3D$dir_path/$(basename $path) > + popd > /dev/null 2>&1 > + elif test -d "$path"; then > + pushd $path > /dev/null 2>&1 > + path=3D`pwd -P` > + popd > /dev/null 2>&1 > + fi > + echo $path > +} > + > +# Search and add file to the tarball file. > +search_addfile() > +{ > + local compiler=3D$1 > + local file_name=3D$2 > + local file_installdir=3D$3 > + local file=3D"" > + > + file=3D$($compiler -print-prog-name=3D$file_name) > + > + if test -z "$file" || test "$file" =3D "$file_name" || ! test -e "$f= ile"; then > + file=3D`$compiler -print-file-name=3D$file_name` > + fi > + > + if ! test -e "$file"; then > + return 1 > + fi > + > + if test -z "$file_installdir"; then > + # The file is going to be added to the tarball > + # in the same path where the compiler found it. > + > + file_installdir=3D$(dirname $file) > + abs_installdir=3D$(abs_path $file_installdir) > + > + if test "$file_installdir" !=3D "$abs_installdir"; then > + # The path where the compiler found the file is relative! > + # If the path where the compiler found the file is relative > + # to compiler's path, we must change it to be relative to > + # /usr/bin path where the compiler is going to be installed > + # in the tarball file. > + # Replacing relative path by abs path because the tar command > + # used to create the tarball file doesn't work well with > + # relative path as installdir. > + > + compiler_basedir=3D$(abs_path ${compiler%/*/*}) > + file_installdir=3D${abs_installdir/$compiler_basedir/"/usr"} > + fi > + fi > + > + add_file "$file" "$file_installdir/$file_name" > + > + return 0 > +} > + > # backward compat > if test "$1" =3D "--respect-path"; then > shift > @@ -67,62 +129,148 @@ if test "$1" =3D "--silent"; then > shift > fi > =20 > - > -added_gcc=3D$1 > -shift > -added_gxx=3D$1 > -shift > -added_as=3D$1 > -shift > -archive_name=3D$1 > - > -if test -z "$added_gcc" || test -z "$added_gxx" ; then > - echo "usage: $0 " > - exit 1 > +if test "$1" !=3D "--gcc" -a "$1" !=3D "--clang"; then > + # backward compat > + added_gcc=3D$1 > + shift > + added_gxx=3D$1 > + shift > + added_as=3D$1 > + shift > + archive_name=3D$1 > + shift > + gcc=3D1 > +else > + if test "$1" =3D "--gcc"; then > + shift > + added_gcc=3D$1 > + shift > + added_gxx=3D$1 > + shift > + added_as=3D$1 > + shift > + archive_name=3D$1 > + shift > + gcc=3D1 > + elif test "$1" =3D "--clang"; then > + shift > + added_clang=3D$1 > + shift > + added_compilerwrapper=3D$1 > + shift > + clang=3D1 > + else > + usage > + exit 1 > + fi > fi > =20 > -if ! test -x "$added_gcc" ; then > - echo "'$added_gcc' is no executable." > - exit 1 > +if test -n "$gcc"; then > + if test -z "$added_gcc" || test -z "$added_gxx"; then > + usage > + exit 1 > + fi > + if ! test -x "$added_gcc" ; then > + echo "'$added_gcc' is no executable." > + exit 1 > + fi > + if ! test -x "$added_gxx" ; then > + echo "'$added_gxx' is no executable." > + exit 1 > + fi > + if test -z "$added_as" ; then > + add_file /usr/bin/as /usr/bin/as > + else > + if ! test -x "$added_as" ; then > + echo "'$added_as' is no executable." > + exit 1 > + fi > + add_file $added_as /usr/bin/as > + fi > fi > =20 > -if ! test -x "$added_gxx" ; then > - echo "'$added_gcc' is no executable." > - exit 1 > +if test -n "$clang"; then > + if ! test -x "$added_clang" ; then > + echo "'$added_clang' is no executable." > + exit 1 > + fi > + if ! test -x "$added_compilerwrapper" ; then > + echo "'$added_compilerwrapper' is no executable." > + exit 1 > + fi > fi > =20 > +extrafiles=3D > +while test "x$1" =3D "x--addfile"; do > + shift > + extrafiles=3D"$extrafiles $1" > + shift > +done > =20 > +tempdir=3D`mktemp -d /tmp/iceccenvXXXXXX` > =20 > -add_file $added_gcc /usr/bin/gcc > -add_file $added_gxx /usr/bin/g++ > +# for testing the environment is usable at all > +add_file /bin/true > =20 > -if test -z "$added_as" ; then > - add_file /usr/bin/as /usr/bin/as > -else > - if ! test -x "$added_as" ; then > - echo "'$added_as' is no executable." > - exit 1 > - fi > +if test -n "$gcc"; then > + # getting compilers abs path > + added_gcc=3D$(abs_path $added_gcc) > + added_gxx=3D$(abs_path $added_gxx) > =20 > - add_file $added_as /usr/bin/as > -fi > + if test -z "$clang"; then > + add_file $added_gcc /usr/bin/gcc > + add_file $added_gxx /usr/bin/g++ > + else > + # HACK: The clang case below will add a wrapper in place of gcc,= so add the real > + # gcc under a different name that the wrapper will call. > + add_file $added_gcc /usr/bin/gcc.bin > + add_file $added_gxx /usr/bin/g++.bin > + fi > + add_file `$added_gcc -print-prog-name=3Dcc1` /usr/bin/cc1 > + add_file `$added_gxx -print-prog-name=3Dcc1plus` /usr/bin/cc1plus > + > + gcc_as=3D$($added_gcc -print-prog-name=3Das) > + if test "$gcc_as" =3D "as"; then > + add_file /usr/bin/as > + else > + add_file "$gcc_as" /usr/bin/as > + fi > =20 > -add_file `$added_gcc -print-prog-name=3Dcc1` /usr/bin/cc1 > -add_file `$added_gxx -print-prog-name=3Dcc1plus` /usr/bin/cc1plus > -specfile=3D`$added_gcc -print-file-name=3Dspecs` > -if test -n "$specfile" && test -e "$specfile"; then > - add_file "$specfile" > + search_addfile $added_gcc specs > + search_addfile $added_gcc liblto_plugin.so > fi > =20 > -ltofile=3D`$added_gcc -print-prog-name=3Dlto1` > -pluginfile=3D"${ltofile%lto1}liblto_plugin.so" > -if test -r "$pluginfile" > -then > - add_file $pluginfile ${pluginfile#*usr} > - add_file $pluginfile /usr${pluginfile#*usr} > +if test -n "$clang"; then > + add_file $added_clang /usr/bin/clang > + # HACK: Older icecream remotes have /usr/bin/{gcc|g++} hardcoded and= wouldn't > + # call /usr/bin/clang at all. So include a wrapper binary that will = call gcc or clang > + # depending on an extra argument added by icecream. > + add_file $added_compilerwrapper /usr/bin/gcc > + add_file $added_compilerwrapper /usr/bin/g++ > + > + add_file $($added_clang -print-prog-name=3Das) /usr/bin/as > + > + # clang always uses its internal .h files > + clangincludes=3D$(dirname $($added_clang -print-file-name=3Dinclude/= limits.h)) > + clangprefix=3D$(dirname $(dirname $added_clang)) > + for file in $(find $clangincludes -type f); do > + # get path without .. > + destfile=3D$(readlink -e $file) > + # and convert from to /usr if needed > + destfile=3D$(echo $destfile | sed "s#$clangprefix#/usr#" ) > + add_file "$file" "$destfile" > + done > fi > =20 > -tempdir=3D`mktemp -d /tmp/iceccenvXXXXXX` > +for extrafile in $extrafiles; do > + add_file $extrafile > +done > + > +# special case for weird multilib setups > +for dir in /lib /lib64 /usr/lib /usr/lib64; do > + test -L $dir && cp -p $dir $tempdir$dir > +done > + > new_target_files=3D > for i in $target_files; do > case $i in > @@ -151,7 +299,7 @@ target_files=3D`for i in $new_target_files; do echo $= i; done | sort` > #if not use the md5 of all files as the archive name > if test -z "$archive_name"; then > md5sum=3DNONE > - for file in /usr/bin/md5sum /bin/md5 /usr/bin/md5; do > + for file in /usr/bin/md5sum /bin/md5 /usr/bin/md5 /sbin/md5; do > if test -x $file; then > md5sum=3D$file > break > diff --git a/meta/recipes-devtools/icecc-create-env/icecc-create-env-nati= ve_0.1.bb b/meta/recipes-devtools/icecc-create-env/icecc-create-env-native_= 0.1.bb > deleted file mode 100644 > index b04474e..0000000 > --- a/meta/recipes-devtools/icecc-create-env/icecc-create-env-native_0.1.= bb > +++ /dev/null > @@ -1,26 +0,0 @@ > -DESCRIPTION =3D "This is a modified version of the icecc-create-env scri= pt in order to\ > -make it work with OE." > -SECTION =3D "base" > -# source file has just a "GPL" word, but upstream is GPLv2+. > -# most probably just GPL would be a mistake > -LICENSE =3D "GPLv2+" > -LIC_FILES_CHKSUM =3D "file://icecc-create-env;beginline=3D2;endline=3D5;= md5=3Dae1df3d6a058bfda40b66094c5f6065f" > - > -PR =3D "r2" > - > -DEPENDS =3D "" > -INHIBIT_DEFAULT_DEPS =3D "1" > - > -FILESPATH =3D "${FILE_DIRNAME}/${PN}/" > - > -inherit native > - > -PATCHTOOL =3D "patch" > -SRC_URI =3D "file://icecc-create-env" > - > -S =3D "${WORKDIR}" > - > -do_install() { > - install -d ${D}/${bindir} > - install -m 0755 ${WORKDIR}/icecc-create-env ${D}/${bindir} > -} > diff --git a/meta/recipes-devtools/icecc-create-env/icecc-create-env-nati= ve_1.0.1.bb b/meta/recipes-devtools/icecc-create-env/icecc-create-env-nativ= e_1.0.1.bb > new file mode 100644 > index 0000000..9f5c9a2 > --- /dev/null > +++ b/meta/recipes-devtools/icecc-create-env/icecc-create-env-native_1.0.= 1.bb > @@ -0,0 +1,26 @@ > +DESCRIPTION =3D "This is a modified version of the icecc-create-env scri= pt in order to\ > +make it work with OE." > +SECTION =3D "base" > +# source file has just a "GPL" word, but upstream is GPLv2+. > +# most probably just GPL would be a mistake > +LICENSE =3D "GPLv2+" > +LIC_FILES_CHKSUM =3D "file://icecc-create-env;beginline=3D2;endline=3D5;= md5=3Dae1df3d6a058bfda40b66094c5f6065f" > + > +PR =3D "r0" drop > + > +DEPENDS =3D "" > +INHIBIT_DEFAULT_DEPS =3D "1" > + > +FILESPATH =3D "${FILE_DIRNAME}/${PN}/" > + > +inherit native > + > +PATCHTOOL =3D "patch" > +SRC_URI =3D "file://icecc-create-env" > + > +S =3D "${WORKDIR}" > + > +do_install() { > + install -d ${D}/${bindir} > + install -m 0755 ${WORKDIR}/icecc-create-env ${D}/${bindir} > +} > --=20 > 1.7.10.4 >=20 > _______________________________________________ > Openembedded-core mailing list > Openembedded-core@lists.openembedded.org > http://lists.openembedded.org/mailman/listinfo/openembedded-core --=20 Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com --llIrKcgUOe3dCx0c Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.22 (GNU/Linux) iEYEARECAAYFAlKgaFMACgkQN1Ujt2V2gBwbmQCcCv3QXFELvQJ4lSEwsRsUis1y TXAAmgMqAuM2odhHHryTe4s/679MQXGa =xaZI -----END PGP SIGNATURE----- --llIrKcgUOe3dCx0c--