Re: Bug#459049: bogofilter: FTBFS with dash: ../../../src/tests/t.grftest: 83: cannot open ../../../src/tests/inputs/msg.?.txt: No such file

On Fri, Jan 04, 2008 at 11:58:11AM +0100, Lucas Nussbaum wrote:
> During a rebuild of all packages in sid using /bin/dash as /bin/sh, your package failed to build.
> 
> Relevant part:
> 
>  > make[6]: Entering directory `/build/user/bogofilter-1.1.6/obj-db/src/tests'
>  > Aborted
>  > PASS: t.abort
>  > PASS: t.env
>  > PASS: t.ctype
>  > PASS: t.bogodir
>  > SKIP: t.leakfind
>  > PASS: t.u_fpe
>  > PASS: t.longoptions
>  > PASS: t.ignore_spam_header
>  > PASS: t.nullstatsprefix
>  > PASS: t.integrity
>  > PASS: t.integrity2
>  > PASS: t.integrity3
>  > PASS: t.passthrough-hb
>  > PASS: t.escaped.html
>  > PASS: t.escaped.url
>  > PASS: t.split
>  > PASS: t.parsing
>  > PASS: t.lexer
>  > PASS: t.lexer.mbx
>  > PASS: t.spam.header.place
>  > PASS: t.block.on.subnets
>  > PASS: t.multiple.tokens.head
>  > PASS: t.multiple.tokens.body

Re: Bug#459049: bogofilter: FTBFS with dash: ../../../src/tests/t.grftest: 83: cannot open ../../../src/tests/inputs/msg.?.txt: No such file

On Fri, Jan 04, 2008 at 10:37:01AM -0500, Clint Adams wrote:
> This is only a problem with dash 0.5.4-5, as it builds fine with
> 0.5.3-7. Attached are the respective t.grftest runs with VERBOSE=-x.

I knew I was going to forget to attach.

+ NODB=1 . ../../../src/tests/t.frame
+ set -e
+ exec
+ : ../../../src/tests
+ : ..
+ : .
+ . ./t.config
+ EXE_EXT=
+ DB_EXT=.tc
+ DB_TYPE=tokyocabinet
+ USE_TRANSACTIONS=
+ USE_UNICODE=
+ export DB_TYPE
+ DB_EXT=tc
+ export DB_EXT
+ BOGOFILTER=../bogofilter
+ BOGOLEXER=../bogolexer
+ BOGOTUNE=../bogotune
+ BOGOUTIL=../bogoutil
+ export BOGOFILTER
+ export BOGOLEXER
+ export BOGOUTIL
+ _POSIX2_VERSION=199209

Re: Bug#459049: bogofilter: FTBFS with dash: ../../../src/tests/t.grftest: 83: cannot open ../../../src/tests/inputs/msg.?.txt: No such file

On Fri, 4 Jan 2008 10:39:45 -0500
Clint Adams wrote:

> On Fri, Jan 04, 2008 at 10:37:01AM -0500, Clint Adams wrote:
> > This is only a problem with dash 0.5.4-5, as it builds fine with
> > 0.5.3-7. Attached are the respective t.grftest runs with VERBOSE=-x.
> 
> I knew I was going to forget to attach.

Clint,

The test failure involves the following line:

  for msg in "$SYSTEST/inputs/"msg.?.txt ; do

in src/tests/t.grftest.  Since dash-0.5.3-7 handles this fine and
dash-0.5.4-7 has trouble with it this seems like a globbing defect in
the new version of dash. 

Regards,

David

_______________________________________________
Bogofilter-dev mailing list
Bogofilter-dev <at> bogofilter.org
http://www.bogofilter.org/mailman/listinfo/bogofilter-dev

Re: Bug#459049: bogofilter: FTBFS with dash: ../../../src/tests/t.grftest: 83: cannot open ../../../src/tests/inputs/msg.?.txt: No such file

On Fri, Jan 04, 2008 at 08:33:24PM -0500, David Relson wrote:
> Clint,
> 
> The test failure involves the following line:
> 
>   for msg in "$SYSTEST/inputs/"msg.?.txt ; do
> 
> in src/tests/t.grftest.  Since dash-0.5.3-7 handles this fine and
> dash-0.5.4-7 has trouble with it this seems like a globbing defect in
> the new version of dash. 

I couldn't reproduce this before, but now

$ mkdir /tmp/blah
$ touch /tmp/blah/blah.1.blah /tmp/blah/blah.2.blah
$ TEMP=/tmp/blah
$ ls -l "$TEMP/"blah.?.blah
ls: /tmp/blah/blah.?.blah: No such file or directory

It's not entirely clear to me that this is incorrect;
as I understand it, the order of expansion should be
1) expand $TEMP to /tmp/blah
2) filename generation on "/tmp/blah/"blah.?.blah
3) quote removal

On the other hand, "$TEMP"/blah.?.blah works.
_______________________________________________
Bogofilter-dev mailing list
Bogofilter-dev <at> bogofilter.org
http://www.bogofilter.org/mailman/listinfo/bogofilter-dev

Re: Bug#459049: bogofilter: FTBFS with dash: ../../../src/tests/t.grftest: 83: cannot open ../../../src/tests/inputs/msg.?.txt: No such file

Hi, this is a bug in dash, and will be fixed in 0.5.4-6.  I'll reassign
and close accordingly with the dash package upload.

Thanks, Gerrit.

On Sat, Jan 05, 2008 at 01:48:46PM -0500, Clint Adams wrote:
> On Fri, Jan 04, 2008 at 08:33:24PM -0500, David Relson wrote:
> > Clint,
> > 
> > The test failure involves the following line:
> > 
> >   for msg in "$SYSTEST/inputs/"msg.?.txt ; do
> > 
> > in src/tests/t.grftest.  Since dash-0.5.3-7 handles this fine and
> > dash-0.5.4-7 has trouble with it this seems like a globbing defect in
> > the new version of dash. 
> 
> I couldn't reproduce this before, but now
> 
> $ mkdir /tmp/blah
> $ touch /tmp/blah/blah.1.blah /tmp/blah/blah.2.blah
> $ TEMP=/tmp/blah
> $ ls -l "$TEMP/"blah.?.blah
> ls: /tmp/blah/blah.?.blah: No such file or directory
> 
> It's not entirely clear to me that this is incorrect;
> as I understand it, the order of expansion should be
> 1) expand $TEMP to /tmp/blah
> 2) filename generation on "/tmp/blah/"blah.?.blah
> 3) quote removal