public inbox for isar-users@googlegroups.com
 help / color / mirror / Atom feed
From: Felix Moessbauer <felix.moessbauer@siemens.com>
To: <isar-users@googlegroups.com>
Cc: <jan.kiszka@siemens.com>,
	Felix Moessbauer <felix.moessbauer@siemens.com>
Subject: [PATCH 1/1] Revert "buildchroot: Do not build cross when there are no arch-specific outputs"
Date: Wed, 22 Dec 2021 13:55:16 +0100	[thread overview]
Message-ID: <20211222125516.1419379-1-felix.moessbauer@siemens.com> (raw)

This reverts commit 563986703d9a0405c70af9b50ecedede2ac59cbd.

The reverted patch made a shortcut to simplify the handling of cross-build
dependencies of architecture:all packages.
This is not valid for the following reasons:

1. Just scanning the control file for 'all' is not sufficient as
a source package might generate both arch specific and :all packages.

2. This breaks architecture specific transitive build dependencies.
These mainly apply to correctly packaged python packages that support
cross building and use setuptools. By that, the transitive dependency
libpython3.9-minimal:<target-arch> is not attracted but instead the version
for the host, leading to very hard to debug build-time issues.

In case packages do not handle their cross-build dependencies correctly, we
should not try to work around in ISAR, but better enforce upstream patches.

Signed-off-by: Felix Moessbauer <felix.moessbauer@siemens.com>
---
 meta/recipes-devtools/buildchroot/files/deps.sh | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/meta/recipes-devtools/buildchroot/files/deps.sh b/meta/recipes-devtools/buildchroot/files/deps.sh
index 1d617bc8..ccfc460c 100644
--- a/meta/recipes-devtools/buildchroot/files/deps.sh
+++ b/meta/recipes-devtools/buildchroot/files/deps.sh
@@ -27,12 +27,6 @@ if [ "$3" != "--download-only" ]; then
         -o APT::Get::List-Cleanup="0"
 fi
 
-# Do not set an architecture when building only 'all' (generic) packages.
-# This can avoid unneeded cross-build issues.
-if ! grep "^Architecture:" debian/control | grep -qv "all"; then
-    set_arch=""
-fi
-
 control_file=$(pwd)/debian/control
 cd ..
 
-- 
2.30.2


             reply	other threads:[~2021-12-22 12:55 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-12-22 12:55 Felix Moessbauer [this message]
2022-01-03  7:31 ` Jan Kiszka
2022-01-04 10:15   ` Moessbauer, Felix
2022-01-04 12:24     ` Jan Kiszka
2022-01-10  7:20       ` Moessbauer, Felix
2022-01-10  7:38         ` Jan Kiszka
2022-01-10  7:50 ` Anton Mikanovich

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20211222125516.1419379-1-felix.moessbauer@siemens.com \
    --to=felix.moessbauer@siemens.com \
    --cc=isar-users@googlegroups.com \
    --cc=jan.kiszka@siemens.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox