Re: [PATCH v2 2/2] debuginfod-client.c: Fix x-debuginfod-size counts differently than CURLINFO_SIZE_DOWNLOAD_T

2023-03-30 Thread Frank Ch. Eigler via Elfutils-devel
Hi - > > Hey, great idea actually tallying up writes in the callback function. > > (We need to take care to clear that counter, in case of client object > > reuse.) Also, can you think of some reason not to just use that value > > at all times, i.e., without any of that "if and only if ..." busin

[OB PATCH] debuginfod-client.c: Avoid sscanf on mixed-case component of string

2023-03-30 Thread Aaron Merey via Elfutils-devel
Committing as obvious. sscanf is used to get the value of x-debuginfod-size from the http headers. The format string used assumes that the header field name is entirely lower case. However mixed-case field names are possible, resulting in the value not being read. Fix this by removing "x-debugi

☺ Buildbot (Sourceware): elfutils - build successful (master)

2023-03-30 Thread builder--- via Elfutils-devel
A restored build has been detected on builder elfutils-gentoo-sparc while building elfutils. Full details are available at: https://builder.sourceware.org/buildbot/#builders/225/builds/48 Build state: build successful Revision: 75800e911b73804cb3354edcc1ba9de3adaefa4e Worker: gentoo-sparc Bu

☠ Buildbot (Sourceware): elfutils - failed test (failure) (master)

2023-03-30 Thread builder--- via Elfutils-devel
A new failure has been detected on builder elfutils-debian-arm64 while building elfutils. Full details are available at: https://builder.sourceware.org/buildbot/#builders/5/builds/170 Build state: failed test (failure) Revision: 1b4cf71db1f72c53ff34c0e5c4a9da67dc24fdd6 Worker: debian-arm64 B

☺ Buildbot (Sourceware): elfutils - build successful (master)

2023-03-30 Thread builder--- via Elfutils-devel
A restored build has been detected on builder elfutils-debian-arm64 while building elfutils. Full details are available at: https://builder.sourceware.org/buildbot/#builders/5/builds/171 Build state: build successful Revision: a568e216765d41dacf8718cf2fabeec50bab16a9 Worker: debian-arm64 Bui

Re: [PATCH v2 2/2] debuginfod-client.c: Fix x-debuginfod-size counts differently than CURLINFO_SIZE_DOWNLOAD_T

2023-03-30 Thread lilydjwg via Elfutils-devel
On Thu, Mar 30, 2023 at 01:24:13PM -0400, Frank Ch. Eigler wrote: > > The written_size is actual file size (uncompressed), but IIUC > > Content-Length is the compressed size if Content-Encoding says the > > content is compressed. I haven't seen any compressed responses with > > Content-Length, but