* [PATCH] psplash: Update to latest version of psplash. Tweak LIC_FILES_CHKSUM variable @ 2012-08-13 20:33 Franklin S Cooper Jr 2012-08-22 18:06 ` Cooper Jr., Franklin 0 siblings, 1 reply; 6+ messages in thread From: Franklin S Cooper Jr @ 2012-08-13 20:33 UTC (permalink / raw) To: meta-arago; +Cc: Franklin S Cooper Jr * Add the change to the variable LIC_FILES_CHKSUM to insure that only the lines that contains license information are used for the checksum * Update to the latest version of psplash. * The above changes are already found in the psplash recipe located in oe-core master Signed-off-by: Franklin S Cooper Jr <fcooper@ti.com> --- .../recipes-core/psplash/psplash_git.bbappend | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-) create mode 100644 meta-arago-upstream/recipes-core/psplash/psplash_git.bbappend diff --git a/meta-arago-upstream/recipes-core/psplash/psplash_git.bbappend b/meta-arago-upstream/recipes-core/psplash/psplash_git.bbappend new file mode 100644 index 0000000..0ddbd6a --- /dev/null +++ b/meta-arago-upstream/recipes-core/psplash/psplash_git.bbappend @@ -0,0 +1,4 @@ +LIC_FILES_CHKSUM = "file://psplash.h;beginline=1;endline=16;md5=840fb2356b10a85bed78dd09dc7745c6" + +SRCREV = "de9979aefbc56af59b4d236a4b63dd19dcdcfb53" +PRINC := "${@int(PRINC) + 1}" -- 1.7.0.4 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] psplash: Update to latest version of psplash. Tweak LIC_FILES_CHKSUM variable 2012-08-13 20:33 [PATCH] psplash: Update to latest version of psplash. Tweak LIC_FILES_CHKSUM variable Franklin S Cooper Jr @ 2012-08-22 18:06 ` Cooper Jr., Franklin 2012-08-22 18:14 ` Denys Dmytriyenko 0 siblings, 1 reply; 6+ messages in thread From: Cooper Jr., Franklin @ 2012-08-22 18:06 UTC (permalink / raw) To: Cooper Jr., Franklin, meta-arago@arago-project.org; +Cc: Cooper Jr., Franklin Ignore this patch. A new patch will be sent out that addresses the concern of patch [meta-arago] [PATCH] meta-arago-upstream: Create new upstream layer -----Original Message----- From: Cooper Jr., Franklin Sent: Monday, August 13, 2012 3:33 PM To: meta-arago@arago-project.org Cc: Cooper Jr., Franklin Subject: [meta-arago][PATCH] psplash: Update to latest version of psplash. Tweak LIC_FILES_CHKSUM variable * Add the change to the variable LIC_FILES_CHKSUM to insure that only the lines that contains license information are used for the checksum * Update to the latest version of psplash. * The above changes are already found in the psplash recipe located in oe-core master Signed-off-by: Franklin S Cooper Jr <fcooper@ti.com> --- .../recipes-core/psplash/psplash_git.bbappend | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-) create mode 100644 meta-arago-upstream/recipes-core/psplash/psplash_git.bbappend diff --git a/meta-arago-upstream/recipes-core/psplash/psplash_git.bbappend b/meta-arago-upstream/recipes-core/psplash/psplash_git.bbappend new file mode 100644 index 0000000..0ddbd6a --- /dev/null +++ b/meta-arago-upstream/recipes-core/psplash/psplash_git.bbappend @@ -0,0 +1,4 @@ +LIC_FILES_CHKSUM = "file://psplash.h;beginline=1;endline=16;md5=840fb2356b10a85bed78dd09dc7745c6" + +SRCREV = "de9979aefbc56af59b4d236a4b63dd19dcdcfb53" +PRINC := "${@int(PRINC) + 1}" -- 1.7.0.4 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] psplash: Update to latest version of psplash. Tweak LIC_FILES_CHKSUM variable 2012-08-22 18:06 ` Cooper Jr., Franklin @ 2012-08-22 18:14 ` Denys Dmytriyenko 2012-08-22 18:17 ` Cooper Jr., Franklin 0 siblings, 1 reply; 6+ messages in thread From: Denys Dmytriyenko @ 2012-08-22 18:14 UTC (permalink / raw) To: Cooper Jr., Franklin; +Cc: meta-arago@arago-project.org On Wed, Aug 22, 2012 at 06:06:43PM +0000, Cooper Jr., Franklin wrote: > Ignore this patch. A new patch will be sent out that addresses the concern > of patch [meta-arago] [PATCH] meta-arago-upstream: Create new upstream layer Right, are you going to use a separate "next" branch or just drop it in "master" for now, as we discussed? -- Denys > -----Original Message----- > From: Cooper Jr., Franklin > Sent: Monday, August 13, 2012 3:33 PM > To: meta-arago@arago-project.org > Cc: Cooper Jr., Franklin > Subject: [meta-arago][PATCH] psplash: Update to latest version of psplash. Tweak LIC_FILES_CHKSUM variable > > * Add the change to the variable LIC_FILES_CHKSUM to insure that only > the lines that contains license information are used for the checksum > * Update to the latest version of psplash. > * The above changes are already found in the psplash recipe located in oe-core > master > > Signed-off-by: Franklin S Cooper Jr <fcooper@ti.com> > --- > .../recipes-core/psplash/psplash_git.bbappend | 4 ++++ > 1 files changed, 4 insertions(+), 0 deletions(-) > create mode 100644 meta-arago-upstream/recipes-core/psplash/psplash_git.bbappend > > diff --git a/meta-arago-upstream/recipes-core/psplash/psplash_git.bbappend b/meta-arago-upstream/recipes-core/psplash/psplash_git.bbappend > new file mode 100644 > index 0000000..0ddbd6a > --- /dev/null > +++ b/meta-arago-upstream/recipes-core/psplash/psplash_git.bbappend > @@ -0,0 +1,4 @@ > +LIC_FILES_CHKSUM = "file://psplash.h;beginline=1;endline=16;md5=840fb2356b10a85bed78dd09dc7745c6" > + > +SRCREV = "de9979aefbc56af59b4d236a4b63dd19dcdcfb53" > +PRINC := "${@int(PRINC) + 1}" > -- > 1.7.0.4 > > _______________________________________________ > meta-arago mailing list > meta-arago@arago-project.org > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] psplash: Update to latest version of psplash. Tweak LIC_FILES_CHKSUM variable 2012-08-22 18:14 ` Denys Dmytriyenko @ 2012-08-22 18:17 ` Cooper Jr., Franklin 2012-08-22 19:29 ` Denys Dmytriyenko 0 siblings, 1 reply; 6+ messages in thread From: Cooper Jr., Franklin @ 2012-08-22 18:17 UTC (permalink / raw) To: Dmytriyenko, Denys; +Cc: meta-arago@arago-project.org, Cooper Jr., Franklin Dropping it into master. I add comments to the recipe and to the commit history explaining that these changes were already made in oe-core master. I don't quite understand Chase's comment in the other thread " Could we perhaps us the PV value in the recipe of 0.1+git${SRCPV} in the bbappend to force the append to that specific version and therefore break on an upgrade?" But I am sure what ever changes he wants we can add later. -----Original Message----- From: Dmytriyenko, Denys Sent: Wednesday, August 22, 2012 1:14 PM To: Cooper Jr., Franklin Cc: meta-arago@arago-project.org Subject: Re: [meta-arago] [PATCH] psplash: Update to latest version of psplash. Tweak LIC_FILES_CHKSUM variable On Wed, Aug 22, 2012 at 06:06:43PM +0000, Cooper Jr., Franklin wrote: > Ignore this patch. A new patch will be sent out that addresses the concern > of patch [meta-arago] [PATCH] meta-arago-upstream: Create new upstream layer Right, are you going to use a separate "next" branch or just drop it in "master" for now, as we discussed? -- Denys > -----Original Message----- > From: Cooper Jr., Franklin > Sent: Monday, August 13, 2012 3:33 PM > To: meta-arago@arago-project.org > Cc: Cooper Jr., Franklin > Subject: [meta-arago][PATCH] psplash: Update to latest version of psplash. Tweak LIC_FILES_CHKSUM variable > > * Add the change to the variable LIC_FILES_CHKSUM to insure that only > the lines that contains license information are used for the checksum > * Update to the latest version of psplash. > * The above changes are already found in the psplash recipe located in oe-core > master > > Signed-off-by: Franklin S Cooper Jr <fcooper@ti.com> > --- > .../recipes-core/psplash/psplash_git.bbappend | 4 ++++ > 1 files changed, 4 insertions(+), 0 deletions(-) > create mode 100644 meta-arago-upstream/recipes-core/psplash/psplash_git.bbappend > > diff --git a/meta-arago-upstream/recipes-core/psplash/psplash_git.bbappend b/meta-arago-upstream/recipes-core/psplash/psplash_git.bbappend > new file mode 100644 > index 0000000..0ddbd6a > --- /dev/null > +++ b/meta-arago-upstream/recipes-core/psplash/psplash_git.bbappend > @@ -0,0 +1,4 @@ > +LIC_FILES_CHKSUM = "file://psplash.h;beginline=1;endline=16;md5=840fb2356b10a85bed78dd09dc7745c6" > + > +SRCREV = "de9979aefbc56af59b4d236a4b63dd19dcdcfb53" > +PRINC := "${@int(PRINC) + 1}" > -- > 1.7.0.4 > > _______________________________________________ > meta-arago mailing list > meta-arago@arago-project.org > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] psplash: Update to latest version of psplash. Tweak LIC_FILES_CHKSUM variable 2012-08-22 18:17 ` Cooper Jr., Franklin @ 2012-08-22 19:29 ` Denys Dmytriyenko 2012-08-22 22:32 ` Cooper Jr., Franklin 0 siblings, 1 reply; 6+ messages in thread From: Denys Dmytriyenko @ 2012-08-22 19:29 UTC (permalink / raw) To: Cooper Jr., Franklin; +Cc: meta-arago@arago-project.org On Wed, Aug 22, 2012 at 02:17:10PM -0400, Cooper Jr., Franklin wrote: > Dropping it into master. I add comments to the recipe and to the commit > history explaining that these changes were already made in oe-core master. I > don't quite understand Chase's comment in the other thread > " Could we perhaps us the PV value in the recipe of 0.1+git${SRCPV} in the > bbappend to force the append to that specific version and therefore break on > an upgrade?" But I am sure what ever changes he wants we can add later. What he meant was that psplash_git.bbappend is too generic and will try to apply a year later, even though underlying psplash.bb recipe could have evolved a lot. We can try creating a more specific bbappend with the version in the name - psplash_${PV}.bbappend. Then when PV changes, our bbappend will no longer apply with a big message that the underlying recipe has changed and require our attention. -- Denys > -----Original Message----- > From: Dmytriyenko, Denys > Sent: Wednesday, August 22, 2012 1:14 PM > To: Cooper Jr., Franklin > Cc: meta-arago@arago-project.org > Subject: Re: [meta-arago] [PATCH] psplash: Update to latest version of psplash. Tweak LIC_FILES_CHKSUM variable > > On Wed, Aug 22, 2012 at 06:06:43PM +0000, Cooper Jr., Franklin wrote: > > Ignore this patch. A new patch will be sent out that addresses the concern > > of patch [meta-arago] [PATCH] meta-arago-upstream: Create new upstream layer > > Right, are you going to use a separate "next" branch or just drop it in > "master" for now, as we discussed? > > -- > Denys > > > > -----Original Message----- > > From: Cooper Jr., Franklin > > Sent: Monday, August 13, 2012 3:33 PM > > To: meta-arago@arago-project.org > > Cc: Cooper Jr., Franklin > > Subject: [meta-arago][PATCH] psplash: Update to latest version of psplash. Tweak LIC_FILES_CHKSUM variable > > > > * Add the change to the variable LIC_FILES_CHKSUM to insure that only > > the lines that contains license information are used for the checksum > > * Update to the latest version of psplash. > > * The above changes are already found in the psplash recipe located in oe-core > > master > > > > Signed-off-by: Franklin S Cooper Jr <fcooper@ti.com> > > --- > > .../recipes-core/psplash/psplash_git.bbappend | 4 ++++ > > 1 files changed, 4 insertions(+), 0 deletions(-) > > create mode 100644 meta-arago-upstream/recipes-core/psplash/psplash_git.bbappend > > > > diff --git a/meta-arago-upstream/recipes-core/psplash/psplash_git.bbappend b/meta-arago-upstream/recipes-core/psplash/psplash_git.bbappend > > new file mode 100644 > > index 0000000..0ddbd6a > > --- /dev/null > > +++ b/meta-arago-upstream/recipes-core/psplash/psplash_git.bbappend > > @@ -0,0 +1,4 @@ > > +LIC_FILES_CHKSUM = "file://psplash.h;beginline=1;endline=16;md5=840fb2356b10a85bed78dd09dc7745c6" > > + > > +SRCREV = "de9979aefbc56af59b4d236a4b63dd19dcdcfb53" > > +PRINC := "${@int(PRINC) + 1}" > > -- > > 1.7.0.4 > > > > _______________________________________________ > > meta-arago mailing list > > meta-arago@arago-project.org > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] psplash: Update to latest version of psplash. Tweak LIC_FILES_CHKSUM variable 2012-08-22 19:29 ` Denys Dmytriyenko @ 2012-08-22 22:32 ` Cooper Jr., Franklin 0 siblings, 0 replies; 6+ messages in thread From: Cooper Jr., Franklin @ 2012-08-22 22:32 UTC (permalink / raw) To: Dmytriyenko, Denys; +Cc: meta-arago@arago-project.org I tried your suggestion and couldn't get it to work. I tried other approaches and couldnt get it to work. Can u point to another bbappend that is named based on the PN_PV when the recipe that it is appending doesn't follow that naming convention ie psplash_git.bb? I just need an example to understand what I am doing incorrectly. On Aug 22, 2012, at 2:29 PM, "Dmytriyenko, Denys" <denys@ti.com> wrote: > On Wed, Aug 22, 2012 at 02:17:10PM -0400, Cooper Jr., Franklin wrote: >> Dropping it into master. I add comments to the recipe and to the commit >> history explaining that these changes were already made in oe-core master. I >> don't quite understand Chase's comment in the other thread >> " Could we perhaps us the PV value in the recipe of 0.1+git${SRCPV} in the >> bbappend to force the append to that specific version and therefore break on >> an upgrade?" But I am sure what ever changes he wants we can add later. > > What he meant was that psplash_git.bbappend is too generic and will try to > apply a year later, even though underlying psplash.bb recipe could have > evolved a lot. We can try creating a more specific bbappend with the version > in the name - psplash_${PV}.bbappend. Then when PV changes, our bbappend will > no longer apply with a big message that the underlying recipe has changed and > require our attention. > > -- > Denys > > >> -----Original Message----- >> From: Dmytriyenko, Denys >> Sent: Wednesday, August 22, 2012 1:14 PM >> To: Cooper Jr., Franklin >> Cc: meta-arago@arago-project.org >> Subject: Re: [meta-arago] [PATCH] psplash: Update to latest version of psplash. Tweak LIC_FILES_CHKSUM variable >> >> On Wed, Aug 22, 2012 at 06:06:43PM +0000, Cooper Jr., Franklin wrote: >>> Ignore this patch. A new patch will be sent out that addresses the concern >>> of patch [meta-arago] [PATCH] meta-arago-upstream: Create new upstream layer >> >> Right, are you going to use a separate "next" branch or just drop it in >> "master" for now, as we discussed? >> >> -- >> Denys >> >> >>> -----Original Message----- >>> From: Cooper Jr., Franklin >>> Sent: Monday, August 13, 2012 3:33 PM >>> To: meta-arago@arago-project.org >>> Cc: Cooper Jr., Franklin >>> Subject: [meta-arago][PATCH] psplash: Update to latest version of psplash. Tweak LIC_FILES_CHKSUM variable >>> >>> * Add the change to the variable LIC_FILES_CHKSUM to insure that only >>> the lines that contains license information are used for the checksum >>> * Update to the latest version of psplash. >>> * The above changes are already found in the psplash recipe located in oe-core >>> master >>> >>> Signed-off-by: Franklin S Cooper Jr <fcooper@ti.com> >>> --- >>> .../recipes-core/psplash/psplash_git.bbappend | 4 ++++ >>> 1 files changed, 4 insertions(+), 0 deletions(-) >>> create mode 100644 meta-arago-upstream/recipes-core/psplash/psplash_git.bbappend >>> >>> diff --git a/meta-arago-upstream/recipes-core/psplash/psplash_git.bbappend b/meta-arago-upstream/recipes-core/psplash/psplash_git.bbappend >>> new file mode 100644 >>> index 0000000..0ddbd6a >>> --- /dev/null >>> +++ b/meta-arago-upstream/recipes-core/psplash/psplash_git.bbappend >>> @@ -0,0 +1,4 @@ >>> +LIC_FILES_CHKSUM = "file://psplash.h;beginline=1;endline=16;md5=840fb2356b10a85bed78dd09dc7745c6" >>> + >>> +SRCREV = "de9979aefbc56af59b4d236a4b63dd19dcdcfb53" >>> +PRINC := "${@int(PRINC) + 1}" >>> -- >>> 1.7.0.4 >>> >>> _______________________________________________ >>> meta-arago mailing list >>> meta-arago@arago-project.org >>> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2012-08-22 22:32 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2012-08-13 20:33 [PATCH] psplash: Update to latest version of psplash. Tweak LIC_FILES_CHKSUM variable Franklin S Cooper Jr 2012-08-22 18:06 ` Cooper Jr., Franklin 2012-08-22 18:14 ` Denys Dmytriyenko 2012-08-22 18:17 ` Cooper Jr., Franklin 2012-08-22 19:29 ` Denys Dmytriyenko 2012-08-22 22:32 ` Cooper Jr., Franklin
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.