Re: Problems with EDE

[Eric M. Ludlam]

I tried this patch in my test-suite and it worked for me also.

I very recently updated the autoconf stuff, prompted by needing better
test suites, and when I read about autoreconf, I tried it but did not
know about the -i option.  It does indeed simplify things a lot.

Eric

On Fri, 2009-10-09 at 14:08 -0400, Chong Yidong wrote:
> Andreas Schwab <address@hidden> writes:
> 
> > Sascha Wilde <address@hidden> writes:
> >
> >> 1. Setup of the autotools infrastructure:
> >>
> >>     -*- mode: compilation; default-directory: "/tmp/cedettest/" -*-
> >>     Compilation started at Fri Oct  9 14:23:30
> >>
> >>     aclocal;automake --add-missing;autoreconf;
> >
> > That should be replaced by a single call to "autoreconf -i".
> >
> >>    This might be actually due to an automake bug, because IMO 
> >>    `automake --add-missing' should generate ltmain.sh -- anyway it
> >>    doesn't (tested with automake 1.10.1 on Debian Lenny).
> >
> > ltmain.sh is part of libtool, not automake.
> >
> >>    Calling libtoolize manually fixes this.
> >
> > Since autoreconf is not called with --install, it won't run libtoolize.
> 
> Sascha, could you try with this patch?
> 
> *** emacs/lisp/cedet/ede/pconf.el.~1.3.~      2009-10-03 13:42:57.000000000 
> -0400
> --- emacs/lisp/cedet/ede/pconf.el     2009-10-09 14:07:14.000000000 -0400
> ***************
> *** 99,111 ****
>          (ede-map-targets this 'ede-proj-tweak-autoconf)))
>       ;; Now save
>       (save-buffer)
> !     ;; Verify aclocal
> !     (setq postcmd "aclocal;")
> !     ;; Always add missing files as needed.
> !     (setq postcmd (concat postcmd "automake --add-missing;"))
> ! 
> !     ;; Always do autoreconf
> !     (setq postcmd (concat postcmd "autoreconf;"))
>       ;; Verify a bunch of files that are required by automake.
>       (ede-proj-configure-test-required-file this "AUTHORS")
>       (ede-proj-configure-test-required-file this "NEWS")
> --- 99,105 ----
>          (ede-map-targets this 'ede-proj-tweak-autoconf)))
>       ;; Now save
>       (save-buffer)
> !     (setq postcmd "autoreconf -i;")
>       ;; Verify a bunch of files that are required by automake.
>       (ede-proj-configure-test-required-file this "AUTHORS")
>       (ede-proj-configure-test-required-file this "NEWS")
> 
> Diff finished.  Fri Oct  9 14:08:11 2009