All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v4] mypaint: new recipe for version 0.9.1
@ 2011-05-02 17:40 Christian Charreyre
  2011-05-04  8:56 ` Paul Menzel
  0 siblings, 1 reply; 12+ messages in thread
From: Christian Charreyre @ 2011-05-02 17:40 UTC (permalink / raw)
  To: openembedded-devel

Signed-off-by: Christian Charreyre <christian.charreyre@cioinfoindus.fr>
---
 recipes/mypaint/files/scons-adapt.patch |   20 +++++++++++++++++
 recipes/mypaint/mypaint_0.9.1.bb        |   36 +++++++++++++++++++++++++++++++
 2 files changed, 56 insertions(+), 0 deletions(-)
 create mode 100644 recipes/mypaint/files/scons-adapt.patch
 create mode 100644 recipes/mypaint/mypaint_0.9.1.bb

diff --git a/recipes/mypaint/files/scons-adapt.patch b/recipes/mypaint/files/scons-adapt.patch
new file mode 100644
index 0000000..dcad6db
--- /dev/null
+++ b/recipes/mypaint/files/scons-adapt.patch
@@ -0,0 +1,50 @@
+Author: Christian Charreyre <christian.charreyre@cioinfoindus.fr>
+
+Adapt scons to OE
+Index: mypaint-0.9.1/lib/SConscript
+===================================================================
+--- mypaint-0.9.1.orig/lib/SConscript	2011-04-21 11:31:29.000000000 +0200
++++ mypaint-0.9.1/lib/SConscript	2011-04-21 11:35:30.000000000 +0200
+@@ -1,5 +1,11 @@
+ Import('env')
+-import sys
++import sys, os
++env.Replace(SHCXX = os.environ['CXX'])
++env.Replace(SHLINK = os.environ['CXX'])
++env.Replace(LDMODULEFLAGS = os.environ['TARGET_LDFLAGS'])
++env.Append(LDMODULEFLAGS = ['-Wl,--hash-style=gnu'])
++env.Append(LDMODULEFLAGS = ['-shared'])
++env.Replace(PATH = os.environ['PATH'])
+ 
+ # For the record: I know that scons supports swig. But it doesn't scan for #include in the generated code.
+ # 
+Index: mypaint-0.9.1/SConstruct
+===================================================================
+--- mypaint-0.9.1.orig/SConstruct	2011-03-04 15:44:59.000000000 +0100
++++ mypaint-0.9.1/SConstruct	2011-05-02 17:21:17.000000000 +0200
+@@ -11,13 +11,6 @@
+ if sys.platform == "win32":
+     python = 'python' # usually no versioned binaries on Windows
+ 
+-try: 
+-    import numpy
+-except ImportError:
+-    print 'You need to have numpy installed.'
+-    print
+-    raise
+-
+ SConsignFile() # no .scsonsign into $PREFIX please
+ 
+ if sys.platform == "darwin":
+@@ -40,8 +33,9 @@
+ 
+ env.Append(CXXFLAGS=' -Wall -Wno-sign-compare -Wno-write-strings')
+ 
+-# Get the numpy include path (for numpy/arrayobject.h).
+-numpy_path = numpy.get_include()
++# Defines the numpy include path (for numpy/arrayobject.h).
++# This is specefic to OE, to avoid to build python-numpy-native
++numpy_path = os.environ['STAGING_DIR_TARGET']+os.environ['PYTHON_SITEPACKAGES_DIR']+'/numpy/core/include'
+ env.Append(CPPPATH=numpy_path)
+ 
+ 
diff --git a/recipes/mypaint/mypaint_0.9.1.bb b/recipes/mypaint/mypaint_0.9.1.bb
new file mode 100644
index 0000000..a7e5077
--- /dev/null
+++ b/recipes/mypaint/mypaint_0.9.1.bb
@@ -0,0 +1,37 @@
+DESCRIPTION="fast and easy open-source graphics application for digital painters"
+LICENSE="GPL"
+SECTION = "x11/applications"
+PRIORITY = "optional"
+HOMEPAGE = "http://mypaint.intilinux.com/"
+
+DEPENDS = "glib-2.0 libpng python-numpy"
+RDEPENDS_${PN} = "python-image python-json"
+
+SRC_URI = "http://download.gna.org/mypaint/${PN}-${PV}.tar.bz2 \
+	file://scons-adapt.patch \
+"
+
+SRC_URI[md5sum] = "6249a16359a438d6dc658f5765b35515"
+SRC_URI[sha256sum] = "407b599f62fb0d6e711fee57d22e64d3aec88825364fb5f7f73b9f0940aa7aed"
+
+inherit distutils scons
+
+do_compile() {
+       STAGING_INCDIR=${STAGING_INCDIR} \
+       STAGING_LIBDIR=${STAGING_LIBDIR} \
+       STAGING_DIR_TARGET=${STAGING_DIR_TARGET} \
+       PYTHON_SITEPACKAGES_DIR=${PYTHON_SITEPACKAGES_DIR} \
+       BUILD_SYS=${BUILD_SYS} \
+       HOST_SYS=${HOST_SYS} \
+       PATH=${PATH} scons_do_compile
+}
+
+do_install() {
+       STAGING_DIR_TARGET=${STAGING_DIR_TARGET} \
+       PYTHON_SITEPACKAGES_DIR=${PYTHON_SITEPACKAGES_DIR} \
+       BUILD_SYS=${BUILD_SYS} \
+       HOST_SYS=${HOST_SYS} \
+       scons_do_install
+}
+
+FILES_${PN} += "${datadir}"
-- 
1.7.0.4




^ permalink raw reply related	[flat|nested] 12+ messages in thread
* Re: [PATCH v4] mypaint: new recipe for version 0.9.1
@ 2011-05-05 15:44 Christian Charreyre
  2011-05-05 18:58 ` Paul Menzel
  0 siblings, 1 reply; 12+ messages in thread
From: Christian Charreyre @ 2011-05-05 15:44 UTC (permalink / raw)
  To: openembedded-devel

[-- Attachment #1: Type: text/plain, Size: 859 bytes --]

Paul,

I've an additional question regarding build configuration :


Le 04/05/2011 17:34, Eric Bénard a écrit :
>  Hi Paul,
>
>  On 04/05/2011 10:56, Paul Menzel wrote:
>>  Please remember to add what build configuration you used. (You can just
>>  paste it from the BitBake output.)
>>
>  You asked this several times on the list but this part is not in the
>  commit log policy :
>  http://wiki.openembedded.org/index.php/Commit_log_example
>

[...]

how must build configuration described : which information to add,
where, and how do I get it from bitbake output ?

Can you give me an example ?

For your information, my machine is derived from beagleboard (very few
differences), and my distribution is derived from angstrom, because I'm
building a complete distribution, not using a standard one.

Thanks.

Christian.



[-- Attachment #2: christian_charreyre.vcf --]
[-- Type: text/x-vcard, Size: 278 bytes --]

begin:vcard
fn:Christian  Charreyre
n:Charreyre;Christian 
org:CIO Informatique Industrielle
adr;dom:;;11 Avenue Marigny;Marseille;;13014
email;internet:christian.charreyre@cioinfoindus.fr
tel;work:04 95 05 19 41
url:http://www.cioinfoindus.fr
version:2.1
end:vcard


^ permalink raw reply	[flat|nested] 12+ messages in thread
* Re: [PATCH v4] mypaint: new recipe for version 0.9.1
@ 2011-05-05 15:45 Christian Charreyre
  2011-05-05 18:55 ` Paul Menzel
  0 siblings, 1 reply; 12+ messages in thread
From: Christian Charreyre @ 2011-05-05 15:45 UTC (permalink / raw)
  To: openembedded-devel

[-- Attachment #1: Type: text/plain, Size: 5808 bytes --]


Le 04/05/2011 10:56, Paul Menzel a écrit :
>  Dear Christian,
>
>
>  we are almost there.
>
>
>  Am Montag, den 02.05.2011, 19:40 +0200 schrieb Christian Charreyre:
>
>  Please remember to add what build configuration you used. (You can just
>  paste it from the BitBake output.)
>
>>  Signed-off-by: Christian Charreyre<christian.charreyre@cioinfoindus.fr>
>>  ---
>
>  If you send patch iterations (v2, v3, v4, …) it is common to put below
>  »---« what changed in the iterations to make it easier for reviewers to
>  know what changed. Put »---« behind that, since it is not needed for the
>  actual commit message.
>
>  Example:
>
>  Commit message.
>
>  S-o-b: Joe User<joe@example.org>
>  ---
>  v2: add foo to DEPENDS
>  v3: reorder to adhere to styleguide
>  v4: change install to …
>  ---
>  diff
>
Noted.

I've tried to be explicite with a first message describing the
evolution, then the patch itself (I used git send-email --compose),
I'll had this in later posts.

>>    recipes/mypaint/files/scons-adapt.patch |   20 +++++++++++++++++
>>    recipes/mypaint/mypaint_0.9.1.bb        |   36 +++++++++++++++++++++++++++++++
>>    2 files changed, 56 insertions(+), 0 deletions(-)
>>    create mode 100644 recipes/mypaint/files/scons-adapt.patch
>>    create mode 100644 recipes/mypaint/mypaint_0.9.1.bb
>>
>>  diff --git a/recipes/mypaint/files/scons-adapt.patch b/recipes/mypaint/files/scons-adapt.patch
>>  new file mode 100644
>>  index 0000000..dcad6db
>>  --- /dev/null
>>  +++ b/recipes/mypaint/files/scons-adapt.patch
>>  @@ -0,0 +1,50 @@
>>  +Author: Christian Charreyre<christian.charreyre@cioinfoindus.fr>
>>  +
>>  +Adapt scons to OE
>>  +Index: mypaint-0.9.1/lib/SConscript
>>  +===================================================================
>>  +--- mypaint-0.9.1.orig/lib/SConscript	2011-04-21 11:31:29.000000000 +0200
>>  ++++ mypaint-0.9.1/lib/SConscript	2011-04-21 11:35:30.000000000 +0200
>>  +@@ -1,5 +1,11 @@
>>  + Import('env')
>>  +-import sys
>>  ++import sys, os
>>  ++env.Replace(SHCXX = os.environ['CXX'])
>>  ++env.Replace(SHLINK = os.environ['CXX'])
>>  ++env.Replace(LDMODULEFLAGS = os.environ['TARGET_LDFLAGS'])
>>  ++env.Append(LDMODULEFLAGS = ['-Wl,--hash-style=gnu'])
>>  ++env.Append(LDMODULEFLAGS = ['-shared'])
>>  ++env.Replace(PATH = os.environ['PATH'])
>>  +
>>  + # For the record: I know that scons supports swig. But it doesn't scan for #include in the generated code.
>>  + #
>>  +Index: mypaint-0.9.1/SConstruct
>>  +===================================================================
>>  +--- mypaint-0.9.1.orig/SConstruct	2011-03-04 15:44:59.000000000 +0100
>>  ++++ mypaint-0.9.1/SConstruct	2011-05-02 17:21:17.000000000 +0200
>>  +@@ -11,13 +11,6 @@
>>  + if sys.platform == "win32":
>>  +     python = 'python' # usually no versioned binaries on Windows
>>  +
>>  +-try:
>>  +-    import numpy
>>  +-except ImportError:
>>  +-    print 'You need to have numpy installed.'
>>  +-    print
>>  +-    raise
>>  +-
>>  + SConsignFile() # no .scsonsign into $PREFIX please
>>  +
>>  + if sys.platform == "darwin":
>>  +@@ -40,8 +33,9 @@
>>  +
>>  + env.Append(CXXFLAGS=' -Wall -Wno-sign-compare -Wno-write-strings')
>>  +
>>  +-# Get the numpy include path (for numpy/arrayobject.h).
>>  +-numpy_path = numpy.get_include()
>>  ++# Defines the numpy include path (for numpy/arrayobject.h).
>>  ++# This is specefic to OE, to avoid to build python-numpy-native
>
>  spec*i*fic
>
>>  ++numpy_path = os.environ['STAGING_DIR_TARGET']+os.environ['PYTHON_SITEPACKAGES_DIR']+'/numpy/core/include'
>>  + env.Append(CPPPATH=numpy_path)
>>  +
>>  +
>>  diff --git a/recipes/mypaint/mypaint_0.9.1.bb b/recipes/mypaint/mypaint_0.9.1.bb
>>  new file mode 100644
>>  index 0000000..a7e5077
>>  --- /dev/null
>>  +++ b/recipes/mypaint/mypaint_0.9.1.bb
>>  @@ -0,0 +1,37 @@
>>  +DESCRIPTION="fast and easy open-source graphics application for digital painters"
>>  +LICENSE="GPL"
>>  +SECTION = "x11/applications"
>>  +PRIORITY = "optional"
>>  +HOMEPAGE = "http://mypaint.intilinux.com/"
>>  +
>>  +DEPENDS = "glib-2.0 libpng python-numpy"
>>  +RDEPENDS_${PN} = "python-image python-json"
>
>  I am now getting the following build error
>
>           + /oe/build-angstrom-next/angstrom-dev/sysroots/i686-linux/usr/bin/scons -j4 'CXX=arm-angstrom-linux-gnueabi-g++ -march=armv7-a -mtune=cortex-a8 -mfpu=neon -mfloat-abi=softfp -mthumb-interwork -mno-thumb --sysroot=/oe/build-angstrom-next/angstrom-dev/sysroots/armv7a-angstrom-linux-gnueabi' PREFIX=/usr prefix=/usr
>           scons: Reading SConscript files ...
>           Building for python2.6
>           swig -o mypaintlib_wrap.cpp -noproxydel -python -c++ mypaintlib.i
>           sh: swig: command not found
>           scons: *** Error 127
>           + oefatal 'scons build execution failed.'
>           + echo FATAL: 'scons build execution failed.'
>           FATAL: scons build execution failed.
>           + exit 1
>
>  which I fixed by building `swig-native` beforehand. Please add
>  `swig-native` to `DEPENDS`. You probably did not encounter this error
>  since you have it installed locally.
>
>  […]
>
>
>  Thank you very much for your patience and improved patches,
>
>  Paul
>
>
I think your right, I have tried to update correctly DEPENDS, but as
swig-native was already there for me I've missed it.

Thanks for your patience to review it and teach me good practices....

Do I submit a v5 with DEPENDS updated ?
>
>  _______________________________________________
>  Openembedded-devel mailing list
>  Openembedded-devel@lists.openembedded.org
>  http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel



[-- Attachment #2: christian_charreyre.vcf --]
[-- Type: text/x-vcard, Size: 278 bytes --]

begin:vcard
fn:Christian  Charreyre
n:Charreyre;Christian 
org:CIO Informatique Industrielle
adr;dom:;;11 Avenue Marigny;Marseille;;13014
email;internet:christian.charreyre@cioinfoindus.fr
tel;work:04 95 05 19 41
url:http://www.cioinfoindus.fr
version:2.1
end:vcard


^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2011-05-06 21:49 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-05-02 17:40 [PATCH v4] mypaint: new recipe for version 0.9.1 Christian Charreyre
2011-05-04  8:56 ` Paul Menzel
2011-05-04 15:34   ` Eric Bénard
2011-05-04 16:48     ` Paul Menzel
2011-05-04 16:57       ` Eric Bénard
2011-05-04 18:51         ` Paul Menzel
  -- strict thread matches above, loose matches on Subject: below --
2011-05-05 15:44 Christian Charreyre
2011-05-05 18:58 ` Paul Menzel
2011-05-06 21:18   ` Christian Charreyre
2011-05-06 21:46     ` Paul Menzel
2011-05-05 15:45 Christian Charreyre
2011-05-05 18:55 ` Paul Menzel

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.