Build is temporarily broken - my apologies

Build is temporarily broken - my apologies

[Eric S. Raymond]

Build is temporarily broken - my apologies.  Might be fixed by the time you
read this.

I seem to have wandered into a weird corner of the build behavioral space where
nothing works, but you don't get to find out it's broken until some
generated file unpredictably goes stale.

Sigh...I'm trying to isolate VC backend code.  There are no real changes yet.
-- 
		<a href="http://www.catb.org/~esr/">Eric S. Raymond</a>

Re: Build is temporarily broken - my apologies

[Eli Zaretskii]

> Date: Sat, 22 Nov 2014 05:55:46 -0500
> From: "Eric S. Raymond" <esr@thyrsus.com>
> 
> Build is temporarily broken - my apologies.  Might be fixed by the time you
> read this.

I still get suspicious warnings:

  In end of data:
  vc/vc-sccs.el:571:1:Warning: the function `vc-master-name' is not known to be
      defined.
  Wrote d:/gnu/git/emacs/trunk/lisp/vc/vc-sccs.elc

  In toplevel form:
  vc/vc-src.el:289:1:Warning: Unused lexical argument `start-revision'
  Wrote d:/gnu/git/emacs/trunk/lisp/vc/vc-src.elc
  Wrote d:/gnu/git/emacs/trunk/lisp/vc/vc.elc

  In end of data:
  vc/vc-rcs.el:1515:1:Warning: the function `vc-master-name' is not known to be
      defined.
  Wrote d:/gnu/git/emacs/trunk/lisp/vc/vc-rcs.elc

> I seem to have wandered into a weird corner of the build behavioral space where
> nothing works, but you don't get to find out it's broken until some
> generated file unpredictably goes stale.
> 
> Sigh...I'm trying to isolate VC backend code.  There are no real changes yet.

Why don't you use a local feature branch, until it is stable enough to
push upstream?

Re: Build is temporarily broken - my apologies

[Eric S. Raymond]

Eli Zaretskii <eliz@gnu.org>:
> Why don't you use a local feature branch, until it is stable enough to
> push upstream?

Habit.  I'll do that, once I climb out of this hole.
-- 
		<a href="http://www.catb.org/~esr/">Eric S. Raymond</a>

Re: Build is temporarily broken - my apologies

[martin rudalics]

 > Build is temporarily broken - my apologies.  Might be fixed by the time you
 > read this.

(Not wanting to distract you.)  To trigger the bug I can use a simple
.emacs file that looks like


(custom-set-variables
  '(inhibit-startup-echo-area-message "martin"))

(find-file-noselect "c:/temp/foo.el")


This gets me here (on Windows XP)

Error: (void-function vc-check-master-templates)

Without setting `inhibit-startup-echo-area-message' I don't get that
error.  Getting a backtrace is difficult but I can construct one if
you're interested.

martin

Re: Build is temporarily broken - my apologies

[Eric S. Raymond]

martin rudalics <rudalics@gmx.at>:
> Without setting `inhibit-startup-echo-area-message' I don't get that
> error.  Getting a backtrace is difficult but I can construct one if
> you're interested.

I figured it out.  Some stuff I thought could move to the new file
lisp/vc-filewise.el needs to stay in vc-hooks because the various backend
functions that check whether a backend is responsible for a file need
the helpers to be loaded that early.  I've added a comment that describes
the problem (in my local feature branch).

I would have spotted this sooner but for at least two stages of load
file generation that made it look like the re-partitioning was OK but
turned out to be like Coyote running on thin air - doomed as soon as he
looks down...

Trunk build seems to be currently OK.  Am beavering away on a feature
branch now.
-- 
		<a href="http://www.catb.org/~esr/">Eric S. Raymond</a>

Re: Build is temporarily broken - my apologies

[martin rudalics]

> Trunk build seems to be currently OK.  Am beavering away on a feature
> branch now.

Works now.  We should get rid of the byte-compile warnings:

vc/vc-rcs.el:1515:1:Warning: the function `vc-master-name' is not known to be
     defined.

vc/vc-sccs.el:571:1:Warning: the function `vc-master-name' is not known to be
     defined.

vc/vc-src.el:289:1:Warning: Unused lexical argument `start-revision'

Thanks, martin

Re: Build is temporarily broken - my apologies

[Eric S. Raymond]

martin rudalics <rudalics@gmx.at>:
> >Trunk build seems to be currently OK.  Am beavering away on a feature
> >branch now.
> 
> Works now.  We should get rid of the byte-compile warnings:
> 
> vc/vc-rcs.el:1515:1:Warning: the function `vc-master-name' is not known to be
>     defined.
> 
> vc/vc-sccs.el:571:1:Warning: the function `vc-master-name' is not known to be
>     defined.
> 
> vc/vc-src.el:289:1:Warning: Unused lexical argument `start-revision'

Got the first two.  I'll fix the third nerxt time I modify SRC mode.
-- 
		<a href="http://www.catb.org/~esr/">Eric S. Raymond</a>