Re: [PATCH] dpkg-raw: take unmodified DEBIAN_DEPENDS for the package

[Claudius Heine <claudius.heine.ext@siemens.com>]

Hi,

On 01/11/2018 12:42 PM, Alexander Smirnov wrote:
> On 12/14/2017 08:36 PM, Henning Schild wrote:
>> To express advanced debian dependencies version numbers in brackets and
>> other constructs are allowed. In such cases you would also have spaces
>> in DEBIAN_DEPENDS which can not just get replaced with commas. Doing
>> that will create invalid "Depends:"-lines.
>>
>> This patch just takes the string as found in the recipe. This way users
>> can express whatever they need and Isar does not need to know anything
>> about the semantics or syntax of that string.
>>
>> This change means that anyone using dpkg-raw will have to comma-separate
>> their DEBIAN_DEPENDS. The change breaks backwards compatibility but
>> simplifies things and hopefully prevents further problems in that space.
>>
> 
> Applied to next, thanks!
>

I am now trying to convert my 'DEBIAN_DEPENDS' to the new format and I 
found some issues doing it that way.

For instance if you are adding DEBAIN_DEPENDS inside a bbclass like this:

     DEBIAN_DEPENDS += " \
     	jq,			\
     	iptables,		\
     	"

This results to the following entry in control file:

     Depends:   	jq,				iptables,			

dpkg-deb does not seems to be happy with this.

That means that this patch in comparison the the one I submitted has 
also the disadvantage of enforcing a less readable format in the recipe 
file and expects special handling of the last ','.

So in the end, the complexity that we save here has to be spend 
somewhere else.

Cheers,
Claudius

P.S.: My proposed patch needs a small change to handle tab characters 
additional to spaces though.

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-54 Fax: (+49)-8142-66989-80 Email: ch@denx.de