* [meta-oe][PATCH] smartmontools: import from OE classic @ 2013-04-26 20:39 Nicolas Dechesne 2013-04-26 21:41 ` Koen Kooi 0 siblings, 1 reply; 18+ messages in thread From: Nicolas Dechesne @ 2013-04-26 20:39 UTC (permalink / raw) To: openembedded-devel; +Cc: Nicolas Dechesne Import smartmontools recipe from OE classic, and updated to upstream v5.40. Also added cosmetic changes in the recipe: INC_PR support and homepage. Signed-off-by: Nicolas Dechesne <nicolas.dechesne@linaro.org> --- meta-oe/recipes-support/smartmontools/smartmontools.inc | 10 ++++++++++ meta-oe/recipes-support/smartmontools/smartmontools_5.40.bb | 5 +++++ 2 files changed, 15 insertions(+) create mode 100644 meta-oe/recipes-support/smartmontools/smartmontools.inc create mode 100644 meta-oe/recipes-support/smartmontools/smartmontools_5.40.bb diff --git a/meta-oe/recipes-support/smartmontools/smartmontools.inc b/meta-oe/recipes-support/smartmontools/smartmontools.inc new file mode 100644 index 0000000..6f23055 --- /dev/null +++ b/meta-oe/recipes-support/smartmontools/smartmontools.inc @@ -0,0 +1,10 @@ +SECTION = "console/utils" +DESCRIPTION = "Control and monitor storage systems using S.M.A.R.T." +HOMEPAGE = "http://smartmontools.sourceforge.net/" +LICENSE = "GPLv2" +LIC_FILES_CHKSUM = "file://COPYING;md5=94d55d512a9ba36caa9b7df079bae19f" +INC_PR = "r1" + +SRC_URI = "${SOURCEFORGE_MIRROR}/smartmontools/smartmontools-${PV}.tar.gz" + +inherit autotools diff --git a/meta-oe/recipes-support/smartmontools/smartmontools_5.40.bb b/meta-oe/recipes-support/smartmontools/smartmontools_5.40.bb new file mode 100644 index 0000000..93dc4db --- /dev/null +++ b/meta-oe/recipes-support/smartmontools/smartmontools_5.40.bb @@ -0,0 +1,5 @@ +require smartmontools.inc +PR = "${INC_PR}.1" + +SRC_URI[md5sum] = "0f0be0239914ad87830a4fff594bda5b" +SRC_URI[sha256sum] = "2a68ed20d5d6bd198e1aa6bf0b6037f0f216e3b14da022115244f1af9d248bfd" -- 1.8.1.2 ^ permalink raw reply related [flat|nested] 18+ messages in thread
* Re: [meta-oe][PATCH] smartmontools: import from OE classic 2013-04-26 20:39 [meta-oe][PATCH] smartmontools: import from OE classic Nicolas Dechesne @ 2013-04-26 21:41 ` Koen Kooi 2013-04-27 8:24 ` Paul Eggleton 0 siblings, 1 reply; 18+ messages in thread From: Koen Kooi @ 2013-04-26 21:41 UTC (permalink / raw) To: openembedded-devel -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Op 26-04-13 22:39, Nicolas Dechesne schreef: > Import smartmontools recipe from OE classic, and updated to upstream > v5.40. > > Also added cosmetic changes in the recipe: INC_PR support and homepage. > > Signed-off-by: Nicolas Dechesne <nicolas.dechesne@linaro.org> --- > meta-oe/recipes-support/smartmontools/smartmontools.inc | 10 > ++++++++++ meta-oe/recipes-support/smartmontools/smartmontools_5.40.bb | > 5 +++++ 2 files changed, 15 insertions(+) create mode 100644 > meta-oe/recipes-support/smartmontools/smartmontools.inc create mode > 100644 meta-oe/recipes-support/smartmontools/smartmontools_5.40.bb > > diff --git a/meta-oe/recipes-support/smartmontools/smartmontools.inc > b/meta-oe/recipes-support/smartmontools/smartmontools.inc new file mode > 100644 index 0000000..6f23055 --- /dev/null +++ > b/meta-oe/recipes-support/smartmontools/smartmontools.inc @@ -0,0 +1,10 > @@ +SECTION = "console/utils" +DESCRIPTION = "Control and monitor storage > systems using S.M.A.R.T." +HOMEPAGE = > "http://smartmontools.sourceforge.net/" +LICENSE = "GPLv2" > +LIC_FILES_CHKSUM = > "file://COPYING;md5=94d55d512a9ba36caa9b7df079bae19f" +INC_PR = "r1" Drop that > + +SRC_URI = > "${SOURCEFORGE_MIRROR}/smartmontools/smartmontools-${PV}.tar.gz" + > +inherit autotools diff --git > a/meta-oe/recipes-support/smartmontools/smartmontools_5.40.bb > b/meta-oe/recipes-support/smartmontools/smartmontools_5.40.bb new file > mode 100644 index 0000000..93dc4db --- /dev/null +++ > b/meta-oe/recipes-support/smartmontools/smartmontools_5.40.bb @@ -0,0 > +1,5 @@ +require smartmontools.inc +PR = "${INC_PR}.1" And drop that. And do you really need a .inc? -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.5 (Darwin) Comment: GPGTools - http://gpgtools.org iD8DBQFRevSRMkyGM64RGpERArE1AJ0XBvO7c35yipb1VMI1/GCwgOzCkQCeJQF1 z4Yu+V8MtCJnn0DZKNyIywk= =Jq0r -----END PGP SIGNATURE----- ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [meta-oe][PATCH] smartmontools: import from OE classic 2013-04-26 21:41 ` Koen Kooi @ 2013-04-27 8:24 ` Paul Eggleton 2013-04-27 10:34 ` Philip Balister 0 siblings, 1 reply; 18+ messages in thread From: Paul Eggleton @ 2013-04-27 8:24 UTC (permalink / raw) To: Koen Kooi; +Cc: openembedded-devel On Friday 26 April 2013 23:41:38 Koen Kooi wrote: > -----BEGIN PGP SIGNED MESSAGE----- > Hash: SHA1 > > Op 26-04-13 22:39, Nicolas Dechesne schreef: > > Import smartmontools recipe from OE classic, and updated to upstream > > v5.40. > > > > Also added cosmetic changes in the recipe: INC_PR support and homepage. > > > > Signed-off-by: Nicolas Dechesne <nicolas.dechesne@linaro.org> --- > > meta-oe/recipes-support/smartmontools/smartmontools.inc | 10 > > ++++++++++ meta-oe/recipes-support/smartmontools/smartmontools_5.40.bb | > > 5 +++++ 2 files changed, 15 insertions(+) create mode 100644 > > meta-oe/recipes-support/smartmontools/smartmontools.inc create mode > > 100644 meta-oe/recipes-support/smartmontools/smartmontools_5.40.bb > > > > diff --git a/meta-oe/recipes-support/smartmontools/smartmontools.inc > > b/meta-oe/recipes-support/smartmontools/smartmontools.inc new file mode > > 100644 index 0000000..6f23055 --- /dev/null +++ > > b/meta-oe/recipes-support/smartmontools/smartmontools.inc @@ -0,0 +1,10 > > @@ +SECTION = "console/utils" +DESCRIPTION = "Control and monitor storage > > systems using S.M.A.R.T." +HOMEPAGE = > > "http://smartmontools.sourceforge.net/" +LICENSE = "GPLv2" > > +LIC_FILES_CHKSUM = > > "file://COPYING;md5=94d55d512a9ba36caa9b7df079bae19f" +INC_PR = "r1" > > Drop that It would help for the sake of the uninitiated if you specified what to drop, since your mail client has wrapped the quoted text incorrectly. > > + +SRC_URI = > > "${SOURCEFORGE_MIRROR}/smartmontools/smartmontools-${PV}.tar.gz" + > > +inherit autotools diff --git > > a/meta-oe/recipes-support/smartmontools/smartmontools_5.40.bb > > b/meta-oe/recipes-support/smartmontools/smartmontools_5.40.bb new file > > mode 100644 index 0000000..93dc4db --- /dev/null +++ > > b/meta-oe/recipes-support/smartmontools/smartmontools_5.40.bb @@ -0,0 > > +1,5 @@ +require smartmontools.inc +PR = "${INC_PR}.1" > > And drop that. > > And do you really need a .inc? Are we removing inc files if they were present in OE Classic? First I've heard if we are... Cheers, Paul -- Paul Eggleton Intel Open Source Technology Centre ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [meta-oe][PATCH] smartmontools: import from OE classic 2013-04-27 8:24 ` Paul Eggleton @ 2013-04-27 10:34 ` Philip Balister 2013-04-27 11:13 ` Paul Eggleton 0 siblings, 1 reply; 18+ messages in thread From: Philip Balister @ 2013-04-27 10:34 UTC (permalink / raw) To: openembedded-devel; +Cc: Koen Kooi On 04/27/2013 04:24 AM, Paul Eggleton wrote: > On Friday 26 April 2013 23:41:38 Koen Kooi wrote: >> -----BEGIN PGP SIGNED MESSAGE----- >> Hash: SHA1 >> >> Op 26-04-13 22:39, Nicolas Dechesne schreef: >>> Import smartmontools recipe from OE classic, and updated to upstream >>> v5.40. >>> >>> Also added cosmetic changes in the recipe: INC_PR support and homepage. >>> >>> Signed-off-by: Nicolas Dechesne <nicolas.dechesne@linaro.org> --- >>> meta-oe/recipes-support/smartmontools/smartmontools.inc | 10 >>> ++++++++++ meta-oe/recipes-support/smartmontools/smartmontools_5.40.bb | >>> 5 +++++ 2 files changed, 15 insertions(+) create mode 100644 >>> meta-oe/recipes-support/smartmontools/smartmontools.inc create mode >>> 100644 meta-oe/recipes-support/smartmontools/smartmontools_5.40.bb >>> >>> diff --git a/meta-oe/recipes-support/smartmontools/smartmontools.inc >>> b/meta-oe/recipes-support/smartmontools/smartmontools.inc new file mode >>> 100644 index 0000000..6f23055 --- /dev/null +++ >>> b/meta-oe/recipes-support/smartmontools/smartmontools.inc @@ -0,0 +1,10 >>> @@ +SECTION = "console/utils" +DESCRIPTION = "Control and monitor storage >>> systems using S.M.A.R.T." +HOMEPAGE = >>> "http://smartmontools.sourceforge.net/" +LICENSE = "GPLv2" >>> +LIC_FILES_CHKSUM = >>> "file://COPYING;md5=94d55d512a9ba36caa9b7df079bae19f" +INC_PR = "r1" >> >> Drop that > > It would help for the sake of the uninitiated if you specified what to drop, > since your mail client has wrapped the quoted text incorrectly. > >>> + +SRC_URI = >>> "${SOURCEFORGE_MIRROR}/smartmontools/smartmontools-${PV}.tar.gz" + >>> +inherit autotools diff --git >>> a/meta-oe/recipes-support/smartmontools/smartmontools_5.40.bb >>> b/meta-oe/recipes-support/smartmontools/smartmontools_5.40.bb new file >>> mode 100644 index 0000000..93dc4db --- /dev/null +++ >>> b/meta-oe/recipes-support/smartmontools/smartmontools_5.40.bb @@ -0,0 >>> +1,5 @@ +require smartmontools.inc +PR = "${INC_PR}.1" >> >> And drop that. >> >> And do you really need a .inc? > > Are we removing inc files if they were present in OE Classic? First I've heard > if we are... If we are trying to reduce the number of versions of recipes we carry, dropping .inc files would seem to be a good idea. I don't have strong feelings, but it seems like something we should consider. Philip ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [meta-oe][PATCH] smartmontools: import from OE classic 2013-04-27 10:34 ` Philip Balister @ 2013-04-27 11:13 ` Paul Eggleton 2013-04-29 6:40 ` Nicolas Dechesne 2013-05-03 14:10 ` Koen Kooi 0 siblings, 2 replies; 18+ messages in thread From: Paul Eggleton @ 2013-04-27 11:13 UTC (permalink / raw) To: Philip Balister; +Cc: Koen Kooi, openembedded-devel On Saturday 27 April 2013 06:34:46 Philip Balister wrote: > On 04/27/2013 04:24 AM, Paul Eggleton wrote: > > On Friday 26 April 2013 23:41:38 Koen Kooi wrote: > >> And do you really need a .inc? > > > > Are we removing inc files if they were present in OE Classic? First I've > > heard if we are... > > If we are trying to reduce the number of versions of recipes we carry, > dropping .inc files would seem to be a good idea. I don't have strong > feelings, but it seems like something we should consider. I agree we should try to keep only one version of each recipe in software layers, however I figure it makes it easier for people to carry their own versions of recipes in distro layers (particularly older, which may be required in certain circumstances) if we do keep inc files where they already exist. Cheers, Paul -- Paul Eggleton Intel Open Source Technology Centre ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [meta-oe][PATCH] smartmontools: import from OE classic 2013-04-27 11:13 ` Paul Eggleton @ 2013-04-29 6:40 ` Nicolas Dechesne 2013-05-03 14:00 ` Nicolas Dechesne 2013-05-03 14:10 ` Koen Kooi 1 sibling, 1 reply; 18+ messages in thread From: Nicolas Dechesne @ 2013-04-29 6:40 UTC (permalink / raw) To: openembedded-devel; +Cc: Koen Kooi On Sat, Apr 27, 2013 at 1:13 PM, Paul Eggleton <paul.eggleton@linux.intel.com> wrote: >> > On Friday 26 April 2013 23:41:38 Koen Kooi wrote: >> >> And do you really need a .inc? >> > >> > Are we removing inc files if they were present in OE Classic? First I've >> > heard if we are... >> >> If we are trying to reduce the number of versions of recipes we carry, >> dropping .inc files would seem to be a good idea. I don't have strong >> feelings, but it seems like something we should consider. > > I agree we should try to keep only one version of each recipe in software > layers, however I figure it makes it easier for people to carry their own > versions of recipes in distro layers (particularly older, which may be > required in certain circumstances) if we do keep inc files where they already > exist. I put the .inc in this patch, indeed because it was there in OE Classic. I can update the patch if there is a consensus to remove the .inc. also for the INC_PR, I added it, because I thought it makes sense for .bb with .inc to have that. again, i can update the patch if you recommend doing this way. thx ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [meta-oe][PATCH] smartmontools: import from OE classic 2013-04-29 6:40 ` Nicolas Dechesne @ 2013-05-03 14:00 ` Nicolas Dechesne 2013-05-03 14:35 ` Paul Eggleton 0 siblings, 1 reply; 18+ messages in thread From: Nicolas Dechesne @ 2013-05-03 14:00 UTC (permalink / raw) To: openembedded-devel; +Cc: Koen Kooi On Mon, Apr 29, 2013 at 8:40 AM, Nicolas Dechesne <nicolas.dechesne@linaro.org> wrote: > On Sat, Apr 27, 2013 at 1:13 PM, Paul Eggleton > <paul.eggleton@linux.intel.com> wrote: >>> > On Friday 26 April 2013 23:41:38 Koen Kooi wrote: >>> >> And do you really need a .inc? >>> > >>> > Are we removing inc files if they were present in OE Classic? First I've >>> > heard if we are... >>> >>> If we are trying to reduce the number of versions of recipes we carry, >>> dropping .inc files would seem to be a good idea. I don't have strong >>> feelings, but it seems like something we should consider. >> >> I agree we should try to keep only one version of each recipe in software >> layers, however I figure it makes it easier for people to carry their own >> versions of recipes in distro layers (particularly older, which may be >> required in certain circumstances) if we do keep inc files where they already >> exist. > > I put the .inc in this patch, indeed because it was there in OE > Classic. I can update the patch if there is a consensus to remove the > .inc. > > also for the INC_PR, I added it, because I thought it makes sense for > .bb with .inc to have that. again, i can update the patch if you > recommend doing this way. > > thx hi, can you please let me know what I should do here? i can update the patch if needed, but not sure there is a clear consensus on what to do! nicolas ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [meta-oe][PATCH] smartmontools: import from OE classic 2013-05-03 14:00 ` Nicolas Dechesne @ 2013-05-03 14:35 ` Paul Eggleton 2013-05-03 14:40 ` Koen Kooi 0 siblings, 1 reply; 18+ messages in thread From: Paul Eggleton @ 2013-05-03 14:35 UTC (permalink / raw) To: Nicolas Dechesne; +Cc: Koen Kooi, openembedded-devel On Friday 03 May 2013 16:00:13 Nicolas Dechesne wrote: > On Mon, Apr 29, 2013 at 8:40 AM, Nicolas Dechesne > > <nicolas.dechesne@linaro.org> wrote: > > On Sat, Apr 27, 2013 at 1:13 PM, Paul Eggleton > > > > <paul.eggleton@linux.intel.com> wrote: > >>> > On Friday 26 April 2013 23:41:38 Koen Kooi wrote: > >>> >> And do you really need a .inc? > >>> > > >>> > Are we removing inc files if they were present in OE Classic? First > >>> > I've > >>> > heard if we are... > >>> > >>> If we are trying to reduce the number of versions of recipes we carry, > >>> dropping .inc files would seem to be a good idea. I don't have strong > >>> feelings, but it seems like something we should consider. > >> > >> I agree we should try to keep only one version of each recipe in software > >> layers, however I figure it makes it easier for people to carry their own > >> versions of recipes in distro layers (particularly older, which may be > >> required in certain circumstances) if we do keep inc files where they > >> already exist. > > > > I put the .inc in this patch, indeed because it was there in OE > > Classic. I can update the patch if there is a consensus to remove the > > .inc. > > > > also for the INC_PR, I added it, because I thought it makes sense for > > .bb with .inc to have that. again, i can update the patch if you > > recommend doing this way. > > hi, can you please let me know what I should do here? i can update the > patch if needed, but not sure there is a clear consensus on what to > do! IMO, let's keep the separate inc file, but drop PR and INC_PR. Cheers, Paul -- Paul Eggleton Intel Open Source Technology Centre ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [meta-oe][PATCH] smartmontools: import from OE classic 2013-05-03 14:35 ` Paul Eggleton @ 2013-05-03 14:40 ` Koen Kooi 2013-05-03 14:48 ` Paul Eggleton 0 siblings, 1 reply; 18+ messages in thread From: Koen Kooi @ 2013-05-03 14:40 UTC (permalink / raw) To: Paul Eggleton; +Cc: openembedded-devel Op 3 mei 2013, om 16:35 heeft Paul Eggleton <paul.eggleton@linux.intel.com> het volgende geschreven: > On Friday 03 May 2013 16:00:13 Nicolas Dechesne wrote: >> On Mon, Apr 29, 2013 at 8:40 AM, Nicolas Dechesne >> >> <nicolas.dechesne@linaro.org> wrote: >>> On Sat, Apr 27, 2013 at 1:13 PM, Paul Eggleton >>> >>> <paul.eggleton@linux.intel.com> wrote: >>>>>> On Friday 26 April 2013 23:41:38 Koen Kooi wrote: >>>>>>> And do you really need a .inc? >>>>>> >>>>>> Are we removing inc files if they were present in OE Classic? First >>>>>> I've >>>>>> heard if we are... >>>>> >>>>> If we are trying to reduce the number of versions of recipes we carry, >>>>> dropping .inc files would seem to be a good idea. I don't have strong >>>>> feelings, but it seems like something we should consider. >>>> >>>> I agree we should try to keep only one version of each recipe in software >>>> layers, however I figure it makes it easier for people to carry their own >>>> versions of recipes in distro layers (particularly older, which may be >>>> required in certain circumstances) if we do keep inc files where they >>>> already exist. >>> >>> I put the .inc in this patch, indeed because it was there in OE >>> Classic. I can update the patch if there is a consensus to remove the >>> .inc. >>> >>> also for the INC_PR, I added it, because I thought it makes sense for >>> .bb with .inc to have that. again, i can update the patch if you >>> recommend doing this way. >> >> hi, can you please let me know what I should do here? i can update the >> patch if needed, but not sure there is a clear consensus on what to >> do! > > IMO, let's keep the separate inc file, but drop PR and INC_PR. I still haven't heard a compelling case why smartmontools needs a .inc and all the other recipes in meta-oe don't. So drop the inc and be consistent with other recipes. ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [meta-oe][PATCH] smartmontools: import from OE classic 2013-05-03 14:40 ` Koen Kooi @ 2013-05-03 14:48 ` Paul Eggleton 0 siblings, 0 replies; 18+ messages in thread From: Paul Eggleton @ 2013-05-03 14:48 UTC (permalink / raw) To: Koen Kooi; +Cc: openembedded-devel On Friday 03 May 2013 16:40:13 Koen Kooi wrote: > Op 3 mei 2013, om 16:35 heeft Paul Eggleton <paul.eggleton@linux.intel.com> > het volgende geschreven: > > IMO, let's keep the separate inc file, but drop PR and INC_PR. > > I still haven't heard a compelling case why smartmontools needs a .inc and > all the other recipes in meta-oe don't. So drop the inc and be consistent > with other recipes. All the other recipes? You mean all of them except for the significant number you can find if you run "git grep require.*inc" in meta-oe? On Friday 03 May 2013 16:39:13 Koen Kooi wrote: > It's unnecessary clutter and in the smartmontools case unneeded as well. How is it any less necessary for smartmontools than any other recipe? Cheers, Paul -- Paul Eggleton Intel Open Source Technology Centre ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [meta-oe][PATCH] smartmontools: import from OE classic 2013-04-27 11:13 ` Paul Eggleton 2013-04-29 6:40 ` Nicolas Dechesne @ 2013-05-03 14:10 ` Koen Kooi 2013-05-03 14:30 ` Paul Eggleton 2013-05-03 14:30 ` Nicolas Dechesne 1 sibling, 2 replies; 18+ messages in thread From: Koen Kooi @ 2013-05-03 14:10 UTC (permalink / raw) To: Paul Eggleton; +Cc: openembedded-devel Op 27 apr. 2013, om 13:13 heeft Paul Eggleton <paul.eggleton@linux.intel.com> het volgende geschreven: > On Saturday 27 April 2013 06:34:46 Philip Balister wrote: >> On 04/27/2013 04:24 AM, Paul Eggleton wrote: >>> On Friday 26 April 2013 23:41:38 Koen Kooi wrote: >>>> And do you really need a .inc? >>> >>> Are we removing inc files if they were present in OE Classic? First I've >>> heard if we are... >> >> If we are trying to reduce the number of versions of recipes we carry, >> dropping .inc files would seem to be a good idea. I don't have strong >> feelings, but it seems like something we should consider. > > I agree we should try to keep only one version of each recipe in software > layers, however I figure it makes it easier for people to carry their own > versions of recipes in distro layers (particularly older, which may be > required in certain circumstances) if we do keep inc files where they already > exist. Can people raise their hand if they want to have a different version of smartmontools in their layer? ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [meta-oe][PATCH] smartmontools: import from OE classic 2013-05-03 14:10 ` Koen Kooi @ 2013-05-03 14:30 ` Paul Eggleton 2013-05-03 14:39 ` Koen Kooi 2013-05-03 14:30 ` Nicolas Dechesne 1 sibling, 1 reply; 18+ messages in thread From: Paul Eggleton @ 2013-05-03 14:30 UTC (permalink / raw) To: Koen Kooi; +Cc: openembedded-devel On Friday 03 May 2013 16:10:40 Koen Kooi wrote: > Op 27 apr. 2013, om 13:13 heeft Paul Eggleton <paul.eggleton@linux.intel.com> het volgende geschreven: > > On Saturday 27 April 2013 06:34:46 Philip Balister wrote: > >> On 04/27/2013 04:24 AM, Paul Eggleton wrote: > >>> On Friday 26 April 2013 23:41:38 Koen Kooi wrote: > >>>> And do you really need a .inc? > >>> > >>> Are we removing inc files if they were present in OE Classic? First I've > >>> heard if we are... > >> > >> If we are trying to reduce the number of versions of recipes we carry, > >> dropping .inc files would seem to be a good idea. I don't have strong > >> feelings, but it seems like something we should consider. > > > > I agree we should try to keep only one version of each recipe in software > > layers, however I figure it makes it easier for people to carry their own > > versions of recipes in distro layers (particularly older, which may be > > required in certain circumstances) if we do keep inc files where they > > already exist. > > Can people raise their hand if they want to have a different version of > smartmontools in their layer? Even assuming everyone who could possibly want this is reading this thread, which is unlikely, in the absence of time machines you won't hear from anyone who doesn't need it now but does in the future. As a general rule, people do often want to build versions of recipes from SCM repositories and having an inc file makes that a bit easier. If we already have the inc file split I can't see a compelling reason to drop it. I would also point out that we were asked to preserve these in the original move to OE-Core and as far as I am aware we have done so. Cheers, Paul -- Paul Eggleton Intel Open Source Technology Centre ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [meta-oe][PATCH] smartmontools: import from OE classic 2013-05-03 14:30 ` Paul Eggleton @ 2013-05-03 14:39 ` Koen Kooi 0 siblings, 0 replies; 18+ messages in thread From: Koen Kooi @ 2013-05-03 14:39 UTC (permalink / raw) To: Paul Eggleton; +Cc: openembedded-devel Op 3 mei 2013, om 16:30 heeft Paul Eggleton <paul.eggleton@linux.intel.com> het volgende geschreven: > On Friday 03 May 2013 16:10:40 Koen Kooi wrote: >> Op 27 apr. 2013, om 13:13 heeft Paul Eggleton > <paul.eggleton@linux.intel.com> het volgende geschreven: >>> On Saturday 27 April 2013 06:34:46 Philip Balister wrote: >>>> On 04/27/2013 04:24 AM, Paul Eggleton wrote: >>>>> On Friday 26 April 2013 23:41:38 Koen Kooi wrote: >>>>>> And do you really need a .inc? >>>>> >>>>> Are we removing inc files if they were present in OE Classic? First I've >>>>> heard if we are... >>>> >>>> If we are trying to reduce the number of versions of recipes we carry, >>>> dropping .inc files would seem to be a good idea. I don't have strong >>>> feelings, but it seems like something we should consider. >>> >>> I agree we should try to keep only one version of each recipe in software >>> layers, however I figure it makes it easier for people to carry their own >>> versions of recipes in distro layers (particularly older, which may be >>> required in certain circumstances) if we do keep inc files where they >>> already exist. >> >> Can people raise their hand if they want to have a different version of >> smartmontools in their layer? > > Even assuming everyone who could possibly want this is reading this thread, > which is unlikely, in the absence of time machines you won't hear from anyone > who doesn't need it now but does in the future. > > As a general rule, people do often want to build versions of recipes from SCM > repositories and having an inc file makes that a bit easier. If we already have > the inc file split I can't see a compelling reason to drop it. > > I would also point out that we were asked to preserve these in the original > move to OE-Core and as far as I am aware we have done so. It's unnecessary clutter and in the smartmontools case unneeded as well. ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [meta-oe][PATCH] smartmontools: import from OE classic 2013-05-03 14:10 ` Koen Kooi 2013-05-03 14:30 ` Paul Eggleton @ 2013-05-03 14:30 ` Nicolas Dechesne 2013-05-03 14:42 ` Paul Eggleton 1 sibling, 1 reply; 18+ messages in thread From: Nicolas Dechesne @ 2013-05-03 14:30 UTC (permalink / raw) To: openembedded-devel On Fri, May 3, 2013 at 4:10 PM, Koen Kooi <koen@dominion.thruhere.net>wrote: > > I agree we should try to keep only one version of each recipe in software > > layers, however I figure it makes it easier for people to carry their own > > versions of recipes in distro layers (particularly older, which may be > > required in certain circumstances) if we do keep inc files where they > already > > exist. > > Can people raise their hand if they want to have a different version of > smartmontools in their layer? i think it would be nice to have a policy for that. i am going to send a couple of recipes for new components, and it would be good to know what's the agreed 'policy' for .inc file. in fact, it's not even clear to me why have .inc file is easier to carry different version in distro layers. well it's just a 'large' recipe to carry over.. nicolas ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [meta-oe][PATCH] smartmontools: import from OE classic 2013-05-03 14:30 ` Nicolas Dechesne @ 2013-05-03 14:42 ` Paul Eggleton 2013-05-03 15:09 ` Martin Jansa 0 siblings, 1 reply; 18+ messages in thread From: Paul Eggleton @ 2013-05-03 14:42 UTC (permalink / raw) To: Nicolas Dechesne; +Cc: openembedded-devel On Friday 03 May 2013 16:30:17 Nicolas Dechesne wrote: > On Fri, May 3, 2013 at 4:10 PM, Koen Kooi <koen@dominion.thruhere.net>wrote: > > > I agree we should try to keep only one version of each recipe in > > > software > > > layers, however I figure it makes it easier for people to carry their > > > own > > > versions of recipes in distro layers (particularly older, which may be > > > required in certain circumstances) if we do keep inc files where they > > > already exist. > > > Can people raise their hand if they want to have a different version of > > smartmontools in their layer? > > i think it would be nice to have a policy for that. i am going to send a > couple of recipes for new components, and it would be good to know what's > the agreed 'policy' for .inc file. We probably should have a stated policy, yes. AFAIK the current perhaps unstated policy is to keep the inc file if it existed in OE-Classic. > in fact, it's not even clear to me why > have .inc file is easier to carry different version in distro layers. well > it's just a 'large' recipe to carry over.. If you carry over the whole recipe rather than "require recipes- xyz/foo/foo.inc", then you won't get any improvements to the non-version- specific parts of the recipe when the base layer updates in the future. Alternatively if you "require recipes-xyz/foo/foo_6.5.bb" because there is no .inc, then your recipe will definitely break the next time the recipe in the base layer is upgraded, plus you may have to override more parts of the recipe that are specific to a different version than the one you are building. Cheers, Paul -- Paul Eggleton Intel Open Source Technology Centre ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [meta-oe][PATCH] smartmontools: import from OE classic 2013-05-03 14:42 ` Paul Eggleton @ 2013-05-03 15:09 ` Martin Jansa 2013-05-03 15:15 ` Philip Balister 0 siblings, 1 reply; 18+ messages in thread From: Martin Jansa @ 2013-05-03 15:09 UTC (permalink / raw) To: openembedded-devel [-- Attachment #1: Type: text/plain, Size: 2528 bytes --] On Fri, May 03, 2013 at 03:42:30PM +0100, Paul Eggleton wrote: > On Friday 03 May 2013 16:30:17 Nicolas Dechesne wrote: > > On Fri, May 3, 2013 at 4:10 PM, Koen Kooi <koen@dominion.thruhere.net>wrote: > > > > I agree we should try to keep only one version of each recipe in > > > > software > > > > layers, however I figure it makes it easier for people to carry their > > > > own > > > > versions of recipes in distro layers (particularly older, which may be > > > > required in certain circumstances) if we do keep inc files where they > > > > already exist. > > > > > Can people raise their hand if they want to have a different version of > > > smartmontools in their layer? > > > > i think it would be nice to have a policy for that. i am going to send a > > couple of recipes for new components, and it would be good to know what's > > the agreed 'policy' for .inc file. > > We probably should have a stated policy, yes. AFAIK the current perhaps > unstated policy is to keep the inc file if it existed in OE-Classic. I don't have strong opinion about keeping/loosing .inc files, but I've removed couple of .inc files when importing stuff from OE-Classic in cases where .inc is relatively short (like in smartmontools). I would keep .inc when it's reused in different recipes or really long/complicated (like mesa.inc). We also have policy that there is only one version per recipe if possible. > > in fact, it's not even clear to me why > > have .inc file is easier to carry different version in distro layers. well > > it's just a 'large' recipe to carry over.. > > If you carry over the whole recipe rather than "require recipes- > xyz/foo/foo.inc", then you won't get any improvements to the non-version- > specific parts of the recipe when the base layer updates in the future. > > Alternatively if you "require recipes-xyz/foo/foo_6.5.bb" because there is no > .inc, then your recipe will definitely break the next time the recipe in the > base layer is upgraded, plus you may have to override more parts of the recipe > that are specific to a different version than the one you are building. > > 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 -- Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 205 bytes --] ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [meta-oe][PATCH] smartmontools: import from OE classic 2013-05-03 15:09 ` Martin Jansa @ 2013-05-03 15:15 ` Philip Balister 2013-05-06 8:19 ` Nicolas Dechesne 0 siblings, 1 reply; 18+ messages in thread From: Philip Balister @ 2013-05-03 15:15 UTC (permalink / raw) To: openembedded-devel; +Cc: Martin Jansa [-- Attachment #1: Type: text/plain, Size: 2857 bytes --] On 05/03/2013 11:09 AM, Martin Jansa wrote: > On Fri, May 03, 2013 at 03:42:30PM +0100, Paul Eggleton wrote: >> On Friday 03 May 2013 16:30:17 Nicolas Dechesne wrote: >>> On Fri, May 3, 2013 at 4:10 PM, Koen Kooi <koen@dominion.thruhere.net>wrote: >>>>> I agree we should try to keep only one version of each recipe in >>>>> software >>>>> layers, however I figure it makes it easier for people to carry their >>>>> own >>>>> versions of recipes in distro layers (particularly older, which may be >>>>> required in certain circumstances) if we do keep inc files where they >>>>> already exist. >>> >>>> Can people raise their hand if they want to have a different version of >>>> smartmontools in their layer? >>> >>> i think it would be nice to have a policy for that. i am going to send a >>> couple of recipes for new components, and it would be good to know what's >>> the agreed 'policy' for .inc file. >> >> We probably should have a stated policy, yes. AFAIK the current perhaps >> unstated policy is to keep the inc file if it existed in OE-Classic. > > I don't have strong opinion about keeping/loosing .inc files, but > I've removed couple of .inc files when importing stuff from OE-Classic > in cases where .inc is relatively short (like in smartmontools). > > I would keep .inc when it's reused in different recipes or really > long/complicated (like mesa.inc). > > We also have policy that there is only one version per recipe if > possible. I agree with Martin. I've dropped some .inc files for recipes I worry about also. Philip > >>> in fact, it's not even clear to me why >>> have .inc file is easier to carry different version in distro layers. well >>> it's just a 'large' recipe to carry over.. >> >> If you carry over the whole recipe rather than "require recipes- >> xyz/foo/foo.inc", then you won't get any improvements to the non-version- >> specific parts of the recipe when the base layer updates in the future. >> >> Alternatively if you "require recipes-xyz/foo/foo_6.5.bb" because there is no >> .inc, then your recipe will definitely break the next time the recipe in the >> base layer is upgraded, plus you may have to override more parts of the recipe >> that are specific to a different version than the one you are building. >> >> 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 > > > > _______________________________________________ > Openembedded-devel mailing list > Openembedded-devel@lists.openembedded.org > http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel > [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 567 bytes --] ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [meta-oe][PATCH] smartmontools: import from OE classic 2013-05-03 15:15 ` Philip Balister @ 2013-05-06 8:19 ` Nicolas Dechesne 0 siblings, 0 replies; 18+ messages in thread From: Nicolas Dechesne @ 2013-05-06 8:19 UTC (permalink / raw) To: openembedded-devel; +Cc: Martin Jansa On Fri, May 3, 2013 at 5:15 PM, Philip Balister <philip@balister.org> wrote: > > I don't have strong opinion about keeping/loosing .inc files, but > > I've removed couple of .inc files when importing stuff from OE-Classic > > in cases where .inc is relatively short (like in smartmontools). > > > > I would keep .inc when it's reused in different recipes or really > > long/complicated (like mesa.inc). > > > > We also have policy that there is only one version per recipe if > > possible. > > I agree with Martin. I've dropped some .inc files for recipes I worry > about also. > ok ,thanks for all the inputs. I will rework the recipe, and remove the .inc as more people seem to be willing it this way. also, i realized that there is newer upstream version of smartmontools, so i will upgrade the recipe to the most recent one too. ^ permalink raw reply [flat|nested] 18+ messages in thread
end of thread, other threads:[~2013-05-06 8:37 UTC | newest] Thread overview: 18+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2013-04-26 20:39 [meta-oe][PATCH] smartmontools: import from OE classic Nicolas Dechesne 2013-04-26 21:41 ` Koen Kooi 2013-04-27 8:24 ` Paul Eggleton 2013-04-27 10:34 ` Philip Balister 2013-04-27 11:13 ` Paul Eggleton 2013-04-29 6:40 ` Nicolas Dechesne 2013-05-03 14:00 ` Nicolas Dechesne 2013-05-03 14:35 ` Paul Eggleton 2013-05-03 14:40 ` Koen Kooi 2013-05-03 14:48 ` Paul Eggleton 2013-05-03 14:10 ` Koen Kooi 2013-05-03 14:30 ` Paul Eggleton 2013-05-03 14:39 ` Koen Kooi 2013-05-03 14:30 ` Nicolas Dechesne 2013-05-03 14:42 ` Paul Eggleton 2013-05-03 15:09 ` Martin Jansa 2013-05-03 15:15 ` Philip Balister 2013-05-06 8:19 ` Nicolas Dechesne
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.