Re: [OE-core] [docs] [PATCH] dev-manual/common-tasks.rst: correct the documentation for debuginfod

2021-04-26 Thread Alexander Kanavin
Thanks, I'll remember to send versioned patches!

Alex

On Mon, 26 Apr 2021 at 18:11, Michael Opdenacker <
michael.opdenac...@bootlin.com> wrote:

> Hi Alex,
>
> On 4/26/21 12:46 PM, Quentin Schulz wrote:
> > Hi Alex,
> >
> > Could you please add the v2/v3/vwhatever in the mail subject next time?
> > You can do so by passing `-v 2` to `git format-patch`. This helps us
> > keep track of which version of a patch should be merged (well, it helps
> > Michael :p) and on which patch series to answer/rewview.
> >
> > No need to do it this time :)
>
>
> I agree :)
>
> Thanks for the patch.
>
> Reviewed-by: Michael Opdenacker 
>
> Merged into the "master-next" branch of the yocto-docs repository
> (http://git.yoctoproject.org/cgit/cgit.cgi/yocto-docs/log/?h=master-next).
> I should send a new pull-request for inclusion in "master" by the end of
> the week.
>
> Thanks again,
>
> Michael.
>
> --
> Michael Opdenacker, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com
>
>

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#151013): 
https://lists.openembedded.org/g/openembedded-core/message/151013
Mute This Topic: https://lists.openembedded.org/mt/82374738/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [docs] [PATCH] dev-manual/common-tasks.rst: correct the documentation for debuginfod

2021-04-26 Thread Quentin Schulz
Hi Alex,

Could you please add the v2/v3/vwhatever in the mail subject next time?
You can do so by passing `-v 2` to `git format-patch`. This helps us
keep track of which version of a patch should be merged (well, it helps
Michael :p) and on which patch series to answer/rewview.

No need to do it this time :)

Thanks for the patch, LGTM!
Quentin

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#151005): 
https://lists.openembedded.org/g/openembedded-core/message/151005
Mute This Topic: https://lists.openembedded.org/mt/82374738/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [docs] [PATCH] dev-manual/common-tasks.rst: correct the documentation for debuginfod

2021-04-26 Thread Alexander Kanavin
On Mon, 26 Apr 2021 at 10:39, Quentin Schulz <
quentin.sch...@streamunlimited.com> wrote:

> > -To run a debuginfod server, you need to do the following:
> > +To run a ``debuginfod`` server, you need to do the following:
> >
> > --  Ensure that this variable is set in your ``local.conf`` file::
> > +-  Ensure that ``debuginfod`` is present in :term:`DISTRO_FEATURES`
> > +   (it already is in ``OpenEmbedded-core`` defaults and ``poky``
> reference distribution).
> > +   If not, set::
>
> I would add where this is supposed to be set (in a distro conf file or
> in local.conf if I'm not mistaken?), since it was an information that
> was there initially.
>
> For the rest:
> Reviewed-by: Quentin Schulz 
>

Thanks, corrected and resent.

Alex

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#151004): 
https://lists.openembedded.org/g/openembedded-core/message/151004
Mute This Topic: https://lists.openembedded.org/mt/82373405/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [docs] [PATCH] dev-manual/common-tasks.rst: correct the documentation for debuginfod

2021-04-26 Thread Quentin Schulz
Hi Alex,

On Fri, Apr 23, 2021 at 07:40:24PM +0200, Alexander Kanavin wrote:
> Particularly,
> - correctly describe the use of DEBUGINFOD_URLS; drop it from bitbake 
> variables
> - all necessary component tweaks are enabled by default via DISTRO_FEATURES
> - provide on-target examples of what to look for when things work properly
> 
> Signed-off-by: Alexander Kanavin 
> ---
>  documentation/dev-manual/common-tasks.rst | 40 ++-
>  documentation/ref-manual/variables.rst|  6 
>  2 files changed, 24 insertions(+), 22 deletions(-)
> 
> diff --git a/documentation/dev-manual/common-tasks.rst 
> b/documentation/dev-manual/common-tasks.rst
> index dcd3a11c3e..e3b9dcef18 100644
> --- a/documentation/dev-manual/common-tasks.rst
> +++ b/documentation/dev-manual/common-tasks.rst
> @@ -9705,39 +9705,47 @@ methods you can use: running a debuginfod server and 
> using gdbserver.
>  Using the debuginfod server method
>  
>  
> -"debuginfod" from "elfutils" is a way to distribute "debuginfo" files.
> -Running a "debuginfod" server makes debug symbols readily available,
> +``debuginfod`` from ``elfutils`` is a way to distribute ``debuginfo`` files.
> +Running a ``debuginfod`` server makes debug symbols readily available,
>  which means you don't need to download debugging information
>  and the binaries of the process being debugged. You can just fetch
>  debug symbols from the server.
>  
> -To run a debuginfod server, you need to do the following:
> +To run a ``debuginfod`` server, you need to do the following:
>  
> --  Ensure that this variable is set in your ``local.conf`` file::
> +-  Ensure that ``debuginfod`` is present in :term:`DISTRO_FEATURES`
> +   (it already is in ``OpenEmbedded-core`` defaults and ``poky`` reference 
> distribution).
> +   If not, set::

I would add where this is supposed to be set (in a distro conf file or
in local.conf if I'm not mistaken?), since it was an information that
was there initially.

For the rest:
Reviewed-by: Quentin Schulz 

Thanks,
Quentin

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#151002): 
https://lists.openembedded.org/g/openembedded-core/message/151002
Mute This Topic: https://lists.openembedded.org/mt/82373405/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [docs] [PATCH] dev-manual/common-tasks.rst: correct the documentation for debuginfod

2021-04-19 Thread Quentin Schulz
Hi Alex,

On Mon, Apr 19, 2021 at 02:56:18PM +0200, Alexander Kanavin wrote:
> Particularly,
> - correctly describe the use of DEBUGINFOD_URLS; drop it from bitbake 
> variables
> - all necessary component tweaks are enabled by default via DISTRO_FEATURES
> - provide on-target examples of what to look for when things work properly
> 

Looks good to me.

Reviewed-by: Quentin Schulz 

Thanks!
Quentin

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#150674): 
https://lists.openembedded.org/g/openembedded-core/message/150674
Mute This Topic: https://lists.openembedded.org/mt/82207950/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-