Re: [OE-core] [PATCH 01/17] icecc-create-env: rename to -native

2011-09-22 Thread Richard Purdie
On Wed, 2011-09-21 at 22:39 +0400, Dmitry Eremin-Solenikov wrote:
 icecc-create-env(-native) is a native package. But if it doesn't have
 the -native suffix, some dependency tracking stuff in oe-core (in
 nativesdk.bbclass) get's crazy about it, trying to generate dependencies
 to icecc-create-env-native packages. To fix that simply add -native
 suffix to this package.
 
 Signed-off-by: Dmitry Eremin-Solenikov dbarysh...@gmail.com
 ---

Thanks for some great cleanups. I've merged most of this to master,
except where there has been feedback/discussion.

Cheers,

Richard


___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 01/17] icecc-create-env: rename to -native

2011-09-22 Thread Dmitry Eremin-Solenikov
On 9/22/11, Richard Purdie richard.pur...@linuxfoundation.org wrote:
 On Wed, 2011-09-21 at 22:39 +0400, Dmitry Eremin-Solenikov wrote:
 icecc-create-env(-native) is a native package. But if it doesn't have
 the -native suffix, some dependency tracking stuff in oe-core (in
 nativesdk.bbclass) get's crazy about it, trying to generate dependencies
 to icecc-create-env-native packages. To fix that simply add -native
 suffix to this package.

 Signed-off-by: Dmitry Eremin-Solenikov dbarysh...@gmail.com
 ---

 Thanks for some great cleanups. I've merged most of this to master,
 except where there has been feedback/discussion.

Thanks! It seems icecc patch file got damaged somewhere in the loop.
I think I'll just drop the patch and the not-so-upstream tarball and
submit the script
to be included in the oe-core tree. Does that sound good?

-- 
With best wishes
Dmitry

___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


[OE-core] [PATCH 01/17] icecc-create-env: rename to -native

2011-09-21 Thread Dmitry Eremin-Solenikov
icecc-create-env(-native) is a native package. But if it doesn't have
the -native suffix, some dependency tracking stuff in oe-core (in
nativesdk.bbclass) get's crazy about it, trying to generate dependencies
to icecc-create-env-native packages. To fix that simply add -native
suffix to this package.

Signed-off-by: Dmitry Eremin-Solenikov dbarysh...@gmail.com
---
 meta/classes/icecc.bbclass |2 +-
 .../icecc-create-env-native/icecc-lto-update.patch |  103 
 .../icecc-create-env-native_0.1.bb |   29 ++
 .../icecc-create-env/icecc-lto-update.patch|  103 
 .../icecc-create-env/icecc-create-env_0.1.bb   |   29 --
 5 files changed, 133 insertions(+), 133 deletions(-)
 create mode 100644 
meta/recipes-devtools/icecc-create-env/icecc-create-env-native/icecc-lto-update.patch
 create mode 100644 
meta/recipes-devtools/icecc-create-env/icecc-create-env-native_0.1.bb
 delete mode 100644 
meta/recipes-devtools/icecc-create-env/icecc-create-env/icecc-lto-update.patch
 delete mode 100644 
meta/recipes-devtools/icecc-create-env/icecc-create-env_0.1.bb

diff --git a/meta/classes/icecc.bbclass b/meta/classes/icecc.bbclass
index e20666b..710d798 100644
--- a/meta/classes/icecc.bbclass
+++ b/meta/classes/icecc.bbclass
@@ -33,7 +33,7 @@ def icecc_dep_prepend(d):
 # we need that built is the responsibility of the patch function / class, 
not
 # the application.
 if not bb.data.getVar('INHIBIT_DEFAULT_DEPS', d):
-return icecc-create-env
+return icecc-create-env-native
 return 
 
 DEPENDS_prepend += ${@icecc_dep_prepend(d)} 
diff --git 
a/meta/recipes-devtools/icecc-create-env/icecc-create-env-native/icecc-lto-update.patch
 
b/meta/recipes-devtools/icecc-create-env/icecc-create-env-native/icecc-lto-update.patch
new file mode 100644
index 000..a7af2e3
--- /dev/null
+++ 
b/meta/recipes-devtools/icecc-create-env/icecc-create-env-native/icecc-lto-update.patch
@@ -0,0 +1,103 @@
+--- a/icecc-create-env 2006-12-14 09:50:46.0 +0300
 b/icecc-create-env 2011-08-31 17:52:45.0 +0400
+@@ -27,9 +27,6 @@
+   # readlink is not portable enough.
+   path=`ls -H $path`
+   toadd=$name=$path
+-  if test $name = $path; then
+-toadd=$path
+-  fi
+   is_contained $toadd  return
+   if test -z $silent; then
+   echo adding file $toadd
+@@ -117,6 +114,14 @@
+   add_file $specfile
+ fi
+
++ltofile=`$added_gcc -print-prog-name=lto1`
++pluginfile=${ltofile%lto1}liblto_plugin.so
++if test -r $pluginfile
++then
++  add_file $pluginfile  ${pluginfile#*usr}
++  add_file $pluginfile  /usr${pluginfile#*usr}
++fi
++
+ tempdir=`mktemp -d /tmp/iceccenvXX`
+ new_target_files=
+ for i in $target_files; do
+@@ -140,49 +147,44 @@
+ done
+
+ #sort the files
+- target_files=`for i in $new_target_files; do echo $i; done | sort`
++target_files=`for i in $new_target_files; do echo $i; done | sort`
+
+ #test if an archive name was supplied
+ #if not use the md5 of all files as the archive name
+ if test -z $archive_name; then
+-md5sum=NONE
+-for file in /usr/bin/md5sum /bin/md5 /usr/bin/md5; do
+-   if test -x $file; then
+-  md5sum=$file
+-break
+-   fi
+-done
++  md5sum=NONE
++  for file in /usr/bin/md5sum /bin/md5 /usr/bin/md5; do
++if test -x $file; then
++  md5sum=$file
++  break
++fi
++  done
+
+-#calculate md5 and use it as the archive name
+-archive_name=`for i in $target_files; do $md5sum $tempdir/$i; done | sed -e 
's/ .*$//' | $md5sum | sed -e 's/ .*$//'` || {
+-  if test -z $silent; then
+-   echo Couldn't compute MD5 sum.
++  #calculate md5 and use it as the archive name
++  archive_name=`for i in $target_files; do test -f $tempdir/$i  $md5sum 
$tempdir/$i; done | sed -e 's/ .*$//' | $md5sum | sed -e 's/ .*$//'`.tar.gz || {
++if test -z $silent; then
++ echo Couldn't compute MD5 sum.
++fi
++exit 2
++  }
++  mydir=`pwd`
++else
++  mydir=`dirname $archive_name`
++
++  #check if we have a full path or only a filename
++  if test $mydir = . ; then
++mydir=`pwd`
++  else
++mydir=
+   fi
+-  exit 2
+-}
+-
+ fi
+
+ if test -z $silent; then
+-echo creating $archive_name.tar.gz
++echo creating $archive_name
+ fi
+
+-if test -z $archive_name; then
+- mydir=`pwd`
+-else
+-# mydir=dirname ${archive_name}
+-  mydir=${archive_name%/*}
+-
+-#check if we have a full path or only a filename
+- if test -z $mydir; then
+-  mydir=`pwd`
+- else
+-  mydir=
+- fi
+-
+-fi
+ cd $tempdir
+-tar -czhf $mydir/$archive_name.tar.gz $target_files || {
++tar -czhf $mydir/$archive_name $target_files || {
+  if test -z $silent; then
+   echo Couldn't create archive
+  fi
diff --git 
a/meta/recipes-devtools/icecc-create-env/icecc-create-env-native_0.1.bb 
b/meta/recipes-devtools/icecc-create-env/icecc-create-env-native_0.1.bb
new file mode 100644
index 000..9a440ba
--- /dev/null
+++ b/meta/recipes-devtools/icecc-create-env/icecc-create-env-native_0.1.bb
@@ -0,0 +1,29 @@