Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-11-24 Thread Andreas Tille
On Thu, Nov 24, 2016 at 08:39:54PM +0100, Petter Reinholdtsen wrote: > [Andreas Tille] > > I just want to hear confirmation from some Debian Edu developer > > whether the upload of blends-dev 0.6.95 is OK. > > As far as I can tell, these changes are intended and reflect changes > done to the sourc

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-11-24 Thread Petter Reinholdtsen
[Andreas Tille] > I just want to hear confirmation from some Debian Edu developer > whether the upload of blends-dev 0.6.95 is OK. As far as I can tell, these changes are intended and reflect changes done to the source code. But I have not tracked the Debian Edu development closely for a while.

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-11-24 Thread Andreas Tille
Hi Ole, sorry for my longer than planned silence but today I found the time to repeat all tests with your new version. On Tue, Nov 15, 2016 at 10:12:00AM +0100, Ole Streicher wrote: > Hi Andreas, Petter and all, > > > On 15.11.2016 07:09, Andreas Tille wrote: > >> Use of uninitialized value $_

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-11-15 Thread Petter Reinholdtsen
[Ole] > Great! Could I in return ask you for the favour looking into the > patched blend-gen-control to see why the lines with a backslash are > not eaten up, so that one has to remove them later again? > > (see one of my previous mails) I will try to find time have a look. Could you prepare and

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-11-15 Thread Ole Streicher
Hi Andreas, Petter and all, On 15.11.2016 07:09, Andreas Tille wrote: > Hi, > > I just want to announce that I'll be de-facto offline today and > tomorrow. So I can not do further testing of the "Use of uninitialized > value" testing. > > Kind regards > > Andreas. > > On Fri, Nov 11, 2016

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-11-14 Thread Andreas Tille
Hi, I just want to announce that I'll be de-facto offline today and tomorrow. So I can not do further testing of the "Use of uninitialized value" testing. Kind regards Andreas. On Fri, Nov 11, 2016 at 12:52:46PM +0100, Andreas Tille wrote: > Hi, > > On Fri, Nov 11, 2016 at 11:50:12AM +0

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-11-11 Thread debian
Hi Petter, On 11.11.2016 11:50, Petter Reinholdtsen wrote: > [Ole Streicher] >> OK, I committed another version, with backslashes everywhere. Adjusted >> tool is attached. > Very good. This one look like it should still work, and fixes a few > mistakes in the process. Thank you very much. > >>

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-11-11 Thread Andreas Tille
Hi, On Fri, Nov 11, 2016 at 11:50:12AM +0100, Petter Reinholdtsen wrote: > [Ole Streicher] > > OK, I committed another version, with backslashes everywhere. Adjusted > > tool is attached. > > Very good. This one look like it should still work, and fixes a few > mistakes in the process. Thank yo

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-11-11 Thread Petter Reinholdtsen
[Ole Streicher] > OK, I committed another version, with backslashes everywhere. Adjusted > tool is attached. Very good. This one look like it should still work, and fixes a few mistakes in the process. Thank you very much. > I tried as much as possible. However, trailing spaces are still > remo

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-11-11 Thread debian
Hi Petter, On 11.11.2016 11:03, Petter Reinholdtsen wrote: > [Ole Streicher] >> I updated all tasks of debian-edu to be RFC834 compliant: continuation >> with indentation, no backslashes, no duplicated keywords. The diff looks >> OK for me; you should however double check. >> >> If you don't like

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-11-11 Thread Petter Reinholdtsen
[Ole Streicher] > I updated all tasks of debian-edu to be RFC834 compliant: continuation > with indentation, no backslashes, no duplicated keywords. The diff looks > OK for me; you should however double check. > > If you don't like it, revert and discuss it. This is not going to fly. The Debian E

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-11-11 Thread debian
Hi Andreas, Petter, and all I updated all tasks of debian-edu to be RFC834 compliant: continuation with indentation, no backslashes, no duplicated keywords. The diff looks OK for me; you should however double check. If you don't like it, revert and discuss it. I wrote a small Python script for t

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-11-11 Thread Ole Streicher
Hi all, On 11.11.2016 08:07, Andreas Tille wrote: > On Thu, Nov 10, 2016 at 10:38:32PM +0100, Ole Streicher wrote: >> --> debian-edu tasks are just broken. They don't follow any rule, and >> depending from the parser one will get always different results. Maybe >> we should fix that? remove all ba

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-11-10 Thread Andreas Tille
Hi, On Thu, Nov 10, 2016 at 10:38:32PM +0100, Ole Streicher wrote: > On 10.11.2016 21:07, Andreas Tille wrote: > > So I confirm that the first problem we detected is solved but there is > > another one breaking Debian Edu. I have again no suspicion why the '\' > > sign is not elimiunated from the

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-11-10 Thread Ole Streicher
Hi Andreas, Petter and all, On 10.11.2016 21:07, Andreas Tille wrote: > So I confirm that the first problem we detected is solved but there is > another one breaking Debian Edu. I have again no suspicion why the '\' > sign is not elimiunated from the list only in those few cases. I can also just

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-11-10 Thread Andreas Tille
Hi, On Thu, Nov 10, 2016 at 01:53:20PM +0100, Ole Streicher wrote: > > > > gbp clone ssh://git.debian.org/git/blends/projects/med.git > > cd med > > make dist > > grep ^, debian/control Debian Med and Debian Science are OK with the patch, however: gbp clone ssh://git.debian.or

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-11-10 Thread Andreas Tille
On Thu, Nov 10, 2016 at 01:53:20PM +0100, Ole Streicher wrote: > > This is what I meant, and it is fixed by my last commit. Please try > again ;-) Quick note from bad connection. Seems to work now. I'll try later more carefully. Thanks for your contribution Andreas. -- http://fam-till

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-11-10 Thread Ole Streicher
Hi Andreas, On 10.11.2016 13:48, Andreas Tille wrote: > Hi Petter, > > On Thu, Nov 10, 2016 at 12:39:07PM +0100, Petter Reinholdtsen wrote: >> Control: tags -1 + pending >> >> [Andreas Tille] Should I commit it? >>> Yes please. Ole, you reported problems with your patch. Could you >>> pleas

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-11-10 Thread Andreas Tille
Hi Petter, On Thu, Nov 10, 2016 at 12:39:07PM +0100, Petter Reinholdtsen wrote: > Control: tags -1 + pending > > [Andreas Tille] > >> Should I commit it? > > > > Yes please. Ole, you reported problems with your patch. Could you > > please be more verbose about this problem - at best based on Pe

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-11-10 Thread Petter Reinholdtsen
Control: tags -1 + pending [Andreas Tille] >> Should I commit it? > > Yes please. Ole, you reported problems with your patch. Could you > please be more verbose about this problem - at best based on Petter's > commit to make sure we are all working on the same code? It is now commited. Please

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-11-10 Thread Andreas Tille
Hi Petter, On Thu, Nov 10, 2016 at 11:07:47AM +0100, Petter Reinholdtsen wrote: > > I have not followed this issue closely, but started reading on Andreas > request. Thanks a lot for your quick response. > First of all, I believe this issue is based on incorrect documentation. Yes. > The tas

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-11-10 Thread Petter Reinholdtsen
I have not followed this issue closely, but started reading on Andreas request. First of all, I believe this issue is based on incorrect documentation. The task files have never been RFC822 compliant. I agree that they should be RFC822 compliant, but the only handled continuation method implemen

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-11-10 Thread Andreas Tille
Hi Ole, On Thu, Nov 10, 2016 at 08:56:50AM +0100, Ole Streicher wrote: > > If one of our main tools is not compliant with our documented > specifications, and this may cause incomplete metapackages (which are > one central part of the blend), then I would still rate this as an RC > bug, independe

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-11-10 Thread Andreas Tille
Hi Ole, On Thu, Nov 10, 2016 at 09:06:52AM +0100, Ole Streicher wrote: > > > >> it's indeed to late in the stretch dev cycle in my opinion. > > > > That would mean to lower the severity of #840094. Ole, are you OK with > > this? > > No, I am not. The tool gives wrong results, and it does so si

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-11-10 Thread Ole Streicher
Hi Andreas and Bas, On 10.11.2016 08:45, Andreas Tille wrote: > On Wed, Nov 09, 2016 at 06:27:13PM +0100, Sebastiaan Couwenberg wrote: >> On 11/09/2016 03:35 PM, Andreas Tille wrote: >>> If you (and Bas and other readers here) think we should fix the issue >>> right now I'm fine if you apply the p

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-11-10 Thread Ole Streicher
Hi Andreas and Bas, On 10.11.2016 08:45, Andreas Tille wrote: > On Wed, Nov 09, 2016 at 06:27:13PM +0100, Sebastiaan Couwenberg wrote: >> On 11/09/2016 03:35 PM, Andreas Tille wrote: >>> If you (and Bas and other readers here) think we should fix the issue >>> right now I'm fine if you apply the p

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-11-09 Thread Ole Streicher
Hi Andreas & all, On 09.11.2016 15:35, Andreas Tille wrote: >> We have a clear definition of how these files should look like, namely >> RFC822, and this also defines continuation lines. > > Unfortunately in this specific feature tasks files are not RFC822 > compliant, which sucks, yes. Its even

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-11-09 Thread Andreas Tille
Hi Bas, On Wed, Nov 09, 2016 at 06:27:13PM +0100, Sebastiaan Couwenberg wrote: > On 11/09/2016 03:35 PM, Andreas Tille wrote: > > If you (and Bas and other readers here) think we should fix the issue > > right now I'm fine if you apply the patch below and we should seriously > > test the metapacka

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-11-09 Thread Andreas Tille
On Wed, Nov 09, 2016 at 01:42:37PM +0100, Ole Streicher wrote: > On 09.11.2016 12:47, Andreas Tille wrote: > > In other words: Once it was defined as syntax for these control files > > that newlines need to be escaped. I do not like it and as I said this > > is fixed in the long-term pending rewri

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-11-09 Thread Ole Streicher
Hi Andreas, On 09.11.2016 12:47, Andreas Tille wrote: > In other words: Once it was defined as syntax for these control files > that newlines need to be escaped. I do not like it and as I said this > is fixed in the long-term pending rewrite. However, the bug is not > serious but at best wishli

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-11-09 Thread Andreas Tille
Hi Ole, On Sat, Oct 08, 2016 at 11:33:27AM +0200, Sebastiaan Couwenberg wrote: > On 10/08/2016 11:15 AM, Ole Streicher wrote: > > When a "Depends:" field in a task file contains more than one line, only > > the first line is used to create the dependency in debian/control. All > > others are silen

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-10-08 Thread Andreas Tille
On Sat, Oct 08, 2016 at 11:33:27AM +0200, Sebastiaan Couwenberg wrote: > On 10/08/2016 11:15 AM, Ole Streicher wrote: > > When a "Depends:" field in a task file contains more than one line, only > > the first line is used to create the dependency in debian/control. All > > others are silently ignor

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-10-08 Thread Sebastiaan Couwenberg
On 10/08/2016 11:15 AM, Ole Streicher wrote: > When a "Depends:" field in a task file contains more than one line, only > the first line is used to create the dependency in debian/control. All > others are silently ignored. You can have dependencies on multiple lines when you escape the newline. S

Bug#840094: blends-dev: Does not recognize multiline dependencies

2016-10-08 Thread Ole Streicher
Package: blends-dev Version: 0.6.94 Severity: serious When a "Depends:" field in a task file contains more than one line, only the first line is used to create the dependency in debian/control. All others are silently ignored. I observed this on the "debian-astro" package which uses blends-dev.