Hi Aaron, On Fri, Apr 22, 2022 at 06:56:41PM -0400, Aaron Merey via Elfutils-devel wrote: > I've updated the patch below with the changes Mark recommended. > > A couple X-DEBUGINFOD-SIZE tests were added in another patch I recently > posted [1] that also fixes a bug when computing this header's value for > an archived file. > > Aaron > > [1] https://sourceware.org/pipermail/elfutils-devel/2022q2/004936.html > > From b56f1568b832fe1c23ffb711aa0486fbd2c5067f Mon Sep 17 00:00:00 2001 > From: Aaron Merey <ame...@redhat.com> > Date: Tue, 11 Jan 2022 22:07:55 -0500 > > debuginfod: Use the debuginfod-size response header > > In some cases the content-length header may not be available in order > to pass to a progressfn. If content-length isn't available then attempt > to get the size of the download from the debuginfod-size header instead. > > It should be mentioned that if a compressed file (ex. gzip) is being > transferred, the actual transfer length will be less than debuginfod-size. > In this case debuginfod-size is a best-guess upper bound on the size of > the transfer.
Looks good. Pleas commit. Cheers, Mark