From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lists.gentoo.org (pigeon.gentoo.org [208.92.234.80]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by finch.gentoo.org (Postfix) with ESMTPS id F21DE15806E for ; Thu, 1 Jun 2023 08:24:47 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id E975DE07F1; Thu, 1 Jun 2023 08:24:46 +0000 (UTC) Received: from smtp.gentoo.org (woodpecker.gentoo.org [140.211.166.183]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits)) (No client certificate requested) by pigeon.gentoo.org (Postfix) with ESMTPS id C7954E07F1 for ; Thu, 1 Jun 2023 08:24:46 +0000 (UTC) Received: from oystercatcher.gentoo.org (oystercatcher.gentoo.org [148.251.78.52]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits)) (No client certificate requested) by smtp.gentoo.org (Postfix) with ESMTPS id CAF4E341253 for ; Thu, 1 Jun 2023 08:24:45 +0000 (UTC) Received: from localhost.localdomain (localhost [IPv6:::1]) by oystercatcher.gentoo.org (Postfix) with ESMTP id 64FFBA53 for ; Thu, 1 Jun 2023 08:24:44 +0000 (UTC) From: "Viorel Munteanu" To: gentoo-commits@lists.gentoo.org Content-Transfer-Encoding: 8bit Content-type: text/plain; charset=UTF-8 Reply-To: gentoo-dev@lists.gentoo.org, "Viorel Munteanu" Message-ID: <1685607829.948452cac4bbf02003eae5279684ead0495b0e8d.ceamac@gentoo> Subject: [gentoo-commits] repo/gentoo:master commit in: sys-apps/busybox/ X-VCS-Repository: repo/gentoo X-VCS-Files: sys-apps/busybox/busybox-1.34.1-r1.ebuild sys-apps/busybox/busybox-1.34.1-r2.ebuild sys-apps/busybox/busybox-1.35.0-r1.ebuild sys-apps/busybox/busybox-1.35.0-r2.ebuild sys-apps/busybox/busybox-1.36.1.ebuild sys-apps/busybox/busybox-9999.ebuild X-VCS-Directories: sys-apps/busybox/ X-VCS-Committer: ceamac X-VCS-Committer-Name: Viorel Munteanu X-VCS-Revision: 948452cac4bbf02003eae5279684ead0495b0e8d X-VCS-Branch: master Date: Thu, 1 Jun 2023 08:24:44 +0000 (UTC) Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Linux mail X-BeenThere: gentoo-commits@lists.gentoo.org X-Auto-Response-Suppress: DR, RN, NRN, OOF, AutoReply X-Archives-Salt: 15e6da66-dd57-41f8-ac3b-eb5d025ccf27 X-Archives-Hash: 39f82ec729a07a97ece7bcc4ec74cdd8 commit: 948452cac4bbf02003eae5279684ead0495b0e8d Author: Viorel Munteanu gentoo org> AuthorDate: Thu Jun 1 06:47:41 2023 +0000 Commit: Viorel Munteanu gentoo org> CommitDate: Thu Jun 1 08:23:49 2023 +0000 URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=948452ca sys-apps/busybox: ignore cp -n error in postinstall Before coreutils-9.2, cp -n (noclobber) would return success even if skipping files. Now it returns an error. See also https://debbugs.gnu.org/cgi/bugreport.cgi?bug=62572 This is used in postinstall with the purpose of not replacing already existing utilities with links to busybox, so failing here is expected. Unfortunately we cannot distinguish this from other errors. Closes: https://bugs.gentoo.org/907432 Closes: https://github.com/gentoo/gentoo/pull/31257 Signed-off-by: Viorel Munteanu gentoo.org> sys-apps/busybox/busybox-1.34.1-r1.ebuild | 6 ++++-- sys-apps/busybox/busybox-1.34.1-r2.ebuild | 4 +++- sys-apps/busybox/busybox-1.35.0-r1.ebuild | 6 ++++-- sys-apps/busybox/busybox-1.35.0-r2.ebuild | 4 +++- sys-apps/busybox/busybox-1.36.1.ebuild | 4 +++- sys-apps/busybox/busybox-9999.ebuild | 4 +++- 6 files changed, 20 insertions(+), 8 deletions(-) diff --git a/sys-apps/busybox/busybox-1.34.1-r1.ebuild b/sys-apps/busybox/busybox-1.34.1-r1.ebuild index 130d7bd7dce9..18721b1191da 100644 --- a/sys-apps/busybox/busybox-1.34.1-r1.ebuild +++ b/sys-apps/busybox/busybox-1.34.1-r1.ebuild @@ -1,4 +1,4 @@ -# Copyright 1999-2022 Gentoo Authors +# Copyright 1999-2023 Gentoo Authors # Distributed under the terms of the GNU General Public License v2 # See `man savedconfig.eclass` for info on how to use USE=savedconfig. @@ -348,7 +348,9 @@ pkg_postinst() { cd "${T}" || die mkdir _install tar xf busybox-links.tar -C _install || die - echo n | cp -ivpPR _install/* "${ROOT}"/ || die "copying links for ${x} failed" + # 907432: cp -n returns error if it skips any file, but that is expected here + # TODO: check if a new coreutils release has a replacement option + cp -nvpPR _install/* "${ROOT}"/ fi if use sep-usr ; then diff --git a/sys-apps/busybox/busybox-1.34.1-r2.ebuild b/sys-apps/busybox/busybox-1.34.1-r2.ebuild index cf4da6210564..e316fec96042 100644 --- a/sys-apps/busybox/busybox-1.34.1-r2.ebuild +++ b/sys-apps/busybox/busybox-1.34.1-r2.ebuild @@ -355,7 +355,9 @@ pkg_postinst() { cd "${T}" || die mkdir _install tar xf busybox-links.tar -C _install || die - echo n | cp -ivpPR _install/* "${ROOT}"/ || die "copying links for ${x} failed" + # 907432: cp -n returns error if it skips any file, but that is expected here + # TODO: check if a new coreutils release has a replacement option + cp -nvpPR _install/* "${ROOT}"/ fi if use sep-usr ; then diff --git a/sys-apps/busybox/busybox-1.35.0-r1.ebuild b/sys-apps/busybox/busybox-1.35.0-r1.ebuild index d08c6a826b69..830157ef61f4 100644 --- a/sys-apps/busybox/busybox-1.35.0-r1.ebuild +++ b/sys-apps/busybox/busybox-1.35.0-r1.ebuild @@ -1,4 +1,4 @@ -# Copyright 1999-2022 Gentoo Authors +# Copyright 1999-2023 Gentoo Authors # Distributed under the terms of the GNU General Public License v2 # See `man savedconfig.eclass` for info on how to use USE=savedconfig. @@ -349,7 +349,9 @@ pkg_postinst() { cd "${T}" || die mkdir _install tar xf busybox-links.tar -C _install || die - echo n | cp -ivpPR _install/* "${ROOT}"/ || die "copying links for ${x} failed" + # 907432: cp -n returns error if it skips any file, but that is expected here + # TODO: check if a new coreutils release has a replacement option + cp -nvpPR _install/* "${ROOT}"/ fi if use sep-usr ; then diff --git a/sys-apps/busybox/busybox-1.35.0-r2.ebuild b/sys-apps/busybox/busybox-1.35.0-r2.ebuild index 711785eae12b..b120932ad31f 100644 --- a/sys-apps/busybox/busybox-1.35.0-r2.ebuild +++ b/sys-apps/busybox/busybox-1.35.0-r2.ebuild @@ -356,7 +356,9 @@ pkg_postinst() { cd "${T}" || die mkdir _install tar xf busybox-links.tar -C _install || die - echo n | cp -ivpPR _install/* "${ROOT}"/ || die "copying links for ${x} failed" + # 907432: cp -n returns error if it skips any file, but that is expected here + # TODO: check if a new coreutils release has a replacement option + cp -nvpPR _install/* "${ROOT}"/ fi if use sep-usr ; then diff --git a/sys-apps/busybox/busybox-1.36.1.ebuild b/sys-apps/busybox/busybox-1.36.1.ebuild index a9f3d85703c7..8a030a712279 100644 --- a/sys-apps/busybox/busybox-1.36.1.ebuild +++ b/sys-apps/busybox/busybox-1.36.1.ebuild @@ -346,7 +346,9 @@ pkg_postinst() { cd "${T}" || die mkdir _install tar xf busybox-links.tar -C _install || die - echo n | cp -ivpPR _install/* "${ROOT}"/ || die "copying links for ${x} failed" + # 907432: cp -n returns error if it skips any file, but that is expected here + # TODO: check if a new coreutils release has a replacement option + cp -nvpPR _install/* "${ROOT}"/ fi if use sep-usr ; then diff --git a/sys-apps/busybox/busybox-9999.ebuild b/sys-apps/busybox/busybox-9999.ebuild index a9f3d85703c7..8a030a712279 100644 --- a/sys-apps/busybox/busybox-9999.ebuild +++ b/sys-apps/busybox/busybox-9999.ebuild @@ -346,7 +346,9 @@ pkg_postinst() { cd "${T}" || die mkdir _install tar xf busybox-links.tar -C _install || die - echo n | cp -ivpPR _install/* "${ROOT}"/ || die "copying links for ${x} failed" + # 907432: cp -n returns error if it skips any file, but that is expected here + # TODO: check if a new coreutils release has a replacement option + cp -nvpPR _install/* "${ROOT}"/ fi if use sep-usr ; then