Message ID | 20200131124321.GM3319@wildebeest.org |
---|---|
State | Superseded |
Headers | show |
Series | [bug#38803] gnu: elfutils: Update to 0.178 | expand |
Context | Check | Description |
---|---|---|
cbaines/comparison | success | View comparision |
cbaines/git branch | success | View Git branch |
cbaines/applying patch | fail | View Laminar job |
Hello! Mark Wielaard <mark@klomp.org> writes: >> Also, for the description, please use full sentences. I.e. keep the >> 'This package provides a collection ...' and 'This includes ...' instead >> of 'A collection ...' and 'Includes ...'. > > OK. Done. See attached patch. Sorry for the nit-picks, but the description still starts 'mid-sentence', i.e. "A collection ..." instead of "elfutils is a collection ...". If you think it's better this way I'm fine with that, it's just that elfutils will "stand out" compared to its neighbours at e.g. <https://guix.gnu.org/packages/E/page/3/>. :-) > Yes. There is are a new network client and server integrated with > elfutils in 0.178. A new client library debuginfod-client.so which > depends on libcurl, which pulls in most of the other stuff. elfutils > libdw.so has a dependency on this, but it is dlopened when > available. So it isn't a hard dependency. In other distros > debuginfod-client is its own elfutils subpackage which is recommended, > but not required. It allows libdw.so to pull in separate debuginfo > files from the network when not locally installed (and an server URL > is configured). Then there is also a little server based on > libmicrohttpd and sqlite which is responsible for the other part of > the new inputs. Other distros put this also in a separate elfutils > subpackage. Thanks for the detailed clarification. Re: dlopen, we typically patch in the absolute file names to avoid surprises on foreign distributions (and getting deterministic results in general), so we should probably do it here too. >> Previously 'mesa' was using our other 'libelf' package, but I switched >> it to elfutils in commit 9b3b4c05a06bb8ef22350706b66043b5e93d8d66 >> because that's what "everyone else" do. Perhaps we should go back to >> that, thoughts? Then we don't have to worry as much about the size of >> elfutils. > > I would get rid of the other libelf. It has been dead upstream for > years. And last year the home page and upstream completely > disappeared. Replacing libelf with elfutils-libelf for guix globally > would make a lot of sense to me. I believe that is what most distros > do these days. Sounds good. Can you submit a patch that replaces libelf with elfutils for all packages except GCC? > I can do that. But it wouldn't change the inputs. The runtime > dependencies of libelf on its own would be reduced to just zlib and > gcclibs. The other libraries (without debuginfod-client) would just > add a couple more compression libraries as runtime dependencies > (although you realy want debuginfod-client also around so that it can > be dlopened). Same for the binaries minus debuginfod-find and > debuginfod. The debuginfod subpackage would be the only thing with > runtime dependencies on everything (including libmicrohttpd and > sqlite). I think a single :bin output with all the executables is fine. Regarding separate :libelf output, I suppose just copying libelf.so* should do the trick. Maybe we need to adjust some pkg-config files too? I can look into this if you like. > I'll go ask around here how to create a minimal package to see if that > helps. Although it feels a bit odd. Upstream doesn't really support > building just a subset of the package (there are some dependencies > between the libraries and binaries which require things to be build > together). If we are to use this libelf for 'gcc', we'll need to find a way to avoid the extraneous inputs to avoid complicating the bootstrap graph too much. We can always ignore the build system altogether and roll our own command line for the GCC variant though. Thanks, Marius
From a686a10451f600889a20b9ca8b08c25253b349f4 Mon Sep 17 00:00:00 2001 From: Mark Wielaard <mark@klomp.org> Date: Sun, 12 Jan 2020 23:54:18 +0100 Subject: [PATCH] gnu: elfutils: Update synopsis and description * gnu/packages/elf.scm (elfutils): Update summaries. [synopsis]: Updated. [description]: Updated. --- gnu/packages/elf.scm | 24 ++++++++++++++++++------ 1 file changed, 18 insertions(+), 6 deletions(-) diff --git a/gnu/packages/elf.scm b/gnu/packages/elf.scm index 75caa54296..01bf0fcf89 100644 --- a/gnu/packages/elf.scm +++ b/gnu/packages/elf.scm @@ -6,6 +6,7 @@ ;;; Copyright © 2017 Leo Famulari <leo@famulari.name> ;;; Copyright © 2018 Tobias Geerinckx-Rice <me@tobias.gr> ;;; Copyright © 2018 Marius Bakke <mbakke@fastmail.com> +;;; Copyright © 2020 Mark Wielaard <mark@klomp.org> ;;; ;;; This file is part of GNU Guix. ;;; @@ -54,9 +55,10 @@ (build-system gnu-build-system) ;; Separate programs because that's usually not what elfutils users want, - ;; and because they duplicate what Binutils provides. + ;; and because they duplicate what Binutils proAvides (but are named + ;; differently, using the eu- prefix and can be installed in parallel). (outputs '("out" ; libelf.so, elfutils/*.h, etc. - "bin")) ; ld, nm, objdump, etc. + "bin")) ; eu-nm, eu-objdump, etc. (arguments ;; Programs don't have libelf.so in their RUNPATH and libraries don't @@ -84,11 +86,21 @@ (native-inputs `(("m4" ,m4))) (inputs `(("zlib" ,zlib))) (home-page "https://sourceware.org/elfutils/") - (synopsis "Linker and ELF manipulation tools") + (synopsis "Collection of utilities and libraries to handle ELF files and +DWARF data") (description - "This package provides command-line tools to manipulate binaries in the -Executable and Linkable Format (@dfn{ELF}). This includes @command{ld}, -@command{ar}, @command{objdump}, @command{addr2line}, and more.") + "A collection of utilities and libraries to read, create and modify +Executable and Linkable Format (@dfn{ELF}) binary files, find and handle +Debugging With Arbitrary Record Formats (@dfn{DWARF}) debug data, symbols, +thread state and stacktraces for processes and core files on GNU/Linux. +Includes @file{libelf} for manipulating ELF files, @file{libdw} for inspecting +DWARF data and process state and utilities like @command{eu-stack} (to show +backtraces), @command{eu-nm} (for listing symbols from object files), +@command{eu-size} (for listing the section sizes of an object or archive +file), @command{eu-strip} (for discarding symbols), @command{eu-readelf} (to +see the raw ELF file structures), @command{eu-elflint} (to check for +well-formed ELF files), @command{eu-elfcompress} (to compress or decompress +ELF sections), and more.") ;; Libraries are dual-licensed LGPLv3.0+ | GPLv2, and programs are GPLv3+. (license lgpl3+))) -- 2.20.1