RE: Miscellaneous Re: pws-14

RE: Miscellaneous Re: pws-14

[Sven Wischnowsky]

Andrej Borsenkow wrote:

> > Peter explicitly mentiond that 5955 was NOT in pws-14; when I applied it
> > to pws-14, exactly one hunk of zle_tricky.c failed:
> >
> 
> Strange. I applied it without any problems (different patch versions? I have
> 2.5.3) The only patch I got problem with was _make patch (5958) - I had to
> use -l option (ignore white space).  Oh, yes, I have 5949 before it - may it
> be the reason? (it is missed in pws-14)

Yes, that's the reason. With 5949 applied it should work. Peter didn't 
include it because he didn't like it, and  5955 reverses the effect of 
it, so there is no harm it applying 5949 first.

Bye
 Sven


--
Sven Wischnowsky                         wischnow@informatik.hu-berlin.de

RE: Miscellaneous Re: pws-14

[Andrej Borsenkow]

>
> Peter explicitly mentiond that 5955 was NOT in pws-14; when I applied it
> to pws-14, exactly one hunk of zle_tricky.c failed:
>

Strange. I applied it without any problems (different patch versions? I have
2.5.3) The only patch I got problem with was _make patch (5958) - I had to
use -l option (ignore white space).  Oh, yes, I have 5949 before it - may it
be the reason? (it is missed in pws-14)

cheers

/andrej

Miscellaneous Re: pws-14

[Bart Schaefer]

Peter didn't mention Sven's patch from 5936 in his list of those included
in pws-14, but it appears to be there nonetheless.

Peter explicitly mentiond that 5955 was NOT in pws-14; when I applied it
to pws-14, exactly one hunk of zle_tricky.c failed:

***************
*** 6721,6728 ****
  	/* We are currently not in a menu-completion, *
  	 * so set the position variables.             */
  	menupos = wb;
- 	/* previously:	menuwe = (cs == we) || isset(ALWAYSTOEND); */
- 	menuwe = isset(ALWAYSTOEND);
  	menuend = we;
      }
      /* If we are already in a menu-completion or if we have done a *
--- 6758,6764 ----
  	/* We are currently not in a menu-completion, *
  	 * so set the position variables.             */
  	menupos = wb;
+ 	menuwe = (movetoend >= 2 || (movetoend = 1 && !menucmp));
  	menuend = we;
      }
      /* If we are already in a menu-completion or if we have done a *


My pws-14 has the code that is inside the /* previously: ... */ comment
above.  Am I missing some other patch that should be applied to pws-14
before applying 5955?

-- 
Bart Schaefer                                 Brass Lantern Enterprises
http://www.well.com/user/barts              http://www.brasslantern.com