Re: [oe] [meta-efl][meta-oe 05/12] id3lib: Import from openembedded classic

2011-11-01 Thread Paul Eggleton
On Monday 31 October 2011 23:12:01 Koen Kooi wrote:
 Different goals I guess. 

That's not much in the way of justification...

 And I don't trust a ruleset that's in a wiki, too
 much room for people coming in and messing it up.

If that's a concern, just apply some access restrictions to the page. Edit 
history is tracked if it's on a wiki at least.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre

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


Re: [oe] [meta-efl][meta-oe 05/12] id3lib: Import from openembedded classic

2011-11-01 Thread Denys Dmytriyenko
On Tue, Nov 01, 2011 at 11:40:49AM +, Paul Eggleton wrote:
 On Monday 31 October 2011 23:12:01 Koen Kooi wrote:
  Different goals I guess. 
 
 That's not much in the way of justification...
 
  And I don't trust a ruleset that's in a wiki, too
  much room for people coming in and messing it up.
 
 If that's a concern, just apply some access restrictions to the page. Edit 
 history is tracked if it's on a wiki at least.

So, where are these new rules then, if not in a wiki? Are they applicable to 
meta-oe only?

-- 
Denys

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


Re: [oe] [meta-efl][meta-oe 05/12] id3lib: Import from openembedded classic

2011-10-31 Thread Paul Eggleton
On Saturday 29 October 2011 13:33:27 Paul Menzel wrote:
  +LIC_FILES_CHKSUM =
  files://COPYING;md5=3bf50002aefd002f49e7bb854063f7e7 
  +PR = r0
 
 According to Koen these are going away.

Well, Koen might reject PR = r0 in meta-oe, that's fine; however 
LIC_FILES_CHKSUM is mandatory or the recipe will not build, and I don't think 
that will be changing any time soon. I have considered if it would be 
desirable/possible to eliminate it for recipes that don't install any files 
(e.g. tasks), so far I haven't done any work on that. For normal recipes it is 
required however.

Cheers,
Paul 


-- 

Paul Eggleton
Intel Open Source Technology Centre

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


Re: [oe] [meta-efl][meta-oe 05/12] id3lib: Import from openembedded classic

2011-10-31 Thread Koen Kooi
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Op 29-10-11 14:33, Paul Menzel schreef:
 Dear Martin, dear Denis,
 
 
 Am Samstag, den 29.10.2011, 12:29 +0200 schrieb Martin Jansa:
 From: Denis 'GNUtoo' Carikli gnu...@no-log.org
 
 Added LIC_FILES_CHKSUM, and fixed LICENSE
 
 Signed-off-by: Denis 'GNUtoo' Carikli gnu...@no-log.org 
 Signed-off-by: Martin Jansa martin.ja...@gmail.com
 
 NACK.
 
 please add the version you import and the commit ID you import from. This
 is all written in the guide lines [1]!

And as I've said before, meta-oe is not oe-dev, so you can't use take the
old 'rules' and apply them to meta-oe 1:1!

-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.5 (Darwin)
Comment: GPGTools - http://gpgtools.org

iD8DBQFOrn10MkyGM64RGpERApcaAJoDeddEn1V2FY8CUFUFdOr7b3aAPQCeL9pn
JSQ8IkWSJe+eqlwZnuX/5fg=
=Xgt+
-END PGP SIGNATURE-


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


Re: [oe] [meta-efl][meta-oe 05/12] id3lib: Import from openembedded classic

2011-10-31 Thread Koen Kooi
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Op 31-10-11 10:55, Paul Eggleton schreef:
 On Saturday 29 October 2011 13:33:27 Paul Menzel wrote:
 +LIC_FILES_CHKSUM = 
 files://COPYING;md5=3bf50002aefd002f49e7bb854063f7e7 +PR = r0
 
 According to Koen these are going away.
 
 Well, Koen might reject PR = r0 in meta-oe, that's fine; however 
 LIC_FILES_CHKSUM is mandatory or the recipe will not build, and I don't
 think that will be changing any time soon. I have considered if it would
 be desirable/possible to eliminate it for recipes that don't install any
 files (e.g. tasks), so far I haven't done any work on that. For normal
 recipes it is required however.

Exactly!
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.5 (Darwin)
Comment: GPGTools - http://gpgtools.org

iD8DBQFOrn2PMkyGM64RGpERAtZRAJ9+q7eDwUbX3o3vuIxOnwKhWSx0VQCgorOy
1FZ3HAsW95D1sXuh3uldPaU=
=OzFY
-END PGP SIGNATURE-


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


Re: [oe] [meta-efl][meta-oe 05/12] id3lib: Import from openembedded classic

2011-10-31 Thread Paul Menzel
Am Montag, den 31.10.2011, 09:55 + schrieb Paul Eggleton:
 On Saturday 29 October 2011 13:33:27 Paul Menzel wrote:
   +LIC_FILES_CHKSUM = 
   files://COPYING;md5=3bf50002aefd002f49e7bb854063f7e7 
   +PR = r0
  
  According to Koen these are going away.
 
 Well, Koen might reject PR = r0 in meta-oe, that's fine; however 
 LIC_FILES_CHKSUM is mandatory or the recipe will not build, and I don't think 
 that will be changing any time soon. I have considered if it would be 
 desirable/possible to eliminate it for recipes that don't install any files 
 (e.g. tasks), so far I haven't done any work on that. For normal recipes it 
 is 
 required however.

I meant »this is« instead of »these are«. (Or did I mean these
occurrences of `PR`.)I am sorry for the confusion.


Thanks,

Paul


signature.asc
Description: This is a digitally signed message part
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel


Re: [oe] [meta-efl][meta-oe 05/12] id3lib: Import from openembedded classic

2011-10-31 Thread Paul Menzel
Am Montag, den 31.10.2011, 11:50 +0100 schrieb Koen Kooi:
 Op 29-10-11 14:33, Paul Menzel schreef:
  Dear Martin, dear Denis,
  
  
  Am Samstag, den 29.10.2011, 12:29 +0200 schrieb Martin Jansa:
  From: Denis 'GNUtoo' Carikli gnu...@no-log.org
  
  Added LIC_FILES_CHKSUM, and fixed LICENSE
  
  Signed-off-by: Denis 'GNUtoo' Carikli gnu...@no-log.org 
  Signed-off-by: Martin Jansa martin.ja...@gmail.com
  
  NACK.
  
  please add the version you import and the commit ID you import from. This
  is all written in the guide lines [1]!
 
 And as I've said before, meta-oe is not oe-dev, so you can't use take the
 old 'rules' and apply them to meta-oe 1:1!

The guide lines I referred to [1] have been written for OE-core. ;-)


Thanks,

Paul


[1] http://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines


signature.asc
Description: This is a digitally signed message part
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel


Re: [oe] [meta-efl][meta-oe 05/12] id3lib: Import from openembedded classic

2011-10-31 Thread Dodji Seketeli
Paul Menzel paulepan...@users.sourceforge.net a écrit:

 The guide lines I referred to [1] have been written for OE-core. ;-)

Just to be sure, do they apply to OE classic as well?  They are neat.

 [1] http://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines

-- 
Dodji


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


Re: [oe] [meta-efl][meta-oe 05/12] id3lib: Import from openembedded classic

2011-10-31 Thread Koen Kooi
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Op 31-10-11 17:24, Paul Menzel schreef:
 Am Montag, den 31.10.2011, 11:50 +0100 schrieb Koen Kooi:
 Op 29-10-11 14:33, Paul Menzel schreef:
 Dear Martin, dear Denis,
 
 
 Am Samstag, den 29.10.2011, 12:29 +0200 schrieb Martin Jansa:
 From: Denis 'GNUtoo' Carikli gnu...@no-log.org
 
 Added LIC_FILES_CHKSUM, and fixed LICENSE
 
 Signed-off-by: Denis 'GNUtoo' Carikli gnu...@no-log.org 
 Signed-off-by: Martin Jansa martin.ja...@gmail.com
 
 NACK.
 
 please add the version you import and the commit ID you import from. 
 This is all written in the guide lines [1]!
 
 And as I've said before, meta-oe is not oe-dev, so you can't use take 
 the old 'rules' and apply them to meta-oe 1:1!
 
 The guide lines I referred to [1] have been written for OE-core. ;-)

Yes, and oe-core still isn't meta-oe


-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.5 (Darwin)
Comment: GPGTools - http://gpgtools.org

iD8DBQFOrt9oMkyGM64RGpERAjfuAJ0f8N80qZ8LjF5XbW7lJ6Gl296cRACePnNK
h4DGgQIV9y/kR+YzHeVOWIA=
=QPHc
-END PGP SIGNATURE-


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


Re: [oe] [meta-efl][meta-oe 05/12] id3lib: Import from openembedded classic

2011-10-31 Thread Philip Balister
On 10/31/2011 01:48 PM, Koen Kooi wrote:
 Op 31-10-11 17:24, Paul Menzel schreef:
 Am Montag, den 31.10.2011, 11:50 +0100 schrieb Koen Kooi:
 Op 29-10-11 14:33, Paul Menzel schreef:
 Dear Martin, dear Denis,


 Am Samstag, den 29.10.2011, 12:29 +0200 schrieb Martin Jansa:
 From: Denis 'GNUtoo' Carikli gnu...@no-log.org

 Added LIC_FILES_CHKSUM, and fixed LICENSE

 Signed-off-by: Denis 'GNUtoo' Carikli gnu...@no-log.org 
 Signed-off-by: Martin Jansa martin.ja...@gmail.com

 NACK.

 please add the version you import and the commit ID you import from. 
 This is all written in the guide lines [1]!

 And as I've said before, meta-oe is not oe-dev, so you can't use take 
 the old 'rules' and apply them to meta-oe 1:1!
 
 The guide lines I referred to [1] have been written for OE-core. ;-)
 
 Yes, and oe-core still isn't meta-oe

Why would we need different guidelines?

Philip

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


Re: [oe] [meta-efl][meta-oe 05/12] id3lib: Import from openembedded classic

2011-10-31 Thread Koen Kooi
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Op 31-10-11 22:18, Philip Balister schreef:
 On 10/31/2011 01:48 PM, Koen Kooi wrote:
 Op 31-10-11 17:24, Paul Menzel schreef:
 Am Montag, den 31.10.2011, 11:50 +0100 schrieb Koen Kooi:
 Op 29-10-11 14:33, Paul Menzel schreef:
 Dear Martin, dear Denis,
 
 
 Am Samstag, den 29.10.2011, 12:29 +0200 schrieb Martin Jansa:
 From: Denis 'GNUtoo' Carikli gnu...@no-log.org
 
 Added LIC_FILES_CHKSUM, and fixed LICENSE
 
 Signed-off-by: Denis 'GNUtoo' Carikli gnu...@no-log.org 
 Signed-off-by: Martin Jansa martin.ja...@gmail.com
 
 NACK.
 
 please add the version you import and the commit ID you import
 from. This is all written in the guide lines [1]!
 
 And as I've said before, meta-oe is not oe-dev, so you can't use
 take the old 'rules' and apply them to meta-oe 1:1!
 
 The guide lines I referred to [1] have been written for OE-core. ;-)
 
 Yes, and oe-core still isn't meta-oe
 
 Why would we need different guidelines?

Different goals I guess. And I don't trust a ruleset that's in a wiki, too
much room for people coming in and messing it up.
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.5 (Darwin)
Comment: GPGTools - http://gpgtools.org

iD8DBQFOrx0xMkyGM64RGpERAi5jAJsHB7JqgQJ/UECPPzXuWZSIxpneyQCgptup
mGmfCQmwJ9RodF2sNPQOpaw=
=7Rza
-END PGP SIGNATURE-


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


Re: [oe] [meta-efl][meta-oe 05/12] id3lib: Import from openembedded classic

2011-10-31 Thread Frans Meulenbroeks
2011/10/31 Koen Kooi k...@dominion.thruhere.net

 -BEGIN PGP SIGNED MESSAGE-
 Hash: SHA1

 Op 31-10-11 22:18, Philip Balister schreef:
  On 10/31/2011 01:48 PM, Koen Kooi wrote:
  Op 31-10-11 17:24, Paul Menzel schreef:
  Am Montag, den 31.10.2011, 11:50 +0100 schrieb Koen Kooi:
  Op 29-10-11 14:33, Paul Menzel schreef:
  Dear Martin, dear Denis,
 
 
  Am Samstag, den 29.10.2011, 12:29 +0200 schrieb Martin Jansa:
  From: Denis 'GNUtoo' Carikli gnu...@no-log.org
 
  Added LIC_FILES_CHKSUM, and fixed LICENSE
 
  Signed-off-by: Denis 'GNUtoo' Carikli gnu...@no-log.org
  Signed-off-by: Martin Jansa martin.ja...@gmail.com
 
  NACK.
 
  please add the version you import and the commit ID you import
  from. This is all written in the guide lines [1]!
 
  And as I've said before, meta-oe is not oe-dev, so you can't use
  take the old 'rules' and apply them to meta-oe 1:1!
 
  The guide lines I referred to [1] have been written for OE-core. ;-)
 
  Yes, and oe-core still isn't meta-oe
 
  Why would we need different guidelines?

 Different goals I guess. And I don't trust a ruleset that's in a wiki, too
 much room for people coming in and messing it up.

 As far as I recall the idea was that meta oe should follow the same
guidelines as oe core.
When was this changed?
Who decided this?
And if there need to be different guidelines, I feel it is useful to
document them somewhere (and probably also why it deviates from oe-core).

Anyway, I think I start to understand why so few people are willing to
contribute to meta-oe.

Have fun! Frans.
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel


[oe] [meta-efl][meta-oe 05/12] id3lib: Import from openembedded classic

2011-10-29 Thread Martin Jansa
From: Denis 'GNUtoo' Carikli gnu...@no-log.org

Added LIC_FILES_CHKSUM, and fixed LICENSE

Signed-off-by: Denis 'GNUtoo' Carikli gnu...@no-log.org
Signed-off-by: Martin Jansa martin.ja...@gmail.com
---
 meta-oe/recipes-multimedia/id3lib/id3lib_3.8.3.bb |   18 ++
 1 files changed, 18 insertions(+), 0 deletions(-)
 create mode 100644 meta-oe/recipes-multimedia/id3lib/id3lib_3.8.3.bb

diff --git a/meta-oe/recipes-multimedia/id3lib/id3lib_3.8.3.bb 
b/meta-oe/recipes-multimedia/id3lib/id3lib_3.8.3.bb
new file mode 100644
index 000..9be4ffb
--- /dev/null
+++ b/meta-oe/recipes-multimedia/id3lib/id3lib_3.8.3.bb
@@ -0,0 +1,18 @@
+DESCRIPTION = Library for interacting with ID3 tags.
+SECTION = libs/multimedia
+PRIORITY = optional
+DEPENDS = zlib
+LICENSE = GPLv2+
+LIC_FILES_CHKSUM = files://COPYING;md5=3bf50002aefd002f49e7bb854063f7e7
+PR = r0
+
+inherit autotools
+
+SRC_URI = ${SOURCEFORGE_MIRROR}/id3lib/id3lib-${PV}.tar.gz;name=archive \
+   
http://ftp.de.debian.org/debian/pool/main/i/id3lib3.8.3/id3lib3.8.3_3.8.3-7.2.diff.gz;name=patch
 \
+  
+
+SRC_URI[archive.md5sum] = 19f27ddd2dda4b2d26a559a4f0f402a7
+SRC_URI[archive.sha256sum] = 
2749cc3c0cd7280b299518b1ddf5a5bcfe2d1100614519b68702230e26c7d079
+SRC_URI[patch.md5sum] = 805c0320a2efb21c40ce06fa13cd7c4b
+SRC_URI[patch.sha256sum] = 
9f03b59ccc8826a5be55a3dcde2f889067d58bdc72bf846416a198c9b933704c
-- 
1.7.7.1


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


Re: [oe] [meta-efl][meta-oe 05/12] id3lib: Import from openembedded classic

2011-10-29 Thread Paul Menzel
Dear Martin, dear Denis,


Am Samstag, den 29.10.2011, 12:29 +0200 schrieb Martin Jansa:
 From: Denis 'GNUtoo' Carikli gnu...@no-log.org
 
 Added LIC_FILES_CHKSUM, and fixed LICENSE
 
 Signed-off-by: Denis 'GNUtoo' Carikli gnu...@no-log.org
 Signed-off-by: Martin Jansa martin.ja...@gmail.com

NACK.

please add the version you import and the commit ID you import from.
This is all written in the guide lines [1]!

 ---
  meta-oe/recipes-multimedia/id3lib/id3lib_3.8.3.bb |   18 ++
  1 files changed, 18 insertions(+), 0 deletions(-)
  create mode 100644 meta-oe/recipes-multimedia/id3lib/id3lib_3.8.3.bb
 
 diff --git a/meta-oe/recipes-multimedia/id3lib/id3lib_3.8.3.bb 
 b/meta-oe/recipes-multimedia/id3lib/id3lib_3.8.3.bb
 new file mode 100644
 index 000..9be4ffb
 --- /dev/null
 +++ b/meta-oe/recipes-multimedia/id3lib/id3lib_3.8.3.bb
 @@ -0,0 +1,18 @@
 +DESCRIPTION = Library for interacting with ID3 tags.
 +SECTION = libs/multimedia
 +PRIORITY = optional
 +DEPENDS = zlib
 +LICENSE = GPLv2+

That is not correct [2], it is `LGPLv2+`. Additionally the header
ordering is not the standard. I thought migration to meta-oe was
supposed to be a chance to clean up the recipes.

 +LIC_FILES_CHKSUM = files://COPYING;md5=3bf50002aefd002f49e7bb854063f7e7
 +PR = r0

According to Koen these are going away. Unfortunately these things are
discussed on oe-core instead of openembedded-devel as described in the
Wiki [3].

 +inherit autotools
 +
 +SRC_URI = ${SOURCEFORGE_MIRROR}/id3lib/id3lib-${PV}.tar.gz;name=archive \
 +   
 http://ftp.de.debian.org/debian/pool/main/i/id3lib3.8.3/id3lib3.8.3_3.8.3-7.2.diff.gz;name=patch
  \
 +  
 +
 +SRC_URI[archive.md5sum] = 19f27ddd2dda4b2d26a559a4f0f402a7
 +SRC_URI[archive.sha256sum] = 
 2749cc3c0cd7280b299518b1ddf5a5bcfe2d1100614519b68702230e26c7d079
 +SRC_URI[patch.md5sum] = 805c0320a2efb21c40ce06fa13cd7c4b
 +SRC_URI[patch.sha256sum] = 
 9f03b59ccc8826a5be55a3dcde2f889067d58bdc72bf846416a198c9b933704c


Thanks,

Paul


[1] http://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines
[2] http://git.openembedded.org/openembedded/log/recipes/id3lib/id3lib_3.8.3.bb
[3] http://www.openembedded.org/wiki/Mailing_lists


signature.asc
Description: This is a digitally signed message part
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel