Re: [Emacs-diffs] emacs/lisp makefile.w32-in

Re: [Emacs-diffs] emacs/lisp makefile.w32-in

[Juanma Barranquero]

On Fri, Oct 2, 2009 at 14:19, Eli Zaretskii <eliz@gnu.org> wrote:

>        (WINS_BASIC): Remove cedet.
>        (WINS_CEDET): Add cedet.

But then, cedet is not added to lisp/subdirs.el, with predictable consequences:

  Checking c:/emacs/lisp/cedet/ede...
  Compiling c:/emacs/lisp/cedet/ede/cpp-root.el...

  In toplevel form:
  cedet/ede/cpp-root.el:158:1:Error: Cannot open load file: ede

BTW, a problem of the current WINS_* setup is that bootstrap-clean
does not delete the .elc files in cedet/*/*.

    Juanma

Re: [Emacs-diffs] emacs/lisp makefile.w32-in

[Eli Zaretskii]

> From: Juanma Barranquero <lekktu@gmail.com>
> Date: Fri, 2 Oct 2009 14:38:47 +0200
> Cc: Emacs developers <emacs-devel@gnu.org>
> 
> On Fri, Oct 2, 2009 at 14:19, Eli Zaretskii <eliz@gnu.org> wrote:
> 
> >        (WINS_BASIC): Remove cedet.
> >        (WINS_CEDET): Add cedet.
> 
> But then, cedet is not added to lisp/subdirs.el

Only if you are using cmd.exe as the shell.

Thanks for catching this, I think I fixed this now in CVS.

Btw, I see these errors in bootstrapping current CVS on Windows:

  cedet/srecode/table.el:31:1:Error: Required feature `srecode/loaddefs' was not provided
  cedet/semantic/analyze/complete.el:33:1:Error: Given parent class semanticdb-project-database is not a class
  cedet/ede/util.el:29:1:Error: Required feature `ede/loaddefs' was not provided

Do you see those as well?

> BTW, a problem of the current WINS_* setup is that bootstrap-clean
> does not delete the .elc files in cedet/*/*.

I will see what I can do, but first I want to solve the errors above.

Re: [Emacs-diffs] emacs/lisp makefile.w32-in

[Juanma Barranquero]

On Fri, Oct 2, 2009 at 17:51, Eli Zaretskii <eliz@gnu.org> wrote:

> Thanks for catching this, I think I fixed this now in CVS.

OK, thanks.

>  cedet/srecode/table.el:31:1:Error: Required feature `srecode/loaddefs' was not provided
>  cedet/semantic/analyze/complete.el:33:1:Error: Given parent class semanticdb-project-database is not a class
>  cedet/ede/util.el:29:1:Error: Required feature `ede/loaddefs' was not provided
>
> Do you see those as well?

After the merge, I saw the loaddefs errors while compiling (not
bootstrapping), added empty loaddefs to the required directories and
was able to compile. Afterwards I've bootstrapped OK and thought the
problem was fixed, but perhaps I was misled and the old loaddefs were
not deleted by bootstrap-clean. I'm going to do a thorough clean and
bootstrap again. I'll report back in about an hour.

    Juanma

Re: [Emacs-diffs] emacs/lisp makefile.w32-in

[Juanma Barranquero]

On Fri, Oct 2, 2009 at 17:51, Eli Zaretskii <eliz@gnu.org> wrote:

> Btw, I see these errors in bootstrapping current CVS on Windows:
>
>  cedet/srecode/table.el:31:1:Error: Required feature `srecode/loaddefs' was not provided
>  cedet/semantic/analyze/complete.el:33:1:Error: Given parent class semanticdb-project-database is not a class
>  cedet/ede/util.el:29:1:Error: Required feature `ede/loaddefs' was not provided
>
> Do you see those as well?

No. Bootstrap from a clean state went like a charm.

    Juanma

Re: [Emacs-diffs] emacs/lisp makefile.w32-in

[Eli Zaretskii]

> From: Juanma Barranquero <lekktu@gmail.com>
> Date: Fri, 2 Oct 2009 19:01:10 +0200
> Cc: emacs-devel@gnu.org
> 
> On Fri, Oct 2, 2009 at 17:51, Eli Zaretskii <eliz@gnu.org> wrote:
> 
> > Btw, I see these errors in bootstrapping current CVS on Windows:
> >
> >  cedet/srecode/table.el:31:1:Error: Required feature `srecode/loaddefs' was not provided
> >  cedet/semantic/analyze/complete.el:33:1:Error: Given parent class semanticdb-project-database is not a class
> >  cedet/ede/util.el:29:1:Error: Required feature `ede/loaddefs' was not provided
> >
> > Do you see those as well?
> 
> No. Bootstrap from a clean state went like a charm.

Now that I re-created loaddefs.el and cedet/*/loaddefs.el, it works
for me as well.  Not sure what went wrong before.

Re: [Emacs-diffs] emacs/lisp makefile.w32-in

[Andreas Schwab]

Eli Zaretskii <eliz@gnu.org> writes:

> Now that I re-created loaddefs.el and cedet/*/loaddefs.el, it works
> for me as well.  Not sure what went wrong before.

autoload-rubric has changed, but your bootstrap-emacs was still using
the old definition.

Andreas.

-- 
Andreas Schwab, schwab@linux-m68k.org
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."

Re: [Emacs-diffs] emacs/lisp makefile.w32-in

[Eli Zaretskii]

> From: Juanma Barranquero <lekktu@gmail.com>
> Date: Fri, 2 Oct 2009 14:38:47 +0200
> Cc: Emacs developers <emacs-devel@gnu.org>
> 
> BTW, a problem of the current WINS_* setup is that bootstrap-clean
> does not delete the .elc files in cedet/*/*.

I think I fixed this now.

Re: [Emacs-diffs] emacs/lisp makefile.w32-in

[Juanma Barranquero]

On Fri, Oct 2, 2009 at 20:47, Eli Zaretskii <eliz@gnu.org> wrote:

> Log message:
>        (WINS_BASIC): Add eieio.
>
[...]
>  WINS_BASIC=\
>        calc \
>        calendar \
> +       eieio \
>        emacs-lisp \
>        emulation \
>        erc \

I see no lisp/eieio/ on my checkout. eieio-* files are in lisp/emacs-lisp/

    Juanma

Re: [Emacs-diffs] emacs/lisp makefile.w32-in

[Eli Zaretskii]

> From: Juanma Barranquero <lekktu@gmail.com>
> Date: Fri, 2 Oct 2009 21:17:57 +0200
> Cc: Emacs developers <emacs-devel@gnu.org>
> 
> On Fri, Oct 2, 2009 at 20:47, Eli Zaretskii <eliz@gnu.org> wrote:
> 
> > Log message:
> >        (WINS_BASIC): Add eieio.
> >
> [...]
> >  WINS_BASIC=\
> >        calc \
> >        calendar \
> > +       eieio \
> >        emacs-lisp \
> >        emulation \
> >        erc \
> 
> I see no lisp/eieio/ on my checkout. eieio-* files are in lisp/emacs-lisp/

Fixed.