[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#1031828: marked as done (debootstrap: Please document --usr-merge option in --help output)

Your message dated Sat, 15 Jul 2023 12:34:04 +0000
with message-id <E1qKeTM-003Fid-NM@fasolo.debian.org>
and subject line Bug#1031828: fixed in debootstrap 1.0.128+nmu3
has caused the Debian Bug report #1031828,
regarding debootstrap: Please document --usr-merge option in --help output
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
1031828: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1031828
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Package: debootstrap
Version: 1.0.128+nmu2
Severity: important
Tags: patch

Dear maintainer:

Because Debian has decided that bookworm will have
usr-merge by default even for building packages,
I would expect usr-merge to be enabled by default
in all cases, including when using the buildd profile.

Currently, such thing does not happen.

I think the attached patch would fix this
(but I've not tested it).

A better approach would be to select --usr-merge
in the buildd profile when appropriate (i.e. bookworm
and above). (How feasible would this be?)

I am aware that the proposed patch (but not the "better approach")
may force some people to do some things differently.

However, let's compare the two scenarios:

A) If the patch is applied, then once that bookworm becomes stable:

- Those who want to build packages for stable the official way
do not have to do anything special. I think this is the category
of users that should be better supported.

- Those still running bullseye who want to build packages for
bullseye (using debootstrap from bullseye) do not have to do
anything special.

- Only people who want to build packages for bullseye from
bookworm would have to add --no-usr-merge by hand, and only
when using the buildd profile. I think this is a special case
and having to add an option by hand for such special case would
not be a big deal.

B) If the patch is not applied, then once that bookworm becomes stable:

- Those who want to build packages for stable the official way
should add --usr-merge by hand when calling debootstrap.

This option is *undocumented* in "debootstrap --help" (!).

Also, failure to add the --usr-merge option may result in packages
being misbuilt, or even failing to build at all. And this would
happen when building packages from stable to be installed in a
stable system.

- Those still running bullseye who want to build packages for
bullseye (using debootstrap from bullseye) do not have to do
anything special.

- Those who want to build packages for bullseye from bookworm
would not have to do anything special.


In summary: I think A is better default than B for bookworm, and
I believe it would be ok to change debootstrap default behaviour
before the release.


In either case, if debootstrap behaviour is not going to change
for bookworm, then option --usr-merge should be much better
documented, both in --help output and maybe in Release Notes.

Thanks.
--- a/functions
+++ b/functions
@@ -1367,7 +1367,7 @@ setup_dselect_method () {
 # either installs the RTLD in a directory different from /lib or builds
 # multilib library packages.
 setup_merged_usr() {
-	if doing_variant buildd && [ -z "$MERGED_USR" ]; then
+	if [ -z "$MERGED_USR" ]; then
 		MERGED_USR="no"
 	fi

--- End Message ---
--- Begin Message --- 
Source: debootstrap
Source-Version: 1.0.128+nmu3
Done: Luca Boccassi <bluca@debian.org>

We believe that the bug you reported is fixed in the latest version of
debootstrap, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 1031828@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Luca Boccassi <bluca@debian.org> (supplier of updated debootstrap package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@ftp-master.debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

Format: 1.8
Date: Sat, 15 Jul 2023 12:56:11 +0100
Source: debootstrap
Architecture: source
Version: 1.0.128+nmu3
Distribution: unstable
Urgency: medium
Maintainer: Debian Install System Team <debian-boot@lists.debian.org>
Changed-By: Luca Boccassi <bluca@debian.org>
Closes: 963804 1031105 1031828
Changes:
 debootstrap (1.0.128+nmu3) unstable; urgency=medium
 .
   * Non-maintainer upload.
 .
   [ Akbarkhon Variskhanov ]
   * functions: Quote $USE_COMPONENTS. Thanks to Robert de Bath for
     reporting the bug and providing the fix (Closes: #963804)
 .
   [ Johannes Schauer Marin Rodrigues ]
   * functions: improve misleading error message when tar fails
   * check if http_proxy is unset instead of checking if it's empty
     (Closes: #1031105)
   * If mount -t proc proc /proc failed, fall back to bind-mounting /proc
 .
   [ Simon McVittie ]
   * Mention --merged-usr in --help output (Closes: #1031828)
   * functions: Default to merged-/usr, even for newer suites' buildds
   * d/source/lintian-overrides: Update overrides syntax
 .
   [ Luca Boccassi ]
   * Bump Standards-Version to 4.6.2, no changes
Checksums-Sha1:
 bfcc970cec8cc140e8ae18e8fd3dc24d9742fac5 2013 debootstrap_1.0.128+nmu3.dsc
 a7c6a25307aacc74ace694c0360fbd9ac8d05149 85046 debootstrap_1.0.128+nmu3.tar.gz
 bbed62c705caa2a7ffa56c25875bf212a3d99b9c 6727 debootstrap_1.0.128+nmu3_source.buildinfo
Checksums-Sha256:
 509226bca91ce164b80fac8bb96b7f474d27c2894616a2aaf470f14646f15351 2013 debootstrap_1.0.128+nmu3.dsc
 07ca04fb2d284018e064ce7910110ed28168c6b4b536b453e16eccf1dc87d9c8 85046 debootstrap_1.0.128+nmu3.tar.gz
 c9656a527d4b8dab94ed2a3e25f63aae55010702abbc55b16430b29c7cb7040a 6727 debootstrap_1.0.128+nmu3_source.buildinfo
Files:
 331afee1fe8281e68da8fa54c03d9304 2013 admin optional debootstrap_1.0.128+nmu3.dsc
 e605986d90835ec69b5b4479d1823157 85046 admin optional debootstrap_1.0.128+nmu3.tar.gz
 6017d4a5f533b7d7c6970578504607dc 6727 admin optional debootstrap_1.0.128+nmu3_source.buildinfo

-----BEGIN PGP SIGNATURE-----

iQJFBAEBCgAvFiEErCSqx93EIPGOymuRKGv37813JB4FAmSyiY0RHGJsdWNhQGRl
Ymlhbi5vcmcACgkQKGv37813JB5uPRAAp+DO7+Rot1BhkZ52gLUIupX7ZfjQk7Hp
UZhU027kAd9I3qgqRPXiAm/37p/z/Tv/VcwQ5z084QqPkEzSH4R9o0kLumcWLYun
KQLiRvs+uwLFcbqjk8AizbQg8l+rkUnQ5kJgQ2bThef+izyDiOLQHtuTdpi2V8B2
kCs57BcFmqXkFFNRVkbn/2VNVRS9cZpl8SCQcbCXQNgEn9l4R2WFxqrT58YUQKk3
l2pHADjSl90nnxOmQbqNZg7hs0IHdM6S8pGFU6V2QoRIhdVQZwIgJunnXmnIWT+N
shmDj47jBZxXEiBQyema+nI3UKRzPKFsrdEfMNMJviIlnCv2Stl71QDRdwgFl606
8kb0td65KG0/PGi5zyfGqnBLjpDb04P/eh+5/4hlVaPKsE82ySkkUMrvn6rkOIra
XbMNFF8MnVHj+IuWVVj+/ddUTTcnTQ6y1HO/zNp1iQQzHOoXvZ7I9ZWKxsVWiud2
UlS/rXKTbBgvSb7vBbpNInrDKsIdIaL/EM0ZqEMInfha7YNMruouj/Ym0nCOSKIw
77oV4wME9jJCX7VHZl6+34sj23ijQwP1Ayy945rMP9ITgeZSW/GxXNMe9I+B4Noq
K5zwYWXbbLos79FFYstcq/44abdC6l/gi638VNwYtGcBzPbB9Sc51Y+gXbomFbRA
rSts/25qfJ4=
=59th
-----END PGP SIGNATURE-----

--- End Message ---
Reply to: