public inbox for isar-users@googlegroups.com
 help / color / mirror / Atom feed
From: Henning Schild <henning.schild@siemens.com>
To: Claudius Heine <ch@denx.de>,
	Alexander Smirnov <asmirnov@ilbers.de>,
	Christian Storm <christian.storm@siemens.com>,
	Jan Kiszka <jan.kiszka@siemens.com>
Cc: <isar-users@googlegroups.com>
Subject: Re: [RFC PATCH v2 0/1] Special debian depends entries
Date: Wed, 3 Jan 2018 17:08:29 +0100	[thread overview]
Message-ID: <20180103170829.538dc2d2@mmd1pvb1c.ad001.siemens.net> (raw)
In-Reply-To: <1513326642.31110.83.camel@denx.de>

Am Fri, 15 Dec 2017 09:30:42 +0100
schrieb Claudius Heine <ch@denx.de>:

> On Fri, 2017-12-15 at 09:08 +0100, Claudius Heine wrote:
> > Hi,
> > 
> > here is a new version of my patch done with python code instead of
> > shell
> > code.  
> 
> Hi,
> 
> Henning and me were currently discussing the options here, and we
> might need some referee to decide how to proceed:
> 
> So the options are backwards compatibility vs. avoiding complexity.
> 
> I think this code is not too complex, and backwards compatibility is
> nice to have. We don't know how many people use dpkg-raw and fixing it
> in a backwards compatible way saves time overall.
> 
> Henning thinks that backwards compatibility is not an issue, because
> its easy to fix in the recipes and the complexity that goes with it
> has to be avoided, because there might be bugs inside, and we have to
> fix them over and over.
> 
> Both vaild arguments IMO.

Alex, are you waiting for someone to jump in on this? Maybe you should
just call it and merge one of the two patches.

With the introduction of BASE_APT Isar might have to actually
understand/use the contents of DEBIAN_DEPENDS. And in this case it might
make sense to format it in a way that is bitbake-friendly and stick
with space-separation and the patch Claudius suggested.

Henning
 
> Cheers,
> Claudius
> 


  reply	other threads:[~2018-01-03 16:08 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-15  8:08 Claudius Heine
2017-12-15  8:08 ` [RFC PATCH v2 1/1] meta/dpkg-raw: support for special debian package dependencies Claudius Heine
2017-12-15  8:30 ` [RFC PATCH v2 0/1] Special debian depends entries Claudius Heine
2018-01-03 16:08   ` Henning Schild [this message]
2018-01-09 13:34     ` Alexander Smirnov

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=20180103170829.538dc2d2@mmd1pvb1c.ad001.siemens.net \
    --to=henning.schild@siemens.com \
    --cc=asmirnov@ilbers.de \
    --cc=ch@denx.de \
    --cc=christian.storm@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