From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from gnu.wildebeest.org (wildebeest.demon.nl [212.238.236.112]) by sourceware.org (Postfix) with ESMTPS id 77C29385782E for ; Wed, 5 May 2021 22:59:32 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org 77C29385782E Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=klomp.org Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=mark@klomp.org Received: from tarox.wildebeest.org (tarox.wildebeest.org [172.31.17.39]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by gnu.wildebeest.org (Postfix) with ESMTPSA id 5DAB2302BBED; Thu, 6 May 2021 00:59:30 +0200 (CEST) Received: by tarox.wildebeest.org (Postfix, from userid 1000) id 3D3B3413CD48; Thu, 6 May 2021 00:59:30 +0200 (CEST) Message-ID: Subject: Re: Some package review feedback (bindir vs libexecdir, docs and find-debuginfo.sh naming) From: Mark Wielaard To: debugedit@sourceware.org Cc: ngompa13@gmail.com, pmatilai@redhat.com Date: Thu, 06 May 2021 00:59:30 +0200 In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Mailer: Evolution 3.28.5 (3.28.5-10.el7) Mime-Version: 1.0 X-Spam-Status: No, score=-5.2 required=5.0 tests=BAYES_00, KAM_DMARC_STATUS, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on server2.sourceware.org X-BeenThere: debugedit@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: debugedit development mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 05 May 2021 22:59:34 -0000 Hi, On Fri, 2021-04-30 at 14:33 +0200, Mark Wielaard wrote: > To fix the documentation issue I submitted patches to make sure > everything has at least a man page. I think these programs might be > used as is by normal users. Although find-debuginfo.sh needs to stop > depending on RPM_environment variables. So IMHO bindir is the more > natural place to install them. >=20 > For a 1.0 release we should make sure the documentation patches are > there. And fix the RPM_environment variables issue:=20 > https://sourceware.org/bugzilla/show_bug.cgi?id=3D27637 >=20 > Please let me know what you think of the above issues and whether there > are any other issues that you think should be resolved before we do a > 1.0 release. Meanwhile I did a 0.2 pre-release which contains the new documentation and various fixes. If you are testing distro packaging issues please try this one and report on anything that you feel should be addressed besides the above bug for the 1.0 release. Thanks, Mark $ git shortlog debugedit-0.1..debugedit-0.2=20 Dmitry V. Levin (4): debugedit: fix exit status in case of wrong number of arguments tests: fix for toolchains producing compressed debug sections debugedit: strip extra newline characters from diagnostic messages debugedit: consistently use error() instead of fprintf(stderr) Ivan A. Melnikov (1): debugedit: add MIPS support Mark Wielaard (10): tests: Check gcc accepts -gdwarf-5 otherwise skip DWARF5 tests tests: Check gcc accepts -gz=3Dnone before usage tests: Use CC, CFLAGS, LD and LDFLAGS to create testcases debugedit: Add manual using help2man sepdebugcrcfix: Add --version, --help and man page. find-debuginfo.sh: Add --help, --version and man page. automake: Add std-options to check --version and --help work as intended. Makefile.am: Add scripts/find-debuginfo.sh to EXTRA_DIST Makefile.am: Don't try to recursively make binaries to run help2man Set version to 0.2 and add upload-release.sh script. Martin Li=C5=A1ka (1): Add --dwz-single-file-mode argument for find-debuginfo.sh. Vitaly Chikunov (1): debugedit: Fix 'Unhandled relocation 0 in .debug_info section' on e2k