unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* CVS HEAD fails to build
@ 2007-05-07 18:34 Randal L. Schwartz
  2007-05-07 18:57 ` Giorgos Keramidas
  2007-05-07 21:41 ` Stefan Monnier
  0 siblings, 2 replies; 13+ messages in thread
From: Randal L. Schwartz @ 2007-05-07 18:34 UTC (permalink / raw)
  To: emacs-devel


% make clean bootstrap install
...
Compiling /Users/merlyn/MIRROR/emacs-CVS/lisp/./eshell/em-term.el
Loading /Users/merlyn/MIRROR/emacs-CVS/lisp/net/ange-ftp.el (source)...

In toplevel form:
eshell/em-term.el:48:1:Error: Wrong type argument: keymapp, nil
make[2]: *** [compile] Error 1
make[1]: *** [bootstrap-build] Error 2
make: *** [bootstrap] Error 2
make clean bootstrap install  168.80s user 32.79s system 67% cpu 4:59.70 total

-- 
Randal L. Schwartz - Stonehenge Consulting Services, Inc. - +1 503 777 0095
<merlyn@stonehenge.com> <URL:http://www.stonehenge.com/merlyn/>
Perl/Unix/security consulting, Technical writing, Comedy, etc. etc.
See PerlTraining.Stonehenge.com for onsite and open-enrollment Perl training!

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: CVS HEAD fails to build
  2007-05-07 18:34 CVS HEAD fails to build Randal L. Schwartz
@ 2007-05-07 18:57 ` Giorgos Keramidas
  2007-05-07 19:50   ` Giorgos Keramidas
  2007-05-07 21:41 ` Stefan Monnier
  1 sibling, 1 reply; 13+ messages in thread
From: Giorgos Keramidas @ 2007-05-07 18:57 UTC (permalink / raw)
  To: Randal L. Schwartz; +Cc: emacs-devel

On 2007-05-07 11:34, "Randal L. Schwartz" <merlyn@stonehenge.com> wrote:
>
> % make clean bootstrap install
> ...
> Compiling /Users/merlyn/MIRROR/emacs-CVS/lisp/./eshell/em-term.el
> Loading /Users/merlyn/MIRROR/emacs-CVS/lisp/net/ange-ftp.el (source)...
>
> In toplevel form:
> eshell/em-term.el:48:1:Error: Wrong type argument: keymapp, nil
> make[2]: *** [compile] Error 1
> make[1]: *** [bootstrap-build] Error 2
> make: *** [bootstrap] Error 2
> make clean bootstrap install  168.80s user 32.79s system 67% cpu 4:59.70 total

Same here.  I'm bisecting through commits to find the first change which
caused this.  Hold on for half an hour or so and I will have more details.

- Giorgos

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: CVS HEAD fails to build
  2007-05-07 18:57 ` Giorgos Keramidas
@ 2007-05-07 19:50   ` Giorgos Keramidas
  2007-05-07 20:51     ` Andreas Schwab
  2007-05-07 20:51     ` Giorgos Keramidas
  0 siblings, 2 replies; 13+ messages in thread
From: Giorgos Keramidas @ 2007-05-07 19:50 UTC (permalink / raw)
  To: Randal L. Schwartz; +Cc: emacs-devel

On 2007-05-07 21:57, Giorgos Keramidas <keramida@ceid.upatras.gr> wrote:
>On 2007-05-07 11:34, "Randal L. Schwartz" <merlyn@stonehenge.com> wrote:
>>
>> % make clean bootstrap install
>> ...
>> Compiling /Users/merlyn/MIRROR/emacs-CVS/lisp/./eshell/em-term.el
>> Loading /Users/merlyn/MIRROR/emacs-CVS/lisp/net/ange-ftp.el (source)...
>>
>> In toplevel form:
>> eshell/em-term.el:48:1:Error: Wrong type argument: keymapp, nil
>> make[2]: *** [compile] Error 1
>> make[1]: *** [bootstrap-build] Error 2
>> make: *** [bootstrap] Error 2
>> make clean bootstrap install  168.80s user 32.79s system 67% cpu 4:59.70 total
>
> Same here.  I'm bisecting through commits to find the first change
> which caused this.  Hold on for half an hour or so and I will have
> more details.

The first bad revision is:
changeset:   81110:6207e966040e
user:        monnier
date:        Mon May 07 15:41:37 2007 +0000
summary:     (Fdefine_key, Flookup_key): Only do the 0x80->meta_modifier

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: CVS HEAD fails to build
  2007-05-07 19:50   ` Giorgos Keramidas
@ 2007-05-07 20:51     ` Andreas Schwab
  2007-05-07 21:03       ` Giorgos Keramidas
  2007-05-07 21:03       ` Randal L. Schwartz
  2007-05-07 20:51     ` Giorgos Keramidas
  1 sibling, 2 replies; 13+ messages in thread
From: Andreas Schwab @ 2007-05-07 20:51 UTC (permalink / raw)
  To: Giorgos Keramidas; +Cc: emacs-devel, Randal L. Schwartz

Giorgos Keramidas <keramida@ceid.upatras.gr> writes:

> The first bad revision is:
> changeset:   81110:6207e966040e
> user:        monnier
> date:        Mon May 07 15:41:37 2007 +0000
> summary:     (Fdefine_key, Flookup_key): Only do the 0x80->meta_modifier

Does it still happen?

Andreas.

-- 
Andreas Schwab, SuSE Labs, schwab@suse.de
SuSE Linux Products GmbH, Maxfeldstraße 5, 90409 Nürnberg, Germany
PGP key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: CVS HEAD fails to build
  2007-05-07 19:50   ` Giorgos Keramidas
  2007-05-07 20:51     ` Andreas Schwab
@ 2007-05-07 20:51     ` Giorgos Keramidas
  1 sibling, 0 replies; 13+ messages in thread
From: Giorgos Keramidas @ 2007-05-07 20:51 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: emacs-devel, Randal L. Schwartz

On 2007-05-07 22:50, Giorgos Keramidas <keramida@ceid.upatras.gr> wrote:
> The first bad revision is:
> changeset:   81110:6207e966040e
> user:        monnier
> date:        Mon May 07 15:41:37 2007 +0000
> summary:     (Fdefine_key, Flookup_key): Only do the 0x80->meta_modifier

Hi Stefan and all,

this part of the change seems to be the one which is causing the
build problem:

% diff -r 62063c2fb043 -r 6207e966040e src/keymap.c
% --- a/src/keymap.c      Mon May 07 08:05:55 2007 +0000
% +++ b/src/keymap.c      Mon May 07 15:41:37 2007 +0000
% @@ -1312,7 +1313,7 @@ recognize the default bindings, just as
%         c = Fevent_convert_list (c);
%
%        /* Turn the 8th bit of string chars into a meta modifier.  */
% -      if (INTEGERP (c) && XINT (c) & 0x80 && STRINGP (key))
% +      if (STRINGP (key) && XINT (c) & 0x8 && !STRING_MULTIBYTE (key))
%          XSETINT (c, (XINT (c) | meta_modifier) & ~0x80);
%
%        /* Allow string since binding for `menu-bar-select-buffer'

I have locally reverted the condition to:

         if (INTEGERP (c) && XINT (c) & 0x80 && STRINGP (key) &&
             !STRING_MULTIBYTE (key))
           XSETINT (c, (XINT (c) | meta_modifier) & ~0x80);

but I am not sure if it matches what you originally intended to do,
Stefan.  Does the new check seem reasonable?

- Giorgos

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: CVS HEAD fails to build
  2007-05-07 20:51     ` Andreas Schwab
@ 2007-05-07 21:03       ` Giorgos Keramidas
  2007-05-07 21:03       ` Randal L. Schwartz
  1 sibling, 0 replies; 13+ messages in thread
From: Giorgos Keramidas @ 2007-05-07 21:03 UTC (permalink / raw)
  To: Andreas Schwab; +Cc: emacs-devel, Randal L. Schwartz

On 2007-05-07 22:51, Andreas Schwab <schwab@suse.de> wrote:
> Giorgos Keramidas <keramida@ceid.upatras.gr> writes:
> > The first bad revision is:
> > changeset:   81110:6207e966040e
> > user:        monnier
> > date:        Mon May 07 15:41:37 2007 +0000
> > summary:     (Fdefine_key, Flookup_key): Only do the 0x80->meta_modifier
> 
> Does it still happen?

I just rsync'ed again and caught your change (-r 1.354 of
keymap.c), and Emacs builds fine again, thanks :)

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: CVS HEAD fails to build
  2007-05-07 20:51     ` Andreas Schwab
  2007-05-07 21:03       ` Giorgos Keramidas
@ 2007-05-07 21:03       ` Randal L. Schwartz
  1 sibling, 0 replies; 13+ messages in thread
From: Randal L. Schwartz @ 2007-05-07 21:03 UTC (permalink / raw)
  To: Andreas Schwab; +Cc: Giorgos Keramidas, emacs-devel

>>>>> "Andreas" == Andreas Schwab <schwab@suse.de> writes:

Andreas> Does it still happen?

After the CVS refresh, builds fine for me.  Seems to launch too.

-- 
Randal L. Schwartz - Stonehenge Consulting Services, Inc. - +1 503 777 0095
<merlyn@stonehenge.com> <URL:http://www.stonehenge.com/merlyn/>
Perl/Unix/security consulting, Technical writing, Comedy, etc. etc.
See PerlTraining.Stonehenge.com for onsite and open-enrollment Perl training!

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: CVS HEAD fails to build
  2007-05-07 18:34 CVS HEAD fails to build Randal L. Schwartz
  2007-05-07 18:57 ` Giorgos Keramidas
@ 2007-05-07 21:41 ` Stefan Monnier
  2007-05-08 18:08   ` Richard Stallman
  1 sibling, 1 reply; 13+ messages in thread
From: Stefan Monnier @ 2007-05-07 21:41 UTC (permalink / raw)
  To: Randal L. Schwartz; +Cc: emacs-devel

> % make clean bootstrap install
> ...
> Compiling /Users/merlyn/MIRROR/emacs-CVS/lisp/./eshell/em-term.el
> Loading /Users/merlyn/MIRROR/emacs-CVS/lisp/net/ange-ftp.el (source)...

> In toplevel form:
> eshell/em-term.el:48:1:Error: Wrong type argument: keymapp, nil
> make[2]: *** [compile] Error 1
> make[1]: *** [bootstrap-build] Error 2
> make: *** [bootstrap] Error 2
> make clean bootstrap install  168.80s user 32.79s system 67% cpu 4:59.70 total

I believe this was a typo, fixed by Andreas soon after I installed it.


        Stefan

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: CVS HEAD fails to build
  2007-05-07 21:41 ` Stefan Monnier
@ 2007-05-08 18:08   ` Richard Stallman
  2007-05-08 18:17     ` Stefan Monnier
  0 siblings, 1 reply; 13+ messages in thread
From: Richard Stallman @ 2007-05-08 18:08 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: emacs-devel, merlyn

The change to deal with multibyte strings differently feels risky to
me.  It is a clever idea, and might offer a way out of the dilemma.
But we should try it when we have more time to fix whatever it might
break.

So I'd rather not have it in the 22.1 release.
Would you please take it out of there, for now?

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: CVS HEAD fails to build
  2007-05-08 18:08   ` Richard Stallman
@ 2007-05-08 18:17     ` Stefan Monnier
  2007-05-09  3:37       ` Richard Stallman
  0 siblings, 1 reply; 13+ messages in thread
From: Stefan Monnier @ 2007-05-08 18:17 UTC (permalink / raw)
  To: rms; +Cc: emacs-devel, merlyn

> The change to deal with multibyte strings differently feels risky to
> me.  It is a clever idea, and might offer a way out of the dilemma.
> But we should try it when we have more time to fix whatever it might
> break.

> So I'd rather not have it in the 22.1 release.
> Would you please take it out of there, for now?

It's definitely not slated for inclusion in Emacs-22.1, of course.


        Stefan

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: CVS HEAD fails to build
  2007-05-08 18:17     ` Stefan Monnier
@ 2007-05-09  3:37       ` Richard Stallman
  2007-05-09 10:11         ` Randal L. Schwartz
  0 siblings, 1 reply; 13+ messages in thread
From: Richard Stallman @ 2007-05-09  3:37 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: emacs-devel, merlyn

    It's definitely not slated for inclusion in Emacs-22.1, of course.

Oh, I see.  Thanks.

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: CVS HEAD fails to build
  2007-05-09  3:37       ` Richard Stallman
@ 2007-05-09 10:11         ` Randal L. Schwartz
  2007-05-09 13:58           ` Stefan Monnier
  0 siblings, 1 reply; 13+ messages in thread
From: Randal L. Schwartz @ 2007-05-09 10:11 UTC (permalink / raw)
  To: emacs-devel

>>>>> "Richard" == Richard Stallman <rms@gnu.org> writes:

Richard>     It's definitely not slated for inclusion in Emacs-22.1, of course.
Richard> Oh, I see.  Thanks.

Yeah, that's why I said "CVS HEAD".  Not sure what to call that version. :)
Oh, it calls itself 22.1.50.  Sorry, next time I'll be more clear.
I hate CVS.

-- 
Randal L. Schwartz - Stonehenge Consulting Services, Inc. - +1 503 777 0095
<merlyn@stonehenge.com> <URL:http://www.stonehenge.com/merlyn/>
Perl/Unix/security consulting, Technical writing, Comedy, etc. etc.
See PerlTraining.Stonehenge.com for onsite and open-enrollment Perl training!

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: CVS HEAD fails to build
  2007-05-09 10:11         ` Randal L. Schwartz
@ 2007-05-09 13:58           ` Stefan Monnier
  0 siblings, 0 replies; 13+ messages in thread
From: Stefan Monnier @ 2007-05-09 13:58 UTC (permalink / raw)
  To: Randal L. Schwartz; +Cc: emacs-devel

> Yeah, that's why I said "CVS HEAD".  Not sure what to call that version. :)
> Oh, it calls itself 22.1.50.  Sorry, next time I'll be more clear.

Every branch has a HEAD.  The "nameless" branch in CVS (which can be
selected via "update -A" or sometimes with "update -r 1") is called the
"trunk".  There's been patches floating around to make `cvs' accept the
special tag TRUNK to refer to the trunk, but I don't know if these will ever
be included (just like the patches that add a special ".BASE" suffix to
select the base of a branch).


        Stefan

^ permalink raw reply	[flat|nested] 13+ messages in thread

end of thread, other threads:[~2007-05-09 13:58 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-05-07 18:34 CVS HEAD fails to build Randal L. Schwartz
2007-05-07 18:57 ` Giorgos Keramidas
2007-05-07 19:50   ` Giorgos Keramidas
2007-05-07 20:51     ` Andreas Schwab
2007-05-07 21:03       ` Giorgos Keramidas
2007-05-07 21:03       ` Randal L. Schwartz
2007-05-07 20:51     ` Giorgos Keramidas
2007-05-07 21:41 ` Stefan Monnier
2007-05-08 18:08   ` Richard Stallman
2007-05-08 18:17     ` Stefan Monnier
2007-05-09  3:37       ` Richard Stallman
2007-05-09 10:11         ` Randal L. Schwartz
2007-05-09 13:58           ` Stefan Monnier

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).