[PATCH 0/1] Add debuginfod support to objdump and readelf

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

[PATCH 0/1] Add debuginfod support to objdump and readelf

Aaron Merey
This patch is a follow-up to a previous RFC:

https://sourceware.org/ml/binutils/2019-11/msg00340.html

Some of the concerns raised in that thread should be addressed by this
patch, including configure check issues and code reuse between binutils'
and GDB's configury. It's worth mentioning that a separate configure
option for controlling progress update output from debuginfod was
not added. Instead output is now controlled on the debuginfod-side by
an environment variable and is disabled by default.

For additional background information regarding debuginfod you can check
out the following links:

https://developers.redhat.com/blog/2019/10/14/introducing-debuginfod-the-elfutils-debuginfo-server/
https://gcc.gnu.org/wiki/cauldron2019#cauldron2019talks.The_elfutils_debuginfo_server
https://developers.redhat.com/blog/2019/12/17/deploying-debuginfod-servers-for-your-developers/

- Aaron Merey

Reply | Threaded
Open this post in threaded view
|

Re: [PATCH 0/1] Add debuginfod support to objdump and readelf

Nick Clifton
Hi Aaron,

> This patch is a follow-up to a previous RFC:

Thanks for making these updates.  I have checked the patch in.
I appreciate that there may need to be some changes made in the
future, but I wanted to get the basic support for debuginfod in
before the new branch is created.

Cheers
  Nick

Reply | Threaded
Open this post in threaded view
|

Re: [PATCH 0/1] Add debuginfod support to objdump and readelf

Aaron Merey
On Thu, Jan 9, 2020 at 8:22 AM Nick Clifton <[hidden email]> wrote:
> Thanks for making these updates.  I have checked the patch in.
> I appreciate that there may need to be some changes made in the
> future, but I wanted to get the basic support for debuginfod in
> before the new branch is created.

Thanks! I noticed that testsuite/binutils-all/debuginfod.exp was not
included in your commit though it was part of the patch I posted,
just wanted to check whether or not this was intended.

Aaron

Reply | Threaded
Open this post in threaded view
|

Re: [PATCH 0/1] Add debuginfod support to objdump and readelf

Nick Clifton
Hi Aaron,

> Thanks! I noticed that testsuite/binutils-all/debuginfod.exp was not
> included in your commit though it was part of the patch I posted,
> just wanted to check whether or not this was intended.

Ooo - thanks for spotting that.  It was a mistake on my part.
For some reason that file was being ignored when I ran "git add"
and so I missed it.  This should now be corrected.

Cheers
  Nick