* Small patch for toaster.bbclass [not found] <1122844108.1873572.1621697779817.ref@mail.yahoo.com> @ 2021-05-22 15:36 ` ilies bogdan 2021-05-25 13:49 ` [Toaster] " Reyna, David 0 siblings, 1 reply; 3+ messages in thread From: ilies bogdan @ 2021-05-22 15:36 UTC (permalink / raw) To: toaster [-- Attachment #1.1: Type: text/plain, Size: 678 bytes --] Hello guys, My name is Bogdan Ilies and while using toaster for a bobby project I ran into a small issue with toaster. Attached I am providing my proposed patch. If you think this is an appropriate fix could you please help me get it merged into poky or help me with write access to poky-contrib so that I can propose the patch in there? The patch is about having a special character like "\t" within the value of BBLAYERS which would lead to trying to consider a layer with name "\t" because during splitting only the value from BBLAYERS is stripped and not also the value for layer after split. Thank you very much for your time. A great weekend to you all,Bogdan Ilies [-- Attachment #1.2: Type: text/html, Size: 1878 bytes --] [-- Attachment #2: 0001-toaster-Filtering-out-erroneously-empty-bblayer-name.patch --] [-- Type: application/octet-stream, Size: 1301 bytes --] From 390685e498e3e52164472c8112c01f45c2040e42 Mon Sep 17 00:00:00 2001 From: Bogdan Ilies <ene-ilies@users.noreply.github.com> Date: Thu, 20 May 2021 17:24:00 +0200 Subject: [PATCH] toaster: Filtering out erroneously empty bblayer names. Within toaster.bbclass on toaster_layerinfo_dumpdata the str.strip is only applied to the value that comes from BBLAYERS. There are chances (also the case that I ran into by mistake) when there is an additional \t which will lead to trying to process a layer with name "\t". IMO this can be accepted and I have added an extra strip on layer name before filtering by empty layer name. Signed-off-by: Bogdan Ilies <ene-ilies@users.noreply.github.com> --- meta/classes/toaster.bbclass | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/meta/classes/toaster.bbclass b/meta/classes/toaster.bbclass index 9518ddf7a4..066cebc066 100644 --- a/meta/classes/toaster.bbclass +++ b/meta/classes/toaster.bbclass @@ -84,7 +84,8 @@ python toaster_layerinfo_dumpdata() { llayerinfo = {} - for layer in { l for l in bblayers.strip().split(" ") if len(l) }: + bblayers_list = [l for l in map(str.strip, bblayers.strip().split(" ")) if len(l)] + for layer in bblayers_list: llayerinfo[layer] = _get_layer_dict(layer) -- 2.25.1 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [Toaster] Small patch for toaster.bbclass 2021-05-22 15:36 ` Small patch for toaster.bbclass ilies bogdan @ 2021-05-25 13:49 ` Reyna, David 2021-05-25 14:20 ` ilies bogdan 0 siblings, 1 reply; 3+ messages in thread From: Reyna, David @ 2021-05-25 13:49 UTC (permalink / raw) To: ilies_06, toaster [-- Attachment #1: Type: text/plain, Size: 1247 bytes --] Hi Bogdan, Thank you for the email. I am looking at your patch and it looks good so far. I can help you with the patch submission. There is an easy process for submitting Toaster patches, and it is captured here: https://wiki.yoctoproject.org/wiki/Contribute_to_Toaster Thanks! David Reyna From: toaster@lists.yoctoproject.org <toaster@lists.yoctoproject.org> On Behalf Of ilies bogdan via lists.yoctoproject.org Sent: Saturday, May 22, 2021 8:36 AM To: toaster@lists.yoctoproject.org Subject: [Toaster] Small patch for toaster.bbclass Hello guys, My name is Bogdan Ilies and while using toaster for a bobby project I ran into a small issue with toaster. Attached I am providing my proposed patch. If you think this is an appropriate fix could you please help me get it merged into poky or help me with write access to poky-contrib so that I can propose the patch in there? The patch is about having a special character like "\t" within the value of BBLAYERS which would lead to trying to consider a layer with name "\t" because during splitting only the value from BBLAYERS is stripped and not also the value for layer after split. Thank you very much for your time. A great weekend to you all, Bogdan Ilies [-- Attachment #2: Type: text/html, Size: 5219 bytes --] ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Toaster] Small patch for toaster.bbclass 2021-05-25 13:49 ` [Toaster] " Reyna, David @ 2021-05-25 14:20 ` ilies bogdan 0 siblings, 0 replies; 3+ messages in thread From: ilies bogdan @ 2021-05-25 14:20 UTC (permalink / raw) To: Reyna, David, toaster [-- Attachment #1: Type: text/plain, Size: 2090 bytes --] Hi David, Thank you very much for your reply. I will use those instructuctions in order to create a PR as soon as I have some time. Regards,Bogdan Ilies On Tue, 25 May 2021 at 16:50, Reyna, David<david.reyna@windriver.com> wrote: #yiv3146724984 #yiv3146724984 -- _filtered {} _filtered {}#yiv3146724984 #yiv3146724984 p.yiv3146724984MsoNormal, #yiv3146724984 li.yiv3146724984MsoNormal, #yiv3146724984 div.yiv3146724984MsoNormal {margin:0in;font-size:11.0pt;font-family:sans-serif;}#yiv3146724984 a:link, #yiv3146724984 span.yiv3146724984MsoHyperlink {color:blue;text-decoration:underline;}#yiv3146724984 .yiv3146724984MsoChpDefault {font-family:sans-serif;} _filtered {}#yiv3146724984 div.yiv3146724984WordSection1 {}#yiv3146724984 Hi Bogdan, Thank you for the email. I am looking at your patch and it looks good so far. I can help you with the patch submission. There is an easy process for submitting Toaster patches, and it is captured here: https://wiki.yoctoproject.org/wiki/Contribute_to_Toaster Thanks! David Reyna From: toaster@lists.yoctoproject.org <toaster@lists.yoctoproject.org>On Behalf Of ilies bogdan via lists.yoctoproject.org Sent: Saturday, May 22, 2021 8:36 AM To: toaster@lists.yoctoproject.org Subject: [Toaster] Small patch for toaster.bbclass Hello guys, My name is Bogdan Ilies and while using toaster for a bobby project I ran into a small issue with toaster. Attached I am providing my proposed patch. If you think this is an appropriate fix could you please help me get it merged into poky or help me with write access to poky-contrib so that I can propose the patch in there? The patch is about having a special character like "\t" within the value of BBLAYERS which would lead to trying to consider a layer with name "\t" because during splitting only the value from BBLAYERS is stripped and not also the value for layer after split. Thank you very much for your time. A great weekend to you all, Bogdan Ilies [-- Attachment #2: Type: text/html, Size: 6016 bytes --] ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-05-25 14:20 UTC | newest] Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- [not found] <1122844108.1873572.1621697779817.ref@mail.yahoo.com> 2021-05-22 15:36 ` Small patch for toaster.bbclass ilies bogdan 2021-05-25 13:49 ` [Toaster] " Reyna, David 2021-05-25 14:20 ` ilies bogdan
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).