* [PATCH] Revert "image.bbclass: don't execute compression commands multiple times" @ 2016-05-23 17:50 Koen Kooi 2016-05-30 11:28 ` Koen Kooi 0 siblings, 1 reply; 11+ messages in thread From: Koen Kooi @ 2016-05-23 17:50 UTC (permalink / raw) To: openembedded-core; +Cc: Koen Kooi The commit breaks image_types_uboot.bbclass: ERROR: Logfile of failure stored in: /build/linaro/build/build/tmp-glibc/work/genericarmv8-oe-linux/linaro-image-minimal-initramfs/1.0-r0/temp/log.do_image_cpio.16506 Log data follows: | DEBUG: Executing python function set_image_size | DEBUG: Python function set_image_size finished | DEBUG: Executing shell function do_image_cpio | 31373 blocks | Image Name: linaro-image-minimal-initramfs-g | Created: Mon May 23 19:38:56 2016 | Image Type: ARM Linux RAMDisk Image (gzip compressed) | Data Size: 7326620 Bytes = 7154.90 kB = 6.99 MB | Load Address: 00000000 | Entry Point: 00000000 | mkimage: Can't open /build/linaro/build/build/tmp-glibc/deploy/images/genericarmv8/linaro-image-minimal-initramfs-genericarmv8-20160523173835.rootfs.cpio.gz: No such file or directory This reverts commit 94f61c2682e5cfd819ac84535650c3e0a654415a. Signed-off-by: Koen Kooi <koen.kooi@linaro.org> --- meta/classes/image.bbclass | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/meta/classes/image.bbclass b/meta/classes/image.bbclass index 9f4c83f..df3bd3a 100644 --- a/meta/classes/image.bbclass +++ b/meta/classes/image.bbclass @@ -410,13 +410,9 @@ python () { # Create input image first. gen_conversion_cmds(type) localdata.setVar('type', type) - cmd = "\t" + localdata.getVar("COMPRESS_CMD_" + ctype, True) - if cmd not in cmds: - cmds.append(cmd) + cmds.append("\t" + localdata.getVar("COMPRESS_CMD_" + ctype, True)) vardeps.add('COMPRESS_CMD_' + ctype) - subimage = type + "." + ctype - if subimage not in subimages: - subimages.append(subimage) + subimages.append(type + "." + ctype) if type not in alltypes: rm_tmp_images.add(localdata.expand("${IMAGE_NAME}${IMAGE_NAME_SUFFIX}.${type}")) -- 2.4.3 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH] Revert "image.bbclass: don't execute compression commands multiple times" 2016-05-23 17:50 [PATCH] Revert "image.bbclass: don't execute compression commands multiple times" Koen Kooi @ 2016-05-30 11:28 ` Koen Kooi 2016-06-04 8:41 ` Koen Kooi 0 siblings, 1 reply; 11+ messages in thread From: Koen Kooi @ 2016-05-30 11:28 UTC (permalink / raw) To: Koen Kooi; +Cc: openembedded-core ping > Op 23 mei 2016, om 19:50 heeft Koen Kooi <koen.kooi@linaro.org> het volgende geschreven: > > The commit breaks image_types_uboot.bbclass: > > ERROR: Logfile of failure stored in: /build/linaro/build/build/tmp-glibc/work/genericarmv8-oe-linux/linaro-image-minimal-initramfs/1.0-r0/temp/log.do_image_cpio.16506 > Log data follows: > | DEBUG: Executing python function set_image_size > | DEBUG: Python function set_image_size finished > | DEBUG: Executing shell function do_image_cpio > | 31373 blocks > | Image Name: linaro-image-minimal-initramfs-g > | Created: Mon May 23 19:38:56 2016 > | Image Type: ARM Linux RAMDisk Image (gzip compressed) > | Data Size: 7326620 Bytes = 7154.90 kB = 6.99 MB > | Load Address: 00000000 > | Entry Point: 00000000 > | mkimage: Can't open /build/linaro/build/build/tmp-glibc/deploy/images/genericarmv8/linaro-image-minimal-initramfs-genericarmv8-20160523173835.rootfs.cpio.gz: No such file or directory > > This reverts commit 94f61c2682e5cfd819ac84535650c3e0a654415a. > > Signed-off-by: Koen Kooi <koen.kooi@linaro.org> > --- > meta/classes/image.bbclass | 8 ++------ > 1 file changed, 2 insertions(+), 6 deletions(-) > > diff --git a/meta/classes/image.bbclass b/meta/classes/image.bbclass > index 9f4c83f..df3bd3a 100644 > --- a/meta/classes/image.bbclass > +++ b/meta/classes/image.bbclass > @@ -410,13 +410,9 @@ python () { > # Create input image first. > gen_conversion_cmds(type) > localdata.setVar('type', type) > - cmd = "\t" + localdata.getVar("COMPRESS_CMD_" + ctype, True) > - if cmd not in cmds: > - cmds.append(cmd) > + cmds.append("\t" + localdata.getVar("COMPRESS_CMD_" + ctype, True)) > vardeps.add('COMPRESS_CMD_' + ctype) > - subimage = type + "." + ctype > - if subimage not in subimages: > - subimages.append(subimage) > + subimages.append(type + "." + ctype) > if type not in alltypes: > rm_tmp_images.add(localdata.expand("${IMAGE_NAME}${IMAGE_NAME_SUFFIX}.${type}")) > > -- > 2.4.3 > > -- > _______________________________________________ > Openembedded-core mailing list > Openembedded-core@lists.openembedded.org > http://lists.openembedded.org/mailman/listinfo/openembedded-core > ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] Revert "image.bbclass: don't execute compression commands multiple times" 2016-05-30 11:28 ` Koen Kooi @ 2016-06-04 8:41 ` Koen Kooi 2016-06-04 13:09 ` Richard Purdie 0 siblings, 1 reply; 11+ messages in thread From: Koen Kooi @ 2016-06-04 8:41 UTC (permalink / raw) To: Koen Kooi; +Cc: openembedded-core ping 2 > Op 30 mei 2016, om 13:28 heeft Koen Kooi <koen@dominion.thruhere.net> het volgende geschreven: > > ping > >> Op 23 mei 2016, om 19:50 heeft Koen Kooi <koen.kooi@linaro.org> het volgende geschreven: >> >> The commit breaks image_types_uboot.bbclass: >> >> ERROR: Logfile of failure stored in: /build/linaro/build/build/tmp-glibc/work/genericarmv8-oe-linux/linaro-image-minimal-initramfs/1.0-r0/temp/log.do_image_cpio.16506 >> Log data follows: >> | DEBUG: Executing python function set_image_size >> | DEBUG: Python function set_image_size finished >> | DEBUG: Executing shell function do_image_cpio >> | 31373 blocks >> | Image Name: linaro-image-minimal-initramfs-g >> | Created: Mon May 23 19:38:56 2016 >> | Image Type: ARM Linux RAMDisk Image (gzip compressed) >> | Data Size: 7326620 Bytes = 7154.90 kB = 6.99 MB >> | Load Address: 00000000 >> | Entry Point: 00000000 >> | mkimage: Can't open /build/linaro/build/build/tmp-glibc/deploy/images/genericarmv8/linaro-image-minimal-initramfs-genericarmv8-20160523173835.rootfs.cpio.gz: No such file or directory >> >> This reverts commit 94f61c2682e5cfd819ac84535650c3e0a654415a. >> >> Signed-off-by: Koen Kooi <koen.kooi@linaro.org> >> --- >> meta/classes/image.bbclass | 8 ++------ >> 1 file changed, 2 insertions(+), 6 deletions(-) >> >> diff --git a/meta/classes/image.bbclass b/meta/classes/image.bbclass >> index 9f4c83f..df3bd3a 100644 >> --- a/meta/classes/image.bbclass >> +++ b/meta/classes/image.bbclass >> @@ -410,13 +410,9 @@ python () { >> # Create input image first. >> gen_conversion_cmds(type) >> localdata.setVar('type', type) >> - cmd = "\t" + localdata.getVar("COMPRESS_CMD_" + ctype, True) >> - if cmd not in cmds: >> - cmds.append(cmd) >> + cmds.append("\t" + localdata.getVar("COMPRESS_CMD_" + ctype, True)) >> vardeps.add('COMPRESS_CMD_' + ctype) >> - subimage = type + "." + ctype >> - if subimage not in subimages: >> - subimages.append(subimage) >> + subimages.append(type + "." + ctype) >> if type not in alltypes: >> rm_tmp_images.add(localdata.expand("${IMAGE_NAME}${IMAGE_NAME_SUFFIX}.${type}")) >> >> -- >> 2.4.3 >> >> -- >> _______________________________________________ >> Openembedded-core mailing list >> Openembedded-core@lists.openembedded.org >> http://lists.openembedded.org/mailman/listinfo/openembedded-core >> > ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] Revert "image.bbclass: don't execute compression commands multiple times" 2016-06-04 8:41 ` Koen Kooi @ 2016-06-04 13:09 ` Richard Purdie 2016-06-06 6:16 ` Koen Kooi 2016-06-18 13:08 ` Koen Kooi 0 siblings, 2 replies; 11+ messages in thread From: Richard Purdie @ 2016-06-04 13:09 UTC (permalink / raw) To: Koen Kooi, Koen Kooi, Alexander Kanevskiy; +Cc: openembedded-core On Sat, 2016-06-04 at 10:41 +0200, Koen Kooi wrote: > ping 2 It would be helpful if you'd cc'd the person who's patch you were suggesting we revert. I added Sasha to cc. I've seen multiple different conflicting patches regarding the image types code and I've not had a chance to try and figure out what the real set of problems are and which of the various proposed patches make things worse or better. Until someone can explain clearly to me what the problems are and how we move forward, or I find time to dig into it myself, we're a bit stuck :/. I just mention this since I expect you're getting frustrated hearing nothing. This has been flagged in my inbox, I'm just lacking the time to do everything I'd like :( Cheers, Richard > > Op 30 mei 2016, om 13:28 heeft Koen Kooi < > > koen@dominion.thruhere.net> het volgende geschreven: > > > > ping > > > > > Op 23 mei 2016, om 19:50 heeft Koen Kooi <koen.kooi@linaro.org> > > > het volgende geschreven: > > > > > > The commit breaks image_types_uboot.bbclass: > > > > > > ERROR: Logfile of failure stored in: > > > /build/linaro/build/build/tmp-glibc/work/genericarmv8-oe > > > -linux/linaro-image-minimal-initramfs/1.0 > > > -r0/temp/log.do_image_cpio.16506 > > > Log data follows: > > > > DEBUG: Executing python function set_image_size > > > > DEBUG: Python function set_image_size finished > > > > DEBUG: Executing shell function do_image_cpio > > > > 31373 blocks > > > > Image Name: linaro-image-minimal-initramfs-g > > > > Created: Mon May 23 19:38:56 2016 > > > > Image Type: ARM Linux RAMDisk Image (gzip compressed) > > > > Data Size: 7326620 Bytes = 7154.90 kB = 6.99 MB > > > > Load Address: 00000000 > > > > Entry Point: 00000000 > > > > mkimage: Can't open /build/linaro/build/build/tmp > > > > -glibc/deploy/images/genericarmv8/linaro-image-minimal > > > > -initramfs-genericarmv8-20160523173835.rootfs.cpio.gz: No such > > > > file or directory > > > > > > This reverts commit 94f61c2682e5cfd819ac84535650c3e0a654415a. > > > > > > Signed-off-by: Koen Kooi <koen.kooi@linaro.org> > > > --- > > > meta/classes/image.bbclass | 8 ++------ > > > 1 file changed, 2 insertions(+), 6 deletions(-) > > > > > > diff --git a/meta/classes/image.bbclass > > > b/meta/classes/image.bbclass > > > index 9f4c83f..df3bd3a 100644 > > > --- a/meta/classes/image.bbclass > > > +++ b/meta/classes/image.bbclass > > > @@ -410,13 +410,9 @@ python () { > > > # Create input image first. > > > gen_conversion_cmds(type) > > > localdata.setVar('type', type) > > > - cmd = "\t" + > > > localdata.getVar("COMPRESS_CMD_" + ctype, True) > > > - if cmd not in cmds: > > > - cmds.append(cmd) > > > + cmds.append("\t" + > > > localdata.getVar("COMPRESS_CMD_" + ctype, True)) > > > vardeps.add('COMPRESS_CMD_' + ctype) > > > - subimage = type + "." + ctype > > > - if subimage not in subimages: > > > - subimages.append(subimage) > > > + subimages.append(type + "." + ctype) > > > if type not in alltypes: > > > > > > rm_tmp_images.add(localdata.expand("${IMAGE_NAME}${IMAGE_NAME_SU > > > FFIX}.${type}")) > > > > > > -- > > > 2.4.3 > > > > > > -- > > > _______________________________________________ > > > Openembedded-core mailing list > > > Openembedded-core@lists.openembedded.org > > > http://lists.openembedded.org/mailman/listinfo/openembedded-core > > > > > > ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] Revert "image.bbclass: don't execute compression commands multiple times" 2016-06-04 13:09 ` Richard Purdie @ 2016-06-06 6:16 ` Koen Kooi 2016-06-18 13:08 ` Koen Kooi 1 sibling, 0 replies; 11+ messages in thread From: Koen Kooi @ 2016-06-06 6:16 UTC (permalink / raw) To: Richard Purdie; +Cc: Koen Kooi, Patches and discussions about the oe-core layer On 4 June 2016 at 15:09, Richard Purdie <richard.purdie@linuxfoundation.org> wrote: > On Sat, 2016-06-04 at 10:41 +0200, Koen Kooi wrote: >> ping 2 > > It would be helpful if you'd cc'd the person who's patch you were > suggesting we revert. I added Sasha to cc. Sorry about that, I had expected 'git revert' to automatically add the original sign-off somewhere. > I've seen multiple different conflicting patches regarding the image > types code and I've not had a chance to try and figure out what the > real set of problems are Disclaimer: it's all incomprehensible python to me. The problem I'm running into is that I want a something.gz inside something-else.gz, the patch that I reverted seems to say "Hey, gz is mentioned twice, let's optimize that out". Patricks patches seem to solve the same thing but not break things. > and which of the various proposed patches make > things worse or better. > > Until someone can explain clearly to me what the problems are and how > we move forward I'm not sure what problems the compression patches are trying to fix, just that this single patches breaks an oe-core class and reverts cleanly. >, or I find time to dig into it myself, we're a bit > stuck :/. I just mention this since I expect you're getting frustrated > hearing nothing. This has been flagged in my inbox, I'm just lacking > the time to do everything I'd like :( Sure, thanks for the heads-up, a response makes the bean counters at work happy :) regards, Koen > > Cheers, > > Richard > >> > Op 30 mei 2016, om 13:28 heeft Koen Kooi < >> > koen@dominion.thruhere.net> het volgende geschreven: >> > >> > ping >> > >> > > Op 23 mei 2016, om 19:50 heeft Koen Kooi <koen.kooi@linaro.org> >> > > het volgende geschreven: >> > > >> > > The commit breaks image_types_uboot.bbclass: >> > > >> > > ERROR: Logfile of failure stored in: >> > > /build/linaro/build/build/tmp-glibc/work/genericarmv8-oe >> > > -linux/linaro-image-minimal-initramfs/1.0 >> > > -r0/temp/log.do_image_cpio.16506 >> > > Log data follows: >> > > > DEBUG: Executing python function set_image_size >> > > > DEBUG: Python function set_image_size finished >> > > > DEBUG: Executing shell function do_image_cpio >> > > > 31373 blocks >> > > > Image Name: linaro-image-minimal-initramfs-g >> > > > Created: Mon May 23 19:38:56 2016 >> > > > Image Type: ARM Linux RAMDisk Image (gzip compressed) >> > > > Data Size: 7326620 Bytes = 7154.90 kB = 6.99 MB >> > > > Load Address: 00000000 >> > > > Entry Point: 00000000 >> > > > mkimage: Can't open /build/linaro/build/build/tmp >> > > > -glibc/deploy/images/genericarmv8/linaro-image-minimal >> > > > -initramfs-genericarmv8-20160523173835.rootfs.cpio.gz: No such >> > > > file or directory >> > > >> > > This reverts commit 94f61c2682e5cfd819ac84535650c3e0a654415a. >> > > >> > > Signed-off-by: Koen Kooi <koen.kooi@linaro.org> >> > > --- >> > > meta/classes/image.bbclass | 8 ++------ >> > > 1 file changed, 2 insertions(+), 6 deletions(-) >> > > >> > > diff --git a/meta/classes/image.bbclass >> > > b/meta/classes/image.bbclass >> > > index 9f4c83f..df3bd3a 100644 >> > > --- a/meta/classes/image.bbclass >> > > +++ b/meta/classes/image.bbclass >> > > @@ -410,13 +410,9 @@ python () { >> > > # Create input image first. >> > > gen_conversion_cmds(type) >> > > localdata.setVar('type', type) >> > > - cmd = "\t" + >> > > localdata.getVar("COMPRESS_CMD_" + ctype, True) >> > > - if cmd not in cmds: >> > > - cmds.append(cmd) >> > > + cmds.append("\t" + >> > > localdata.getVar("COMPRESS_CMD_" + ctype, True)) >> > > vardeps.add('COMPRESS_CMD_' + ctype) >> > > - subimage = type + "." + ctype >> > > - if subimage not in subimages: >> > > - subimages.append(subimage) >> > > + subimages.append(type + "." + ctype) >> > > if type not in alltypes: >> > > >> > > rm_tmp_images.add(localdata.expand("${IMAGE_NAME}${IMAGE_NAME_SU >> > > FFIX}.${type}")) >> > > >> > > -- >> > > 2.4.3 >> > > >> > > -- >> > > _______________________________________________ >> > > Openembedded-core mailing list >> > > Openembedded-core@lists.openembedded.org >> > > http://lists.openembedded.org/mailman/listinfo/openembedded-core >> > > >> > >> -- Koen Kooi Builds and Baselines | Release Manager Linaro.org | Open source software for ARM SoCs ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] Revert "image.bbclass: don't execute compression commands multiple times" 2016-06-04 13:09 ` Richard Purdie 2016-06-06 6:16 ` Koen Kooi @ 2016-06-18 13:08 ` Koen Kooi 2016-06-18 18:21 ` Richard Purdie 1 sibling, 1 reply; 11+ messages in thread From: Koen Kooi @ 2016-06-18 13:08 UTC (permalink / raw) To: Richard Purdie; +Cc: Patches and discussions about the oe-core layer, Koen Kooi > Op 4 jun. 2016, om 15:09 heeft Richard Purdie <richard.purdie@linuxfoundation.org> het volgende geschreven: > > On Sat, 2016-06-04 at 10:41 +0200, Koen Kooi wrote: >> ping 2 > > It would be helpful if you'd cc'd the person who's patch you were > suggesting we revert. I added Sasha to cc. Since the original committer has shown zero interest in responding here or fixing the problem in the past month, can you please commit this? This fixes a clear regression, which alone should have been reason enough to commit this. > > I've seen multiple different conflicting patches regarding the image > types code and I've not had a chance to try and figure out what the > real set of problems are and which of the various proposed patches make > things worse or better. > > Until someone can explain clearly to me what the problems are and how > we move forward, or I find time to dig into it myself, we're a bit > stuck :/. I just mention this since I expect you're getting frustrated > hearing nothing. This has been flagged in my inbox, I'm just lacking > the time to do everything I'd like :( > > Cheers, > > Richard > >>> Op 30 mei 2016, om 13:28 heeft Koen Kooi < >>> koen@dominion.thruhere.net> het volgende geschreven: >>> >>> ping >>> >>>> Op 23 mei 2016, om 19:50 heeft Koen Kooi <koen.kooi@linaro.org> >>>> het volgende geschreven: >>>> >>>> The commit breaks image_types_uboot.bbclass: >>>> >>>> ERROR: Logfile of failure stored in: >>>> /build/linaro/build/build/tmp-glibc/work/genericarmv8-oe >>>> -linux/linaro-image-minimal-initramfs/1.0 >>>> -r0/temp/log.do_image_cpio.16506 >>>> Log data follows: >>>>> DEBUG: Executing python function set_image_size >>>>> DEBUG: Python function set_image_size finished >>>>> DEBUG: Executing shell function do_image_cpio >>>>> 31373 blocks >>>>> Image Name: linaro-image-minimal-initramfs-g >>>>> Created: Mon May 23 19:38:56 2016 >>>>> Image Type: ARM Linux RAMDisk Image (gzip compressed) >>>>> Data Size: 7326620 Bytes = 7154.90 kB = 6.99 MB >>>>> Load Address: 00000000 >>>>> Entry Point: 00000000 >>>>> mkimage: Can't open /build/linaro/build/build/tmp >>>>> -glibc/deploy/images/genericarmv8/linaro-image-minimal >>>>> -initramfs-genericarmv8-20160523173835.rootfs.cpio.gz: No such >>>>> file or directory >>>> >>>> This reverts commit 94f61c2682e5cfd819ac84535650c3e0a654415a. >>>> >>>> Signed-off-by: Koen Kooi <koen.kooi@linaro.org> >>>> --- >>>> meta/classes/image.bbclass | 8 ++------ >>>> 1 file changed, 2 insertions(+), 6 deletions(-) >>>> >>>> diff --git a/meta/classes/image.bbclass >>>> b/meta/classes/image.bbclass >>>> index 9f4c83f..df3bd3a 100644 >>>> --- a/meta/classes/image.bbclass >>>> +++ b/meta/classes/image.bbclass >>>> @@ -410,13 +410,9 @@ python () { >>>> # Create input image first. >>>> gen_conversion_cmds(type) >>>> localdata.setVar('type', type) >>>> - cmd = "\t" + >>>> localdata.getVar("COMPRESS_CMD_" + ctype, True) >>>> - if cmd not in cmds: >>>> - cmds.append(cmd) >>>> + cmds.append("\t" + >>>> localdata.getVar("COMPRESS_CMD_" + ctype, True)) >>>> vardeps.add('COMPRESS_CMD_' + ctype) >>>> - subimage = type + "." + ctype >>>> - if subimage not in subimages: >>>> - subimages.append(subimage) >>>> + subimages.append(type + "." + ctype) >>>> if type not in alltypes: >>>> >>>> rm_tmp_images.add(localdata.expand("${IMAGE_NAME}${IMAGE_NAME_SU >>>> FFIX}.${type}")) >>>> >>>> -- >>>> 2.4.3 >>>> >>>> -- >>>> _______________________________________________ >>>> Openembedded-core mailing list >>>> Openembedded-core@lists.openembedded.org >>>> http://lists.openembedded.org/mailman/listinfo/openembedded-core >>>> >>> >> > -- > _______________________________________________ > Openembedded-core mailing list > Openembedded-core@lists.openembedded.org > http://lists.openembedded.org/mailman/listinfo/openembedded-core ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] Revert "image.bbclass: don't execute compression commands multiple times" 2016-06-18 13:08 ` Koen Kooi @ 2016-06-18 18:21 ` Richard Purdie 2016-06-19 7:46 ` Koen Kooi 0 siblings, 1 reply; 11+ messages in thread From: Richard Purdie @ 2016-06-18 18:21 UTC (permalink / raw) To: Koen Kooi; +Cc: Patches and discussions about the oe-core layer, Koen Kooi On Sat, 2016-06-18 at 15:08 +0200, Koen Kooi wrote: > > Op 4 jun. 2016, om 15:09 heeft Richard Purdie < > > richard.purdie@linuxfoundation.org> het volgende geschreven: > > > > On Sat, 2016-06-04 at 10:41 +0200, Koen Kooi wrote: > > > ping 2 > > > > It would be helpful if you'd cc'd the person who's patch you were > > suggesting we revert. I added Sasha to cc. > > Since the original committer has shown zero interest in responding > here or fixing the problem in the past month, can you please commit > this? This fixes a clear regression, which alone should have been > reason enough to commit this. Did http://git.yoctoproject.org/cgit.cgi/poky/commit/?id=7028192246552f d9e4aa3674da9c30670fd4ec86 fix this? Cheers, Richard ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] Revert "image.bbclass: don't execute compression commands multiple times" 2016-06-18 18:21 ` Richard Purdie @ 2016-06-19 7:46 ` Koen Kooi 2016-06-19 7:50 ` Richard Purdie 0 siblings, 1 reply; 11+ messages in thread From: Koen Kooi @ 2016-06-19 7:46 UTC (permalink / raw) To: Richard Purdie; +Cc: Patches and discussions about the oe-core layer > Op 18 jun. 2016, om 20:21 heeft Richard Purdie <richard.purdie@linuxfoundation.org> het volgende geschreven: > > On Sat, 2016-06-18 at 15:08 +0200, Koen Kooi wrote: >>> Op 4 jun. 2016, om 15:09 heeft Richard Purdie < >>> richard.purdie@linuxfoundation.org> het volgende geschreven: >>> >>> On Sat, 2016-06-04 at 10:41 +0200, Koen Kooi wrote: >>>> ping 2 >>> >>> It would be helpful if you'd cc'd the person who's patch you were >>> suggesting we revert. I added Sasha to cc. >> >> Since the original committer has shown zero interest in responding >> here or fixing the problem in the past month, can you please commit >> this? This fixes a clear regression, which alone should have been >> reason enough to commit this. > > Did http://git.yoctoproject.org/cgit.cgi/poky/commit/?id=7028192246552f > d9e4aa3674da9c30670fd4ec86 fix this? Don’t know since I use OE, not poky. — Koen Kooi Builds and Baselines | Release Manager Linaro.org | Open source software for ARM SoCs ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] Revert "image.bbclass: don't execute compression commands multiple times" 2016-06-19 7:46 ` Koen Kooi @ 2016-06-19 7:50 ` Richard Purdie 2016-06-19 17:27 ` Koen Kooi 2016-08-15 15:38 ` Alexander Kanevskiy 0 siblings, 2 replies; 11+ messages in thread From: Richard Purdie @ 2016-06-19 7:50 UTC (permalink / raw) To: Koen Kooi; +Cc: Patches and discussions about the oe-core layer On Sun, 2016-06-19 at 09:46 +0200, Koen Kooi wrote: > > > > Op 18 jun. 2016, om 20:21 heeft Richard Purdie < > > richard.purdie@linuxfoundation.org> het volgende geschreven: > > > > On Sat, 2016-06-18 at 15:08 +0200, Koen Kooi wrote: > > > > Op 4 jun. 2016, om 15:09 heeft Richard Purdie < > > > > richard.purdie@linuxfoundation.org> het volgende geschreven: > > > > > > > > On Sat, 2016-06-04 at 10:41 +0200, Koen Kooi wrote: > > > > > ping 2 > > > > > > > > It would be helpful if you'd cc'd the person who's patch you > > > > were > > > > suggesting we revert. I added Sasha to cc. > > > > > > Since the original committer has shown zero interest in > > > responding > > > here or fixing the problem in the past month, can you please > > > commit > > > this? This fixes a clear regression, which alone should have been > > > reason enough to commit this. > > > > Did > > http://git.yoctoproject.org/cgit.cgi/poky/commit/?id=7028192246552f > > d9e4aa3674da9c30670fd4ec86 fix this? > > Don’t know since I use OE, not poky. Ok. Let me paste the hash in "From OE-Core rev: 46bc438374de74af76d288520c6252c9b7840767" into a new url. I obviously have nothing better to do with my time. Did http://git.openembedded.org/openembedded-core/commit/?id=46bc438374de74af76d288520c6252c9b7840767 fix this? Or in case you also don't have a web browser, did revision in the openembedded-core repository with hash 46bc438374de74af76d288520c6252c9b7840767 fix this issue? Cheers, Richard ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] Revert "image.bbclass: don't execute compression commands multiple times" 2016-06-19 7:50 ` Richard Purdie @ 2016-06-19 17:27 ` Koen Kooi 2016-08-15 15:38 ` Alexander Kanevskiy 1 sibling, 0 replies; 11+ messages in thread From: Koen Kooi @ 2016-06-19 17:27 UTC (permalink / raw) To: Richard Purdie; +Cc: Patches and discussions about the oe-core layer On 19 June 2016 at 09:50, Richard Purdie <richard.purdie@linuxfoundation.org> wrote: > On Sun, 2016-06-19 at 09:46 +0200, Koen Kooi wrote: >> >> >> > Op 18 jun. 2016, om 20:21 heeft Richard Purdie < >> > richard.purdie@linuxfoundation.org> het volgende geschreven: >> > >> > On Sat, 2016-06-18 at 15:08 +0200, Koen Kooi wrote: >> > > > Op 4 jun. 2016, om 15:09 heeft Richard Purdie < >> > > > richard.purdie@linuxfoundation.org> het volgende geschreven: >> > > > >> > > > On Sat, 2016-06-04 at 10:41 +0200, Koen Kooi wrote: >> > > > > ping 2 >> > > > >> > > > It would be helpful if you'd cc'd the person who's patch you >> > > > were >> > > > suggesting we revert. I added Sasha to cc. >> > > >> > > Since the original committer has shown zero interest in >> > > responding >> > > here or fixing the problem in the past month, can you please >> > > commit >> > > this? This fixes a clear regression, which alone should have been >> > > reason enough to commit this. >> > >> > Did >> > http://git.yoctoproject.org/cgit.cgi/poky/commit/?id=7028192246552f >> > d9e4aa3674da9c30670fd4ec86 fix this? >> >> Don’t know since I use OE, not poky. > > Ok. Let me paste the hash in "From OE-Core rev: 46bc438374de74af76d288520c6252c9b7840767" > into a new url. I obviously have nothing better to do with my time. > > Did http://git.openembedded.org/openembedded-core/commit/?id=46bc438374de74af76d288520c6252c9b7840767 > fix this? > > Or in case you also don't have a web browser, did revision in the > openembedded-core repository with hash > 46bc438374de74af76d288520c6252c9b7840767 fix this issue? With that included and the overlayed image.bbclass dropped the build succeeds. So the revert isn't needed anymore. -- Koen Kooi Builds and Baselines | Release Manager Linaro.org | Open source software for ARM SoCs ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] Revert "image.bbclass: don't execute compression commands multiple times" 2016-06-19 7:50 ` Richard Purdie 2016-06-19 17:27 ` Koen Kooi @ 2016-08-15 15:38 ` Alexander Kanevskiy 1 sibling, 0 replies; 11+ messages in thread From: Alexander Kanevskiy @ 2016-08-15 15:38 UTC (permalink / raw) To: Richard Purdie; +Cc: Koen Kooi, Patches and discussions about the oe-core layer [-- Attachment #1: Type: text/plain, Size: 1393 bytes --] On Sun, Jun 19, 2016 at 10:50 AM, Richard Purdie < richard.purdie@linuxfoundation.org> wrote: > On Sun, 2016-06-19 at 09:46 +0200, Koen Kooi wrote: > > > > > > > It would be helpful if you'd cc'd the person who's patch you > > > > > were > > > > > suggesting we revert. I added Sasha to cc. > > > > > > > > Since the original committer has shown zero interest in > > > > responding > > > > here or fixing the problem in the past month, can you please > > > > commit > > > > this? This fixes a clear regression, which alone should have been > > > > reason enough to commit this. Koen, Richard sorry for not noticing that discussion earlier :( > > Ok. Let me paste the hash in "From OE-Core rev: > 46bc438374de74af76d288520c6252c9b7840767" > into a new url. I obviously have nothing better to do with my time. > > Did http://git.openembedded.org/openembedded-core/commit/?id= > 46bc438374de74af76d288520c6252c9b7840767 > fix this? > > Unfortunately this patch creates another issue: chaining compressions doesn't work anymore. e.g. ext4.xz.sha256sum wouldn't work. We need something better done here. Koen, can you provide some test scenario that I can reproduce with Poky and run some tests ? I'd like to make another patch that both your usecases and our chained compression usecase would work reliable. Thanks. -- br, Alexander Kanevskiy [-- Attachment #2: Type: text/html, Size: 2459 bytes --] ^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2016-08-15 15:38 UTC | newest] Thread overview: 11+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2016-05-23 17:50 [PATCH] Revert "image.bbclass: don't execute compression commands multiple times" Koen Kooi 2016-05-30 11:28 ` Koen Kooi 2016-06-04 8:41 ` Koen Kooi 2016-06-04 13:09 ` Richard Purdie 2016-06-06 6:16 ` Koen Kooi 2016-06-18 13:08 ` Koen Kooi 2016-06-18 18:21 ` Richard Purdie 2016-06-19 7:46 ` Koen Kooi 2016-06-19 7:50 ` Richard Purdie 2016-06-19 17:27 ` Koen Kooi 2016-08-15 15:38 ` Alexander Kanevskiy
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.