Re: FYI: `make distcheck' non-interactive again

2001-03-09 Thread Pavel Roskin
Hello! > ChangeLog: > * Makefile.am (maintainer-check): Scan all *.am files and tests > for invocations of `rm' without `-f'. > * tests/mclean.test: Adjusted to prevent being caught by the > above test. I've checked it in. Regards, Pavel Roskin

Re: multiple AM_CONDITIONALs doens't work in CVS Automake

2001-03-09 Thread Akim Demaille
"Lars J. Aas" <[EMAIL PROTECTED]> writes: > On Fri, Mar 09, 2001 at 05:26:29PM +0100, Akim Demaille wrote: > : Hm, in fact, could you try again? I applied one of the patches which > : were OK'ed, which is addressing issues in this area. Maybe it cures > : your problem. I couldn't build a test

Re: multiple AM_CONDITIONALs doens't work in CVS Automake

2001-03-09 Thread Lars J. Aas
On Fri, Mar 09, 2001 at 01:53:17PM +0100, Lars J. Aas wrote: : When I bootstrap with the current Automake CVS sources, the strudels (@s) : between the conditionals are missing: : : > :@SIMAGE_JPEG_SUPPORT_FALSESIMAGE_PIC_SUPPORT_FALSESIMAGE_PNG_SUPPORT_FALSESIMAGE_RGB_SUPPORT_FALSESIMAGE_TGA_SUP

Re: multiple AM_CONDITIONALs doens't work in CVS Automake

2001-03-09 Thread Lars J. Aas
On Fri, Mar 09, 2001 at 05:26:29PM +0100, Akim Demaille wrote: : Hm, in fact, could you try again? I applied one of the patches which : were OK'ed, which is addressing issues in this area. Maybe it cures : your problem. I couldn't build a test case reproducing your problem :( It still doesn't

Re: 16-rule-define.patch

2001-03-09 Thread Akim Demaille
Tom Tromey <[EMAIL PROTECTED]> writes: > Akim> * automake.in (&rule_define): Extract from... > Akim> (&read_am_file): here. > > I'd prefer a name to parallel whatever happens with "variable_define". > > If these are just subroutines of read_am_file then one approach is to > say

Re: 14-variable-define.patch

2001-03-09 Thread Akim Demaille
Tom Tromey <[EMAIL PROTECTED]> writes: > Akim> * automake.in (&variable_define): Extract from... > Akim> (&read_am_file): here. > > I don't like this. With this patch we have subroutines named > define_variable and variable_define. I understand. > How about a longer, more desc

Re: multiple AM_CONDITIONALs doens't work in CVS Automake

2001-03-09 Thread Akim Demaille
Hm, in fact, could you try again? I applied one of the patches which were OK'ed, which is addressing issues in this area. Maybe it cures your problem. I couldn't build a test case reproducing your problem :(

Re: multiple AM_CONDITIONALs doens't work in CVS Automake

2001-03-09 Thread Lars J. Aas
On Fri, Mar 09, 2001 at 04:40:35PM +0100, Akim Demaille wrote: : sub make_condition : { : print STDERR "MAKE0: " . scalar(@_) . "\n"; I added this line too. : my $res = join ('@@', @_); : print STDERR "MAKE1: $res\n"; : return '' : unless $res; : : $res = '@' . $res .

Re: multiple AM_CONDITIONALs doens't work in CVS Automake

2001-03-09 Thread Akim Demaille
"Lars J. Aas" <[EMAIL PROTECTED]> writes: > When I bootstrap with the current Automake CVS sources, the strudels (@s) > between the conditionals are missing: Hi Lars! > > >@SIMAGE_JPEG_SUPPORT_FALSESIMAGE_PIC_SUPPORT_FALSESIMAGE_PNG_SUPPORT_FALSESIMAGE_RGB_SUPPORT_FALSESIMAGE_TGA_SUPPORT_FALSE

Re: 13-single-var-storage.patch

2001-03-09 Thread Akim Demaille
Tom Tromey <[EMAIL PROTECTED]> writes: > Akim> Patch is several chunks because it took me several steps to > Akim> reach the right one. > > That makes it a lot harder to read. Please don't do that. I didn't mean it :( It really that I discovered side effects later on. I should have built a i

multiple AM_CONDITIONALs doens't work in CVS Automake

2001-03-09 Thread Lars J. Aas
When I bootstrap with the current Automake CVS sources, the strudels (@s) between the conditionals are missing: > >@SIMAGE_JPEG_SUPPORT_FALSESIMAGE_PIC_SUPPORT_FALSESIMAGE_PNG_SUPPORT_FALSESIMAGE_RGB_SUPPORT_FALSESIMAGE_TGA_SUPPORT_FALSESIMAGE_TIFF_SUPPORT_FALSESIMAGE_UNGIF_SUPPORT_TRUE@am_libsi