Re: [error-report-web][PATCH] report-error.bbclass: Add layer and bitbake version info to error report

["Milan Shah" <mshah@mvista.com>]

[-- Attachment #1: Type: text/plain, Size: 3746 bytes --]

Hi Richard,

It seems like patches sent for *error-report-web* are not much important to
the community as this patch has *not been reviewed/commented on* by
anyone *since
6th January 2021* (*more than 2.5 months*).

Still, I'm sending a reminder one last time for [*YOCTO #9700*].
https://lists.yoctoproject.org/g/yocto/message/51891

Please review/comment on it so that things can move further.

Thanks & Regards,
Milan Shah

<https://www.mvista.com/>
*Milan Shah* * | Software Engineer*
*a: * MontaVista Software, LLC | Bangalore, India
*e:* info@mvista.com | *w:* www.mvista.com/
*p:* +91-80-4939-5000


On Mon, Mar 15, 2021 at 4:36 PM Milan Shah <mshah@mvista.com> wrote:

> Hi,
>
> Review reminder for [YOCTO #9700].
>
> https://lists.yoctoproject.org/g/yocto/message/51891
>
> Thanks & Regards,
> Milan Shah
>
> <https://www.mvista.com/>
> *Milan Shah* * | Software Engineer*
> *a: * MontaVista Software, LLC | Bangalore, India
> *e:* info@mvista.com | *w:* www.mvista.com/
> *p:* +91-80-4939-5000
>
>
> On Tue, Feb 23, 2021 at 3:21 PM Milan Shah <mshah@mvista.com> wrote:
>
>> Hi All,
>>
>> This has not been reviewed yet and it is given since January 6th.
>> Please review it and provide review comments if any as soon as possible
>> to resolve this issue.
>>
>> Thanks & Regards,
>> Milan Shah
>>
>> <https://www.mvista.com/>
>> *Milan Shah* * | Software Engineer*
>> *a: * MontaVista Software, LLC | Bangalore, India
>> *e:* info@mvista.com | *w:* www.mvista.com/
>> *p:* +91-80-4939-5000
>>
>>
>> On Mon, Feb 1, 2021 at 10:06 AM Milan Shah <mshah@mvista.com> wrote:
>>
>>> Hi All,
>>>
>>> A gentle reminder to review this patch.
>>>
>>> -----------------------
>>> Thanks & Regards,
>>> *Milan Shah*
>>> MontaVista Software, Bangalore, India
>>>
>>>
>>> On Mon, Jan 11, 2021 at 6:45 PM Milan Shah <mshah@mvista.com> wrote:
>>>
>>>> Hi All,
>>>>
>>>> This is a Gentle Reminder to review this Patch.
>>>>
>>>> -----------------------
>>>> Thanks & Regards,
>>>> *Milan Shah*
>>>> MontaVista Software, Bangalore, India
>>>>
>>>>
>>>> On Wed, Jan 6, 2021 at 7:09 PM Milan Shah <mshah@mvista.com> wrote:
>>>>
>>>>> Instead of just providing local.conf info, add layer names and their
>>>>> revisions with bitbake version information into error report
>>>>> makes it easier to understand and reproduce failed build.
>>>>>
>>>>> [YOCTO #9700]
>>>>>
>>>>> Signed-off-by: Milan Shah <mshah@mvista.com>
>>>>> ---
>>>>>  meta/classes/report-error.bbclass | 4 ++++
>>>>>  1 file changed, 4 insertions(+)
>>>>>
>>>>> diff --git a/meta/classes/report-error.bbclass
>>>>> b/meta/classes/report-error.bbclass
>>>>> index 1a12db1..9cb6b0b 100644
>>>>> --- a/meta/classes/report-error.bbclass
>>>>> +++ b/meta/classes/report-error.bbclass
>>>>> @@ -6,6 +6,8 @@
>>>>>  #
>>>>>  # Licensed under the MIT license, see COPYING.MIT for details
>>>>>
>>>>> +inherit base
>>>>> +
>>>>>  ERR_REPORT_DIR ?= "${LOG_DIR}/error-report"
>>>>>
>>>>>  def errorreport_getdata(e):
>>>>> @@ -64,6 +66,8 @@ python errorreport_handler () {
>>>>>              data['failures'] = []
>>>>>              data['component'] = " ".join(e.getPkgs())
>>>>>              data['branch_commit'] = str(base_detect_branch(e.data)) +
>>>>> ": " + str(base_detect_revision(e.data))
>>>>> +            data['bitbake_version'] = e.data.getVar("BB_VERSION")
>>>>> +            data['layer_version'] = get_layers_branch_rev(e.data)
>>>>>              data['local_conf'] = get_conf_data(e, 'local.conf')
>>>>>              data['auto_conf'] = get_conf_data(e, 'auto.conf')
>>>>>              lock = bb.utils.lockfile(datafile + '.lock')
>>>>> --
>>>>> 2.7.4
>>>>>
>>>>>

[-- Attachment #2: Type: text/html, Size: 16179 bytes --]