From: Baurzhan Ismagulov <ibr@radix50.net>
To: isar-users@googlegroups.com
Subject: Re: [PATCH 1/2] meta: remove dead code from buildchroot script
Date: Wed, 6 May 2020 20:49:28 +0200 [thread overview]
Message-ID: <20200506184927.m454nzwzixri6w6v@yssyq.m.ilbers.de> (raw)
In-Reply-To: <20200506174044.1b4cf37f@md1za8fc.ad001.siemens.net>
On Wed, May 06, 2020 at 05:40:44PM +0200, Henning Schild wrote:
> I would conclude that those patches seem to work. We can merge them
> later with the fix, but whoever comes up with the fix might want to use
> them.
Not to mean your patches are useless, those look good to me, I'd like to apply
them in the course of the usual review process. My response was aimed at
sharing the current state of the analysis to move forward faster and avoid
duplicated effort.
> I think if we find a hen-and-egg problem we found a Debian bug and can
> lean back after reporting. But that needs further analysis and someone
> taking care of the issue and possible involving upstream.
Seems I haven't described it clearly. The hen-and-egg problem I refer to is:
Current state: Don't parse Build-Depends of e.g. libhello-dev to avoid
maintaining own parser -> use mk-build-deps to generate a dummy package that
Depends on Build-Depends of libhello-dev -> mk-build-deps in bullseye seems to
generate a source package that has Build-Depends itself. What now? We have to
install Build-Depends of libhello-cross-build-deps before we build and install
it. So I don't think this is a Debian bug, it's a matter of how we handle
installing Build-Depends of e.g. libhello-dev. Yes, we'll deal with that in a
second step, this explanation says nothing against your patches.
The solutions I was talking about:
Option 1: If libhello source package is in isar-apt, the Debian way would be to
apt-get build-dep libhello. If not, we should look at having it there.
Option 2: If the above is not possible ATM, we should look at parsing the
strings. Seems that dpkg has a Perl API. From ELBE folks I've heard that they
use python-debian (not sure whether it has this specific feature). I agree that
it should be an upstream solution. However, we might need to maintain it from
time to time.
> At the moment i will not be able to do that, i just felt i should fix
> my code that swallowed the debug output.
Got it, I see no problem with that.
With kind regards,
Baurzhan.
next prev parent reply other threads:[~2020-05-06 18:49 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-05-05 11:00 Henning Schild
2020-05-05 11:00 ` [PATCH 2/2] meta: make builddep installation verbose Henning Schild
2020-05-05 11:02 ` [PATCH 1/2] meta: remove dead code from buildchroot script Henning Schild
2020-05-06 8:57 ` Baurzhan Ismagulov
2020-05-06 15:40 ` Henning Schild
2020-05-06 15:43 ` Jan Kiszka
2020-05-06 18:49 ` Baurzhan Ismagulov [this message]
2020-05-25 13:47 ` Baurzhan Ismagulov
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=20200506184927.m454nzwzixri6w6v@yssyq.m.ilbers.de \
--to=ibr@radix50.net \
--cc=isar-users@googlegroups.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