Re: contrasting elfutils debuginfod and clr-debug-info server

2020-02-07 Thread Mark Wielaard
Hi Arjan, On Fri, 2020-02-07 at 06:47 -0800, Arjan van de Ven wrote: > On 2/7/2020 1:13 AM, Mark Wielaard wrote: > > On Wed, Feb 05, 2020 at 02:05:42PM -0800, Arjan van de Ven wrote: > > > On 2/5/2020 11:14 AM, Frank Ch. Eigler wrote: > > > > AIUI that works by using the "/usr/lib/debug/.build- >

Re: patch rfc: debuginfod -Z (generalized archive) support

2020-02-07 Thread Frank Ch. Eigler
Hi - > My C++ is pretty terrible, but couldn't you handle that with something > like: Ah, yes, good eye, pclose and fclose are interchangeable function pointers. (close(2) along the lines of your other suggestion wouldn't be.) - FChE

Re: patch rfc: debuginfod -Z (generalized archive) support

2020-02-07 Thread Mark Wielaard
Hi Frank, On Fri, 2020-02-07 at 08:57 -0500, Frank Ch. Eigler wrote: > > Yes, lets make =CMD optional. But why assume "cat"? Can't you then > > simply use fopen instaed of popen in the code to get the FILE* to pass > > to archive_read_open_FILE? Or even just open and archive_read_open_fd? > > One

[Bug debuginfod/25502] debuginfod client should default to XDG cache

2020-02-07 Thread tromey at sourceware dot org
https://sourceware.org/bugzilla/show_bug.cgi?id=25502 --- Comment #3 from Tom Tromey --- (In reply to Mark Wielaard from comment #2) > What about the Default database file? Currently $HOME/.debuginfod.sqlite > > Should that go under $XDG_DATA_HOME/debuginfod/ ? > And then probably not as hidden

[PATCH] elflint: Check sh_info for symtab isn't too big.

2020-02-07 Thread Mark Wielaard
The sh_info field of the symtab says how many symbols are in the section. Make sure at least that many symbols fit in the section. Reported-by: Ulrich Drepper Signed-off-by: Mark Wielaard --- src/ChangeLog | 4 src/elflint.c | 4 2 files changed, 8 insertions(+) diff --git a/src/Chan

Re: contrasting elfutils debuginfod and clr-debug-info server

2020-02-07 Thread Arjan van de Ven
On 2/7/2020 1:13 AM, Mark Wielaard wrote: Hi, On Wed, Feb 05, 2020 at 02:05:42PM -0800, Arjan van de Ven wrote: On 2/5/2020 11:14 AM, Frank Ch. Eigler wrote: clear-linuxelfutils indexed by filename

Re: contrasting elfutils debuginfod and clr-debug-info server

2020-02-07 Thread Arjan van de Ven
On 2/7/2020 1:06 AM, Mark Wielaard wrote: Some contrasts in a tabular form: clear-linuxelfutils indexed by filenameby buildid client side sw fuse serverapprx. no

Re: patch rfc: debuginfod -Z (generalized archive) support

2020-02-07 Thread Frank Ch. Eigler
Hi - > Yes, lets make =CMD optional. But why assume "cat"? Can't you then > simply use fopen instaed of popen in the code to get the FILE* to pass > to archive_read_open_FILE? Or even just open and archive_read_open_fd? One reason: the defer_dtor<>-based auto-closing of these objects is different

Re: patch rfc: debuginfod -Z (generalized archive) support

2020-02-07 Thread Frank Ch. Eigler
Hi - > Yes, lets make =CMD optional. But why assume "cat"? Can't you then > simply use fopen instaed of popen in the code to get the FILE* to pass > to archive_read_open_FILE? Or even just open and archive_read_open_fd? Sounds like a plan. - FChE

Re: patch rfc: debuginfod -Z (generalized archive) support

2020-02-07 Thread Mark Wielaard
Hi Frank, On Fri, Feb 07, 2020 at 06:44:02AM -0500, Frank Ch. Eigler wrote: > > [...] I think it would be better if you could simply leave off the > > conversion CMD and just provide an EXT (without =). This would then > > just pass the file with the given extension to libarchive directly > > (the

Re: patch rfc: debuginfod -Z (generalized archive) support

2020-02-07 Thread Frank Ch. Eigler
Hi - > I really like how simple adding this feature is. Shows the design for > adding new formats was done with some foresight. Thanks! > [...] I think it would be better if you could simply leave off the > conversion CMD and just provide an EXT (without =). This would then > just pass the file

Re: patch rfc: debuginfod -Z (generalized archive) support

2020-02-07 Thread Mark Wielaard
Hi Frank, On Wed, Feb 05, 2020 at 03:09:18PM -0500, Frank Ch. Eigler wrote: > A little extension lets us process arch-linux archives. Awaiting > for some small test .pkg's from the arch folks for the elfutils > testsuite. However, hand-testing on severa larger files works! > > commit b51ae89bef

Re: contrasting elfutils debuginfod and clr-debug-info server

2020-02-07 Thread Mark Wielaard
Hi, On Wed, Feb 05, 2020 at 02:05:42PM -0800, Arjan van de Ven wrote: > On 2/5/2020 11:14 AM, Frank Ch. Eigler wrote: > > > > clear-linux > > > > elfutils > > > > > > > > indexed by filenameby > > >

Re: contrasting elfutils debuginfod and clr-debug-info server

2020-02-07 Thread Mark Wielaard
Hi, On Wed, Feb 05, 2020 at 12:32:49PM -0500, Frank Ch. Eigler wrote: > Through some comments on phoronix [0], just became (re?)aware of the > neat clear-linux debuginfo server facility [1]. It's similar to the > nixos one we have mentioned before in that it relies on a local fuse > server expose