Re: Flex prefix strip-off bug (?)

2014-12-09 Thread Alfredo Braunstein
(-1)*(-1) A/ On Mon, Dec 8, 2014 at 9:24 AM, Jean-Marc Lasgouttes wrote: > Le 08/12/2014 09:00, Jürgen Spitzmüller a écrit : >> >> Today, I'll have to agree that I disagree with what I stated yesterday. >> Now I >> agree with JMarc and Richard. Not sure if I agree with Alfredo, though. > > > +1

Re: Flex prefix strip-off bug (?)

2014-12-08 Thread Jean-Marc Lasgouttes
Le 08/12/2014 09:00, Jürgen Spitzmüller a écrit : Today, I'll have to agree that I disagree with what I stated yesterday. Now I agree with JMarc and Richard. Not sure if I agree with Alfredo, though. +1 JMarc

Re: Flex prefix strip-off bug (?)

2014-12-08 Thread Jürgen Spitzmüller
Alfredo Braunstein wrote: > > But I agree with the fact that Richard agrees with me :) > > I'll have to disagree with the fact that he disagrees. Today, I'll have to agree that I disagree with what I stated yesterday. Now I agree with JMarc and Richard. Not sure if I agree with Alfredo, though.

Re: Flex prefix strip-off bug (?)

2014-12-07 Thread Alfredo Braunstein
On Sun, Dec 7, 2014 at 8:09 PM, Jean-Marc Lasgouttes wrote: > Le 07/12/2014 17:22, Richard Heck a écrit : >>> >>> The point is that InsetLayout const & DocumentClass::insetLayout() >>> does not >>> find Flex:Foo, but only Foo. >> >> >> No, it will find Flex:Emph, for example, from the Logical Mark

Re: Flex prefix strip-off bug (?)

2014-12-07 Thread Jean-Marc Lasgouttes
Le 07/12/2014 17:22, Richard Heck a écrit : The point is that InsetLayout const & DocumentClass::insetLayout() does not find Flex:Foo, but only Foo. No, it will find Flex:Emph, for example, from the Logical Markup module. I put some debugging code in to verify this. I do not have time to loo

Re: Flex prefix strip-off bug (?)

2014-12-07 Thread Richard Heck
On 12/07/2014 11:07 AM, Jürgen Spitzmüller wrote: Richard Heck wrote: We need to strip suffixes both times. One could have an "InsetLayout Flex", as well. I think there was a time that we did not require the prefix "Flex:", which might be what is confusing things. There is

Re: Flex prefix strip-off bug (?)

2014-12-07 Thread Jürgen Spitzmüller
Richard Heck wrote: > We need to strip suffixes both times. One could have an "InsetLayout > Flex", as well. > > I think there was a time that we did not require the prefix "Flex:", > which might be what > is confusing things. There is code elsewhere that

Re: Flex prefix strip-off bug (?)

2014-12-07 Thread Richard Heck
's largely pointless in this case, but with things like Note:Foo it makes more sense. You could have an "InsetLayout Note" that was the default for all notes. I see. But then, we have a severe design flaw if we need to strip off prefixes in one case (Flex) and suffixed in the other (

Re: Flex prefix strip-off bug (?)

2014-12-07 Thread Jürgen Spitzmüller
this case, but with > things like Note:Foo > it makes more sense. You could have an "InsetLayout Note" that was the > default for all > notes. I see. But then, we have a severe design flaw if we need to strip off prefixes in one case (Flex) and suffixed in the other (Note). Jürgen

Re: Flex prefix strip-off bug (?)

2014-12-07 Thread Richard Heck
On 12/07/2014 06:17 AM, Jürgen Spitzmüller wrote: Jean-Marc Lasgouttes wrote: Thé purpose is really to strip thé suffixes. If I have an unknown type of Flex like Flex:Foo, it will fall back to Flex, that can put it in red and show that it is unknown. It is poor man object hierarchy

Re: Flex prefix strip-off bug (?)

2014-12-07 Thread Jürgen Spitzmüller
Jean-Marc Lasgouttes wrote: > Thé purpose is really to strip thé suffixes. If I have an unknown type of > Flex like Flex:Foo, it will fall back to Flex, that can put it in red and > show that it is unknown. > > It is poor man object hierarchy...Therefore I think you are wrong

Re: Flex prefix strip-off bug (?)

2014-12-07 Thread Jean-Marc Lasgouttes
Thé purpose is really to strip thé suffixes. If I have an unknown type of Flex like Flex:Foo, it will fall back to Flex, that can put it in red and show that it is unknown. It is poor man object hierarchy...Therefore I think you are wrong :) JMarc Le 7 décembre 2014 11:06:44 CET, "J

Flex prefix strip-off bug (?)

2014-12-07 Thread Jürgen Spitzmüller
In InsetLayout const & DocumentClass::insetLayout(), the following code is supposed to strip off layout prefixes (i.e., Flex::Foo -> Foo) from the string n in order to find a layout by name: size_t i = n.find(':'); if (i == string::npos)

Re: [LyX/master] lyx_pot.py: Do not strip blanks in HelpText.

2014-04-22 Thread Kornel Benko
Am Dienstag, 22. April 2014 um 10:11:34, schrieb Richard Heck > On 04/22/2014 06:01 AM, Kornel Benko wrote: > > > > Am Dienstag, 22. April 2014 um 11:54:09, schrieb Jürgen Spitzmüller > > > > > > > > Why the extra "\s*" ? > > > > > > > > > > > > > > > > I cannot remember. This is 7 year old code

Re: [LyX/master] lyx_pot.py: Do not strip blanks in HelpText.

2014-04-22 Thread Richard Heck
On 04/22/2014 06:01 AM, Kornel Benko wrote: Am Dienstag, 22. April 2014 um 11:54:09, schrieb Jürgen Spitzmüller > > Why the extra "\s*" ? > > > > I cannot remember. This is 7 year old code. Most likely I just took an > existing regexp as a model. Actually, the existence of this extra co

Re: [LyX/master] lyx_pot.py: Do not strip blanks in HelpText.

2014-04-22 Thread Jürgen Spitzmüller
2014-04-22 12:01 GMT+02:00 Kornel Benko: > Actually, the existence of this extra code was also the reason why I > thought that > > the intent in the original version was to remove the trailing white space. > It might make a difference wrt newlines (e.g. \r on *nix); I don't know. In any case, I

Re: [LyX/master] lyx_pot.py: Do not strip blanks in HelpText.

2014-04-22 Thread Kornel Benko
Am Dienstag, 22. April 2014 um 11:54:09, schrieb Jürgen Spitzmüller > > Why the extra "\s*" ? > > > > I cannot remember. This is 7 year old code. Most likely I just took an > existing regexp as a model. Actually, the existence of this extra code was also the reason why I thought that the inten

Re: [LyX/master] lyx_pot.py: Do not strip blanks in HelpText.

2014-04-22 Thread Jürgen Spitzmüller
2014-04-22 11:39 GMT+02:00 Kornel Benko: > > I did > > > not write the original regex. > Well, actually I did. > > > I meant the difference between > > re.compile(r'\s*(\S.*)\s*$') > > and > > re.compile(r'\s*(\S.*)$') > > > > Why the extra "\s*" ? > I cannot remember. This is 7 year old code.

Re: [LyX/master] lyx_pot.py: Do not strip blanks in HelpText.

2014-04-22 Thread Kornel Benko
Am Dienstag, 22. April 2014 um 11:31:36, schrieb Jürgen Spitzmüller > 2014-04-22 11:20 GMT+02:00 Kornel Benko: > > > > - HelpTextSection = re.compile(r'\s*(\S.*\S)\s*$') > > > > > + HelpTextSection = re.compile(r'\s*(\S.*)\s*$') > > > > > > > > I do not understand. > > > > What is the differenc

Re: [LyX/master] lyx_pot.py: Do not strip blanks in HelpText.

2014-04-22 Thread Jürgen Spitzmüller
2014-04-22 11:20 GMT+02:00 Kornel Benko: > > - HelpTextSection = re.compile(r'\s*(\S.*\S)\s*$') > > > + HelpTextSection = re.compile(r'\s*(\S.*)\s*$') > > > > I do not understand. > > What is the difference to re'\s*(\S.*)$'? Does '.' not include white space? > I just restored the working versio

Re: [LyX/master] lyx_pot.py: Do not strip blanks in HelpText.

2014-04-22 Thread Kornel Benko
Am Dienstag, 22. April 2014 um 10:45:54, schrieb Juergen Spitzmueller > commit 5e530e075734cee46f39d52a9f6fd7cda40545f5 > Author: Juergen Spitzmueller > Date: Tue Apr 22 10:35:20 2014 +0200 > > lyx_pot.py: Do not strip blanks in HelpText. > > This fixes a

Re: [PATCH] Strip unused originaldir flag from Converters.{cpp,h}

2007-06-18 Thread Richard Heck
Georg Baum wrote: Am Samstag, 16. Juni 2007 17:20 schrieb Richard Heck: Jose, JMarc, Georg, Views about whether this should now go in? I think so, but why this return statement? +// We're not waiting for the result, so we can't do anything +// else here. +r

Re: [PATCH] Strip unused originaldir flag from Converters.{cpp,h}

2007-06-16 Thread Georg Baum
Am Samstag, 16. Juni 2007 17:20 schrieb Richard Heck: > > Jose, JMarc, Georg, > > Views about whether this should now go in? I think so, but why this return statement? >+   // We're not waiting for the result, so we can't do anything >+  

Re: [PATCH] Strip unused originaldir flag from Converters.{cpp,h}

2007-06-16 Thread Richard Heck
Jose, JMarc, Georg, Views about whether this should now go in? rh Richard Heck wrote: Georg Baum wrote: Jean-Marc Lasgouttes wrote: Extended.lyx has a reference to it related to literate programming. Are you sure it is not relevant anymore? The problem is that it is horribly broken cur

Re: [PATCH] Strip unused originaldir flag from Converters.{cpp,h}

2007-06-15 Thread Richard Heck
Georg Baum wrote: Jean-Marc Lasgouttes wrote: Richard> As said. This flag was removed from configure.py a while ago. Richard> It was only ever used with HTML export, and it didn't work Richard> there, anyway. I didn't remove it from Converters.* at that Richard> time, because I was intending

Re: [PATCH] Strip unused originaldir flag from Converters.{cpp,h}

2007-06-15 Thread Georg Baum
Jean-Marc Lasgouttes wrote: >> "Richard" == Richard Heck >> <[EMAIL PROTECTED]> writes: > > Richard> This time with the patch... > > Richard> As said. This flag was removed from configure.py a while ago. > Richard> It was only ever used with HTML export, and it didn't work > Richard> the

Re: [PATCH] Strip unused originaldir flag from Converters.{cpp,h}

2007-06-14 Thread Jean-Marc Lasgouttes
> "Richard" == Richard Heck <[EMAIL PROTECTED]> writes: Richard> This time with the patch... Richard> As said. This flag was removed from configure.py a while ago. Richard> It was only ever used with HTML export, and it didn't work Richard> there, anyway. I didn't remove it from Converters.*

[PATCH] Strip unused originaldir flag from Converters.{cpp,h}

2007-06-14 Thread Richard Heck
This time with the patch... As said. This flag was removed from configure.py a while ago. It was only ever used with HTML export, and it didn't work there, anyway. I didn't remove it from Converters.* at that time, because I was intending to make more extensive changes to this file in connection

[PATCH] Strip unused originaldir flag from Converters.{cpp,h}

2007-06-14 Thread Richard Heck
As said. This flag was removed from configure.py a while ago. It was only ever used with HTML export, and it didn't work there, anyway. I didn't remove it from Converters.* at that time, because I was intending to make more extensive changes to this file in connection with HTML export. But it

Re: latest cvs: no make install-strip

2002-04-07 Thread Lars Gullik Bjønnes
Herbert Voss <[EMAIL PROTECTED]> writes: | what goes wrong?? my fault... Use "make -k install" The problem is that "makefile" overrides "Makefile" -- Lgb

Re: latest cvs: no make install-strip

2002-04-07 Thread Herbert Voss
1 > make[3]: Leaving directory `/home/voss/lyx-devel.neu/src/frontends' > make[2]: *** [install-recursive] Error 1 > make[2]: Leaving directory `/home/voss/lyx-devel.neu/src' > make[1]: *** [install-recursive] Error 1 > make[1]: Leaving directory `/home/voss/lyx-devel.neu&#x

latest cvs: no make install-strip

2002-04-07 Thread Herbert Voss
me/voss/lyx-devel.neu/src/frontends' make[2]: *** [install-recursive] Error 1 make[2]: Leaving directory `/home/voss/lyx-devel.neu/src' make[1]: *** [install-recursive] Error 1 make[1]: Leaving directory `/home/voss/lyx-devel.neu' make: *** [install-strip] Error 2 -- http://www.lyx.org/help/

Re: strip()

2001-08-26 Thread John Levon
On Mon, Aug 27, 2001 at 01:47:34AM +0200, Lars Gullik Bjønnes wrote: > John Levon <[EMAIL PROTECTED]> writes: > > | p.s. can someone check if adding XSynchronize(fl_get_display(), True) > | makes the xforms build core-dump too ? > > and what do you use it for? I'm trying to debug the Qt static

Re: strip()

2001-08-26 Thread John Levon
On Mon, Aug 27, 2001 at 01:32:08AM +0200, Lars Gullik Bjønnes wrote: > John Levon <[EMAIL PROTECTED]> writes: > > | I've just been bitten again by strip()'s name. > | > | It should be called backStrip(), and we hsould have a strip() > | that does wha

Re: [PATCH] (resent) www-user strip slashes from comment

2001-01-25 Thread Baruch Even
On 25 Jan 2001, Jean-Marc Lasgouttes wrote: > > "Baruch" == Baruch Even <[EMAIL PROTECTED]> writes: > > Baruch> Sorry the former > Baruch> was a patch to www-devel. Again, it should fix the mail with > Baruch> slashes problem. > > Applied. We'll see how well it works. Should the other patch

Re: [PATCH] (resent) www-user strip slashes from comment

2001-01-25 Thread Jean-Marc Lasgouttes
> "Baruch" == Baruch Even <[EMAIL PROTECTED]> writes: Baruch> Sorry the former Baruch> was a patch to www-devel. Again, it should fix the mail with Baruch> slashes problem. Applied. We'll see how well it works. Should the other patch be applied as well? JMarc

[PATCH] (resent) www-user strip slashes from comment

2001-01-24 Thread Baruch Even
Sorry the former was a patch to www-devel. Again, it should fix the mail with slashes problem. -- Baruch Even http://baruch.ev-en.org/ (My Site) http://www.redrival.com/jindor/ (My brothers AD&D site) " Learn to laugh ... it's the path to true love! " - The Angel in the movie

Re: [PATCH] www-user strip slashes from comments

2001-01-24 Thread Lars Gullik Bjønnes
Baruch Even <[EMAIL PROTECTED]> writes: | Attached is the patch to remove the slashes from the mailed | comments. No, it does not. Lgb

Re: [PATCH] www-user strip slashes from comments

2001-01-24 Thread Jean-Marc Lasgouttes
> "Baruch" == Baruch Even <[EMAIL PROTECTED]> writes: Baruch> Attached is the Baruch> patch to remove the slashes from the mailed comments. Huh? What I see is a patch to tasks.php3... Should it be applied anyway? JMarc

[PATCH] www-user strip slashes from comments

2001-01-24 Thread Baruch Even
Attached is the patch to remove the slashes from the mailed comments. -- Baruch Even http://baruch.ev-en.org/ (My Site) http://www.redrival.com/jindor/ (My brothers AD&D site) " Learn to laugh ... it's the path to true love! " - The Angel in the movie Michael Index: tasks.p

Re: make install-strip again

2000-06-26 Thread Jean-Marc Lasgouttes
>>>>> "Garst" == Garst R Reese <[EMAIL PROTECTED]> writes: Garst> I tried it. It fails. It bails out when it cannot strip Garst> something. In particular reLyX. Anything after that, e.g., man Garst> pages, does not get installed. Garst This is a bad inte

Re: make install-strip again

2000-06-09 Thread Angus Leeming
Garst> > Here's what I would do (as in the RPM spec file): Garst> > CXXFLAGS="-O2" CFLAGS="-O2" ./configure --prefix=/usr Garst> > And then the make happens without the debug flag. Garst> I used to do that, but then it got more complicated with the Garst> no-exceptions, no-rtii flags (or

Re: make install-strip again

2000-06-08 Thread Garst R. Reese
"Kayvan A. Sylvan" wrote: > > On Fri, Jun 09, 2000 at 12:34:25AM -0300, Garst R. Reese wrote: > > I tried it. It fails. It bails out when it cannot strip something. In > > particular reLyX. Anything after that, e.g., man pages, does not get > > installed. >

Re: make install-strip again

2000-06-08 Thread Kayvan A. Sylvan
On Fri, Jun 09, 2000 at 12:34:25AM -0300, Garst R. Reese wrote: > I tried it. It fails. It bails out when it cannot strip something. In > particular reLyX. Anything after that, e.g., man pages, does not get > installed. > Garst Here's what I would do (as in the RPM spec file):

install-strip

2000-06-08 Thread Garst R. Reese
For the benefit of those with limited machine capacity, wouldn't it make more sense to have a --disable-debug flag in the configure script so that the -g does not go in the compile flags? Garst

make install-strip again

2000-06-08 Thread Garst R. Reese
I tried it. It fails. It bails out when it cannot strip something. In particular reLyX. Anything after that, e.g., man pages, does not get installed. Garst

Re: strip commande in install procedure

2000-06-08 Thread Jean-Marc Lasgouttes
>>>>> "Yann" == Yann MORERE <[EMAIL PROTECTED]> writes: Yann> i made the strip command on binaries and i saved 32MB disk space Yann> on the lyx program. Yann> so is it possible to put this command in the install procedure? What version was that? The s

strip commande in install procedure

2000-06-07 Thread Yann MORERE
Hello, firstly, i would like to thank larry for his help I was surprised at the size of the binary on a PowerPC (11MB), but even more surprised when I did "strip /usr/bin/lyx" -- the resulting binary is only 1.6MB! So I suspect if you strip your binary, you will have plenty of ro