Re: LyX/Literate programming bug report and fix

2000-09-18 Thread Allan Rae

On Fri, 15 Sep 2000, Juergen Vigna wrote:

  
  Nothing, as I said above in the first case.  The only problem comes with
  the fact that new files are not included by the -N.  Only new files that
  have been added to cvs using `cvs add` are actually caught by the -N flag.
  To be able to run `cvs add` you have to be given at least read-only access
  to cvs by Lars.  This just means extra work for Lars and extra hassle for
  budding contributors just to be able to submit a patch that will be in an
  acceptable form.  This is a silly requirement IMO as it is overly
  restrictive.
 
 Well I guess you didn't get enough sleep last night, isn't it!
 
 I've never seen that a patch as John sends them with a cvs diff -u and
 the newfiles in a tar-ball are rejected,

I'm sure we've had more than occasion where contributors have forgotten
those new files simply because they expected diff would add the new files
to the patch for them.  Sure they should have checked their patch and
noticed this but getting a patch and a tarball is a pain.  Why not get a
patch from everyone?  The modified diffutils provides a way for that.

 I've only seen requests to Lars
 to add some of the "not fast disapearing developers" to the cvs-readonly
 list so that is is easier for them to do a cvs diff and include also the
 newfiles (and deleted files) in the patch!

So we make life easier for some who've proven worthy and others just have
to struggle.  I wish it was friday then I could leave out the smiley's ;-)

 I understand that you want publicity for your patched diff-utils!

Gotta start somewhere.

Allan. (ARRae)




Re: make dist oversight

2000-09-18 Thread Lars Gullik Bjønnes

"Kayvan A. Sylvan" [EMAIL PROTECTED] writes:

| Hi folk,
| 
| This small patch fixes a ``make dist'' oversight (and lets my auto-compile
| and RPM generation proceed). However, even with this, I get the following
| errors on compilation:
| 
| table.C:791: new declaration `class string LyXTable::GetPWidth(int)'
| table.h:173: ambiguates old declaration `const class string
| LyXTable::GetPWidth(

This is strange since table.C and table. are not part of the build
process anymore.

Lgb



Re: make dist oversight

2000-09-18 Thread Juergen Vigna


On 18-Sep-2000 Lars Gullik Bjønnes wrote:
 "Kayvan A. Sylvan" [EMAIL PROTECTED] writes:
 
| Hi folk,
| 
| This small patch fixes a ``make dist'' oversight (and lets my auto-compile
| and RPM generation proceed). However, even with this, I get the following
| errors on compilation:
| 
| table.C:791: new declaration `class string LyXTable::GetPWidth(int)'
| table.h:173: ambiguates old declaration `const class string
| LyXTable::GetPWidth(
 
 This is strange since table.C and table. are not part of the build
 process anymore.
 

Well he modified the Makefile and put them in again ;)

I fixed this and IMO they still should be part of the build process
at least until we remove all references to them in the LyX-Source and
remove them from the repository!

  Jürgen

--
-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._

Dr. Jürgen VignaE-Mail:  [EMAIL PROTECTED]
Italienallee 13/N   Tel/Fax: +39-0471-450260 / +39-0471-450253
I-39100 Bozen   Web: http://www.sad.it/~jug

-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._

With your bare hands?!?




Re: patch

2000-09-18 Thread Lars Gullik Bjønnes

Juergen Vigna [EMAIL PROTECTED] writes:

| On 15-Sep-2000 John Levon wrote:
| 
|  You are right, I have added the moc files to the make clean target in the
|  attached patch, incremental to my previous one today.
|  
| 
| IMO moc files should only be removed on a distclean not on a simple
| clean, but if you insist that it should be done also on a make clean
| then I'll apply this patch!

distclean is certainly the right target for this, not clean.

| BTW.: I got a bit out of sync with KDE/GNOME patches as I had to wait
|   for Lars to have a look at one of them, could you please tell me
|   (John  Marko) which ones haven't been aplied?

Yes, please. I lost a bit of the oversight recently.

Lgb



Re: make dist oversight

2000-09-18 Thread Lars Gullik Bjønnes

Juergen Vigna [EMAIL PROTECTED] writes:

| On 18-Sep-2000 Lars Gullik Bjønnes wrote:
|  "Kayvan A. Sylvan" [EMAIL PROTECTED] writes:
|  
| | Hi folk,
| | 
| | This small patch fixes a ``make dist'' oversight (and lets my auto-compile
| | and RPM generation proceed). However, even with this, I get the following
| | errors on compilation:
| | 
| | table.C:791: new declaration `class string LyXTable::GetPWidth(int)'
| | table.h:173: ambiguates old declaration `const class string
| | LyXTable::GetPWidth(
|  
|  This is strange since table.C and table. are not part of the build
|  process anymore.
|  
| 
| Well he modified the Makefile and put them in again ;)
| 
| I fixed this and IMO they still should be part of the build process
| at least until we remove all references to them in the LyX-Source and
| remove them from the repository!

The only reason why I haven't done that already is that you might need
them a bit further. If you don't need those files I will remove them.

Lgb



Re: patch

2000-09-18 Thread Marko Vendelin



On Mon, 18 Sep 2000, Juergen Vigna wrote:

 BTW.: I got a bit out of sync with KDE/GNOME patches as I had to wait
   for Lars to have a look at one of them, could you please tell me
   (John  Marko) which ones haven't been aplied?

Gnome patches are

http://www.mail-archive.com/lyx-devel%40lists.lyx.org/msg14188.html
http://www.mail-archive.com/lyx-devel%40lists.lyx.org/msg14247.html

They both depend on TOC lists tracking implemented in the first patch.
These patches were results of the discussion on the GUI update policy of
LyX. Note, that in the present form the first patch may slow down LyX
considerably as indicated in the e-mail.

All other patches (Gnome frontend) were applied.

Marko




math-mode feature or bug?

2000-09-18 Thread Marko Vendelin


Hi!

I would like to report the following bug in the mathmode:

1. create a new document (File/New)
2. open math-mode by clicking on the toolbar or by selecting Math/Math
   Mode in menu
3. type "\alpha " and enjoy LyX dumping a core.

Marko

PS: I hope it is not only in my build (Gnome frontend)




[f95ts@efd.lth.se] [Lyx-feedback] Feedback from www.lyx.org

2000-09-18 Thread Lars Gullik Bjønnes



Tommy Sjögren ([EMAIL PROTECTED]) entered the 
following feedback message on the LyX home page:


Hi there,

I\'m trying to compile LyX, but I don\'t have Perl on my system, I guess
this is the main problemanyway
I thought it wasn\'t necessary to have Perl in order to comile the sourece
configure works fine, but when make enters relyx it doesn\'t find a
Makefile, wich is true as there isn\'t a Makefile.
I\'ve tried with \"export PERL=/bin/ls\" but didn\'t work either.

Do have to have Perl installed in order to compile?

Regards
Tommy
___
Lyx-feedback mailing list
[EMAIL PROTECTED]
http://www.lyx.org/mailman/listinfo/lyx-feedback





Re: make dist oversight

2000-09-18 Thread Juergen Vigna


On 18-Sep-2000 Lars Gullik Bjønnes wrote:

| I fixed this and IMO they still should be part of the build process
| at least until we remove all references to them in the LyX-Source and
| remove them from the repository!
 
 The only reason why I haven't done that already is that you might need
 them a bit further. If you don't need those files I will remove them.

I don't need them anymore (if I do I always could download them from
the cvs repository in an old version, couldn't I?

But what I meant is that before removing this files we have to remove
ALL NEW_TABULAR defines in the sources so that LyXTable is not mentioned
anywhere.

  Jürgen

--
-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._

Dr. Jürgen VignaE-Mail:  [EMAIL PROTECTED]
Italienallee 13/N   Tel/Fax: +39-0471-450260 / +39-0471-450253
I-39100 Bozen   Web: http://www.sad.it/~jug

-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._

Love is like the measles; we all have to go through it.
-- Jerome K. Jerome




Re: patch

2000-09-18 Thread Juergen Vigna


On 18-Sep-2000 Lars Gullik Bjønnes wrote:

| http://www.mail-archive.com/lyx-devel%40lists.lyx.org/msg14247.html
 
 I have not looked at this.

Well as this one depends on the first one IMO we cannot include this
too.

   Jürgen

--
-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._

Dr. Jürgen VignaE-Mail:  [EMAIL PROTECTED]
Italienallee 13/N   Tel/Fax: +39-0471-450260 / +39-0471-450253
I-39100 Bozen   Web: http://www.sad.it/~jug

-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._

Ours is a world of nuclear giants and ethical infants.
-- General Omar N. Bradley




Re: make dist oversight

2000-09-18 Thread Lars Gullik Bjønnes

Juergen Vigna [EMAIL PROTECTED] writes:

| On 18-Sep-2000 Lars Gullik Bjønnes wrote:
| 
| | I fixed this and IMO they still should be part of the build process
| | at least until we remove all references to them in the LyX-Source and
| | remove them from the repository!
|  
|  The only reason why I haven't done that already is that you might need
|  them a bit further. If you don't need those files I will remove them.
| 
| I don't need them anymore (if I do I always could download them from
| the cvs repository in an old version, couldn't I?
| 
| But what I meant is that before removing this files we have to remove
| ALL NEW_TABULAR defines in the sources so that LyXTable is not mentioned
| anywhere.

Sure... it is ok for you if I do this now?
And also remove table.C and table.h?

Lgb



Re: make dist oversight

2000-09-18 Thread Juergen Vigna


On 18-Sep-2000 Lars Gullik Bjønnes wrote:
 
 Sure... it is ok for you if I do this now?
 And also remove table.C and table.h?
 

I guess yes, but IMO we should ask the others.

We have seen that there ARE problems with the new tabular inset
(especially with the text-inset) so before removing the old code
and so definitively remove the old table-handling-code we should
also open our mind that this means making the textinset feature
complete (IMO this is not that much work, but all has to be tested!)

At least we have to redo the paragraph-layout and disable stuff
there if we are inside a text inset!

I need help for this as otherwise it might take some time!

I'm just now trying to fix Dekel's reported 'missing features' :)

 Jürgen

--
-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._

Dr. Jürgen VignaE-Mail:  [EMAIL PROTECTED]
Italienallee 13/N   Tel/Fax: +39-0471-450260 / +39-0471-450253
I-39100 Bozen   Web: http://www.sad.it/~jug

-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._

A straw vote only shows which way the hot air blows.
-- O'Henry




Re: patch

2000-09-18 Thread John Levon

On Mon, 18 Sep 2000, Juergen Vigna wrote:

 
 On 15-Sep-2000 John Levon wrote:
 
  You are right, I have added the moc files to the make clean target in the
  attached patch, incremental to my previous one today.
  
 
 IMO moc files should only be removed on a distclean not on a simple
 clean, but if you insist that it should be done also on a make clean
 then I'll apply this patch!


I have no strong feelings either way, so it should be left as it is.
 
 BTW.: I got a bit out of sync with KDE/GNOME patches as I had to wait
   for Lars to have a look at one of them, could you please tell me
   (John  Marko) which ones haven't been aplied?
 
   Jürgen
 

AFAICR I have no pending patches as Lars applied my last one

thanks
john

-- 
"Tragedy is if I cut my finger; comedy is if you walk into
 an open sewer and die."
- Mel Brooks





Re: make dist oversight

2000-09-18 Thread Juergen Vigna


On 18-Sep-2000 Juergen Vigna wrote:
 
 I need help for this as otherwise it might take some time!
 

Replying to myself ;)

There are a lot of functions in lyxfunc which modify or do something
with LyXText (bv-text!). Now all this functions should be handled
also by the text-inset. I now would need a nice way to do this for
all of this functions but without the need to copy their behaviour
into InsetText::LocalDispatch(...) so that 1 codebase could be used
for this functions.

One idea would be that we group all this functions so that we know
what they are and then make a LocalDispatch for them in LyXText.

A second is to copy the behaviour into InsetText::LocalDispatch(...)

What should I do?

  Jürgen

--
-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._

Dr. Jürgen VignaE-Mail:  [EMAIL PROTECTED]
Italienallee 13/N   Tel/Fax: +39-0471-450260 / +39-0471-450253
I-39100 Bozen   Web: http://www.sad.it/~jug

-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._

If all men were brothers, would you let one marry your sister?




Re: patch

2000-09-18 Thread Marko Vendelin



On 18 Sep 2000, Lars Gullik Bjønnes wrote:

 Marko Vendelin [EMAIL PROTECTED] writes:
 
 | On Mon, 18 Sep 2000, Juergen Vigna wrote:
 | 
 |  BTW.: I got a bit out of sync with KDE/GNOME patches as I had to wait
 |for Lars to have a look at one of them, could you please tell me
 |(John  Marko) which ones haven't been aplied?
 | 
 | Gnome patches are
 | 
 | http://www.mail-archive.com/lyx-devel%40lists.lyx.org/msg14188.html
 
 The idea is good, but the current way to update the lists is not good
 at all. I tink we will let this be suboptimal for the time beeing and
 leave everything as it is.

What do you mean by "suboptimal"? Is it the basic idea of updating the
list(s) by using buffer()-getTocList() (that means parsing LyX document
for headings etc.) or that it is done after every action and not
during "idle" time?

 Later we (or atleast I want to) have separate insets for headings,
 then it will be easy to emit the signals directly from the buffer when
 headings etc. change.

That would be nice. It should be the same for figures, tables, algorithms
and references too.

Marko




Re: patch

2000-09-18 Thread Lars Gullik Bjønnes

Marko Vendelin [EMAIL PROTECTED] writes:

| On 18 Sep 2000, Lars Gullik Bjønnes wrote:
| 
|  Marko Vendelin [EMAIL PROTECTED] writes:
|  
|  | On Mon, 18 Sep 2000, Juergen Vigna wrote:
|  | 
|  |  BTW.: I got a bit out of sync with KDE/GNOME patches as I had to wait
|  |for Lars to have a look at one of them, could you please tell me
|  |(John  Marko) which ones haven't been aplied?
|  | 
|  | Gnome patches are
|  | 
|  | http://www.mail-archive.com/lyx-devel%40lists.lyx.org/msg14188.html
|  
|  The idea is good, but the current way to update the lists is not good
|  at all. I tink we will let this be suboptimal for the time beeing and
|  leave everything as it is.
| 
| What do you mean by "suboptimal"? Is it the basic idea of updating the
| list(s) by using buffer()-getTocList() (that means parsing LyX document
| for headings etc.) or that it is done after every action and not
| during "idle" time?

More tat the support in gnome/kde will not be perfect from the
beginning.

| 
|  Later we (or atleast I want to) have separate insets for headings,
|  then it will be easy to emit the signals directly from the buffer when
|  headings etc. change.
| 
| That would be nice. It should be the same for figures, tables, algorithms
| and references too.

Yes, that is the idea.

Lgb



Re: Heretics and Perverts [was: Re: I'm back, take 2 :)

2000-09-18 Thread Amir Karger

On Fri, Sep 15, 2000 at 01:13:22PM -0400, [EMAIL PROTECTED] wrote:
 
 
 the vi pervert Andre audiated
 
 The problem is that there are quite a few things missing from the
 LyX kernel to make such a thing viable. I don't think it is possible
 at all to simulate vi's modes by fiddling with keybindings only.
 
 I think we'd need to run from the mini-buffer to avoid an 
 escape-sequence for every command . . . hey, wait, there's only one 
 mode in vi.  Insert is a command, not a mode (at least until vim 
 blurred the distinciton . . .

Huh? vi has insert mode! It's entered by commands like i,a,o,s,c, and their
corresponding capitals. I'm not sure whether vi admits to replace mode being
separate from insert mode, though.

 I believe Dvorak found that two hands is *better* than one. 
 
 But dvorak has mostly been debunked.  His studies were run by a 
 relative of the person trying to sell the keyboards.

Except that the debunking of dvorak has mostly been debunked.

http://www.mwbrooks.com/dvorak/dissent.html

And, btw, even if you don't believe that web page, I'll bet that many
ergonomics folks would still agree with the statement that two hands are
better than one.

 I've also used vi on a number of occasions to edit lyx files (and emacs 
 a couple of times, too--I still haven't figured out how to include line 
 breaks in vi search/replace . . .)

C-v C-m

In general, you can use ^V to quote unquotables, like Esc. I think in Vim,
the ^V'ing is a bit more user friendly, but IIRC, it works just fine in
vanilla vi.

-Amir



slides-only in slide class

2000-09-18 Thread hawk



Has anyone tried to use slides-only or notes-only in the slides class?  
I've tried a few times, but never succeeded in getting it to work.

The errors are "undefined control sequence"  Since I can'tselect from 
the lyx merror windows, I've exported to latex:

...
(/usr/share/texmf/tex/latex/config/geometry.cfg))
! Undefined control sequence.
write Only Slides \slides 

l.51 \onlyslides{\slides}
 
? 
Only Slides 
No file random__variables.aux.
! Undefined control sequence.
\@doglslidelist -\slides 
  ,99,99
l.55 \begin{document}
 
? 
--- Geometry parameters
mode: letterpaper 
h-parts: 0.75in, 505.89pt, 0.75in
v-parts: 0.75in, 686.56499pt, 0.75in
--- Page layout dimensions
\paperwidth  614.295pt
\paperheight 794.96999pt
\textwidth  505.89pt
\textheight 632.56499pt
\oddsidemargin  -18.06749pt
\evensidemargin -18.06749pt
\topmargin  -18.06749pt
\headheight 14.0pt
\headsep15.0pt
\footskip   25.0pt
\hoffset 0.0pt
\voffset 0.0pt
(1in=72.27pt, 1cm=28.45pt)
---
! Missing number, treated as zero.
to be read again 
   \relax 
l.58 \lyxnewslide{}
   
?  
)
Runaway argument?
\fi  
! File ended while scanning use of \@ggobtoend.
inserted text 
\par 
* random__variables.tex
 
? 


hawk

-- 





[gfox@chuma1.cas.usf.edu] [Lyx-feedback] Feedback from www.lyx.org

2000-09-18 Thread Lars Gullik Bjønnes



Gordon Fox ([EMAIL PROTECTED]) entered the 
following feedback message on the LyX home page:


Importing some LaTeX files fails on my machine, and I\'m not quite sure why.

The messages I get are:
 reLyX, the LaTeX to LyX translator. Revision date 2000/06/06

Reading LaTeX command syntax
(quiz1.tex: Splitting Preamble
Creating LyX preamble
Uncommented text before \\documentclass command ignored!
Use of uninitialized value at /usr/local/bin/../share/lyx/reLyX/MakePreamble.pm
line 182, PREAMBLE chunk 17.
Use of uninitialized value at /usr/local/bin/../share/lyx/reLyX/MakePreamble.pm
line 210, PREAMBLE chunk 17.
Cannot find layout file 2000/06/06.layout in dir(s) /home/gfox/.lyx/layouts at /
usr/local/bin/../share/lyx/reLyX/ReadCommands.pm line 267, PREAMBLE chunk 17.
Exited due to fatal Error!   

I\'m not really sure why it\'s looking for a file with this name -- 
obviously something isn\'t set up properly. Any suggestions as to
how to fix this?

Thanks.

Gordon Fox
___
Lyx-feedback mailing list
[EMAIL PROTECTED]
http://www.lyx.org/mailman/listinfo/lyx-feedback





Re: math-mode feature or bug?

2000-09-18 Thread Garst R. Reese

Marko Vendelin wrote:
 
 Hi!
 
 I would like to report the following bug in the mathmode:
 
 1. create a new document (File/New)
 2. open math-mode by clicking on the toolbar or by selecting Math/Math
Mode in menu
 3. type "\alpha " and enjoy LyX dumping a core.
 
 Marko
 
 PS: I hope it is not only in my build (Gnome frontend)
It's not. xforms does the same.
Garst



Re: Making patches: one possible solution.

2000-09-18 Thread Amir Karger

On Mon, Sep 18, 2000 at 02:41:03PM +1000, Allan Rae wrote:
 On Fri, 15 Sep 2000, Amir Karger wrote:
 
  On Fri, Sep 15, 2000 at 01:48:42PM +1000, Allan Rae wrote:
   
   I have modified GNU diff to be able to emulate `cvs diff`.  This allows
   anyone with two copies of a cvs snapshot to modify one and generate a
   diff between the two that is the equivalent of what they'd get if we gave
   them cvs access.
  
  are you sure one couldn't hack
  together a perl script that would use diff as it already stands? 
 
 I tried this with a shell script.  It was ugly and it required me to
 identify which files were present in one directory and not the other
 (something diff already does for directory comparisons) and call diff on
 each individual pair of files like:
   diff -u dir1/file1 /dev/null

Sounds like this worked, even if it was ugly. Still not as ugly as requiring
everyone to use a patched version of diffutils. (What happens when they
upgrade diffultils? What if someone else wants them to use a differently
patched version?) I guess if you sent in your patches to the diffutils
folks, that might make more sense. And might even be useful since lots of
people in the world who use diff also want to use it for cvs-diff-like
things.

 Either way required changes to diffutils sources so I wrote the one that
 made better sense to me.  

I don't understand why the shell script you described required changing
diffultils.

Alternatively, if you're worrying about ignoring the right files, why not
write a script that creates a file of files to ignore in the diff, and then
call diff -X? 

-Amir



Re: Making patches: one possible solution.

2000-09-18 Thread John Levon

On Mon, 18 Sep 2000, Amir Karger wrote:

 I don't understand why the shell script you described required changing
 diffultils.
 
 Alternatively, if you're worrying about ignoring the right files, why not
 write a script that creates a file of files to ignore in the diff, and then
 call diff -X? 
 
 -Amir
 

I have a prototype dontdiff file for LyX at home if anyone wants it ...

it's not complete though

john

-- 
"Tragedy is if I cut my finger; comedy is if you walk into
 an open sewer and die."
- Mel Brooks




Re: [gfox@chuma1.cas.usf.edu] [Lyx-feedback] Feedback from www.lyx.org

2000-09-18 Thread Amir Karger

Hi.

On Mon, Sep 18, 2000 at 05:49:19PM +0200, Lars Gullik Bjønnes wrote:
 
 Gordon Fox ([EMAIL PROTECTED]) entered the 
 following feedback message on the LyX home page:
 
 
 Reading LaTeX command syntax
 (quiz1.tex: Splitting Preamble
 Creating LyX preamble
 Uncommented text before \\documentclass command ignored!
 Use of uninitialized value at /usr/local/bin/../share/lyx/reLyX/MakePreamble.pm
 line 182, PREAMBLE chunk 17.
 Use of uninitialized value at /usr/local/bin/../share/lyx/reLyX/MakePreamble.pm
 line 210, PREAMBLE chunk 17.
 Cannot find layout file 2000/06/06.layout in dir(s) /home/gfox/.lyx/layouts at /
 usr/local/bin/../share/lyx/reLyX/ReadCommands.pm line 267, PREAMBLE chunk 17.
 Exited due to fatal Error!   
 
 I\'m not really sure why it\'s looking for a file with this name -- 
 obviously something isn\'t set up properly. Any suggestions as to
 how to fix this?

I agree that something weird is going on.

Could you send a small file that exhibits this error to
[EMAIL PROTECTED]? (In fact, it looks like it's just the beginning of
your file that's confusing reLyX; it never even gets to most of the
translating part.)

-Amir Karger



Re: math-mode feature or bug?

2000-09-18 Thread Lior Silberman

On Mon, 18 Sep 2000, Marko Vendelin wrote:

 
 Hi!
 
 I would like to report the following bug in the mathmode:
 
 1. create a new document (File/New)
 2. open math-mode by clicking on the toolbar or by selecting Math/Math
Mode in menu
 3. type "\alpha " and enjoy LyX dumping a core.
 
 Marko
 
 PS: I hope it is not only in my build (Gnome frontend)
 
 

This also happens in xforms. The culprit appears to be the following line
(line 787 of mathed/math_cursor.C):

imacro-SetName(0);

which is:

void MathedInset::SetName(string const  n) { name = n; }
[with: string name].

This activates basic_string's copy constructor with a null reference, gets
to a 'strlen(0)' (in /usr/include/g++-2/std/straits.h:125), and hence the
SEGV.

I think this should say one of:
imacro-SetName(string());
imacro-SetName(string(""));
imacro-SetName("");

but I don't know anything about the string template, so I don't know which
one is correct. The first one does solve the problem (tested). Is there a
global 'emptystring' instance?

Lior.




Re: math-mode feature or bug?

2000-09-18 Thread Lior Silberman

On Mon, 18 Sep 2000, Lior Silberman wrote:

 On Mon, 18 Sep 2000, Marko Vendelin wrote:
 
  
  Hi!
  
  I would like to report the following bug in the mathmode:
  
  1. create a new document (File/New)
  2. open math-mode by clicking on the toolbar or by selecting Math/Math
 Mode in menu
  3. type "\alpha " and enjoy LyX dumping a core.
  
  Marko
  
  PS: I hope it is not only in my build (Gnome frontend)
  
  
 
 I think this should say one of:
   imacro-SetName(string());
   imacro-SetName(string(""));
   imacro-SetName("");
 
 but I don't know anything about the string template, so I don't know which
 one is correct. The first one does solve the problem (tested). Is there a
 global 'emptystring' instance?
 
 Lior.
 
 

I must say didn't delve into the mathed, so this may break something, but
it seems to me that using the null pointer as a reference is wrong in most
cases.

Lior.




Re: math-mode feature or bug?

2000-09-18 Thread Lars Gullik Bjønnes

Lior Silberman [EMAIL PROTECTED] writes:

| This also happens in xforms. The culprit appears to be the following line
| (line 787 of mathed/math_cursor.C):
| 
|   imacro-SetName(0);

I really wonder why gcc accepts that.

| which is:
| 
| void MathedInset::SetName(string const  n) { name = n; }
| [with: string name].
| 
| This activates basic_string's copy constructor with a null reference, gets
| to a 'strlen(0)' (in /usr/include/g++-2/std/straits.h:125), and hence the
| SEGV.
| 
| I think this should say one of:
|   imacro-SetName(string());
|   imacro-SetName(string(""));
|   imacro-SetName("");

Either of the above should work.
(First or last looks best)

| but I don't know anything about the string template, so I don't know which
| one is correct. The first one does solve the problem (tested). Is there a
| global 'emptystring' instance?

Yes. Inside the std::string so you never have to worry about that.
(and if even if std::string does not have a shared empty string: do
not care about issues like that.)

Lgb




Re: math-mode feature or bug?

2000-09-18 Thread Lars Gullik Bjønnes

Lior Silberman [EMAIL PROTECTED] writes:

| I meant to ask whether some version of imacro-SetName(emptystring);
| is the correct syntax. Anyway: the following implements the first
| option.

I already did it with the last option. (+ another memory leak fix)

Lgb



InsetTabular tasks

2000-09-18 Thread Lars Gullik Bjønnes


Jürgen, can you make a task list on what needs to be fixed with
tabular?

Lgb



Re: cvs --with-pspell no go

2000-09-18 Thread Garst R. Reese

Juergen Vigna wrote:
 
 On 16-Sep-2000 Garst R. Reese wrote:
  pspell-.11.2
  aspell-.32.5
  gzipped log attached.
  cvs 1900h ADT Sep 15
  Garst
 
 Fixed as Kevin said (word.c_str())!
 
   Jürgen
 
Did you commit that? I don't see any changes to spellchecker.C in 4PM
cvs update.
Garst



Re: cvs --with-pspell no go

2000-09-18 Thread Lars Gullik Bjønnes

"Garst R. Reese" [EMAIL PROTECTED] writes:

|  Fixed as Kevin said (word.c_str())!
|  
|Jürgen
|  
| Did you commit that? I don't see any changes to spellchecker.C in 4PM
| cvs update.

I'll commit some stuff to fix this right away.

btw. Have you tried the current cvs with xforms 0.89 and C-space?

Lgb



Re: cvs --with-pspell no go

2000-09-18 Thread Garst R. Reese

"Lars Gullik Bjønnes" wrote:
 
 "Garst R. Reese" [EMAIL PROTECTED] writes:
 
 |  Fixed as Kevin said (word.c_str())!
 | 
 |Jürgen
 | 
 | Did you commit that? I don't see any changes to spellchecker.C in 4PM
 | cvs update.
 
 I'll commit some stuff to fix this right away.
Thanks
 btw. Have you tried the current cvs with xforms 0.89 and C-space?
 
 Lgb
No, but I'll do that and let you know in the AM.
Garst



Re: cvs --with-pspell no go

2000-09-18 Thread Garst R. Reese

"Lars Gullik Bjønnes" wrote:
 
 "Garst R. Reese" [EMAIL PROTECTED] writes:
 
 |  Fixed as Kevin said (word.c_str())!
 | 
 |Jürgen
 | 
 | Did you commit that? I don't see any changes to spellchecker.C in 4PM
 | cvs update.
 
 I'll commit some stuff to fix this right away.
 
 btw. Have you tried the current cvs with xforms 0.89 and C-space?
 
 Lgb
\alpha worked, C-space worked, xforms 0.89
compiled --with-pspell
spellchecker put up an error box saying "unable to load the aspell
module."
That's probably a matter of reading the docs, but that sucks. I went to
the aspell docs and get referred to the pspell docs. They are copius and
probably tell me what I need to know, but there should be a default
config. 
Three Thumbs Down. No Smiley.
If I open the Options menu, there is a tab at the far right that starts
with Pri,
but the box isn't big enough to show it all. If I click on the partial
tab,
LyX sigsevs.
Garst



Re: cvs --with-pspell no go

2000-09-18 Thread Kevin Atkinson

On Mon, 18 Sep 2000, Garst R. Reese wrote:

 "Lars Gullik Bjønnes" wrote:
  
  "Garst R. Reese" [EMAIL PROTECTED] writes:
  
  |  Fixed as Kevin said (word.c_str())!
  | 
  |Jürgen
  | 
  | Did you commit that? I don't see any changes to spellchecker.C in 4PM
  | cvs update.
  
  I'll commit some stuff to fix this right away.
  
  btw. Have you tried the current cvs with xforms 0.89 and C-space?
  
  Lgb
 \alpha worked, C-space worked, xforms 0.89
 compiled --with-pspell
 spellchecker put up an error box saying "unable to load the aspell
 module."

That is probably because you don't have Pspell compiled correctly.  If you
are using Linux the easiest thing to do is compile Pspell with
--enable-ltdl.  Otherwise you need to link in the Aspell code with Pspell
like it says in the manual.

--- 
Kevin Atkinson
kevina at users sourceforge net
http://metalab.unc.edu/kevina/




Re: cvs --with-pspell no go

2000-09-18 Thread Garst R. Reese

Kevin Atkinson wrote:
 
 On Mon, 18 Sep 2000, Garst R. Reese wrote:
 
  spellchecker put up an error box saying "unable to load the aspell
  module."
 
 That is probably because you don't have Pspell compiled correctly.  If you
 are using Linux the easiest thing to do is compile Pspell with
 --enable-ltdl.  Otherwise you need to link in the Aspell code with Pspell
 like it says in the manual.
That should be in the top level README.



Re: cvs --with-pspell no go

2000-09-18 Thread Garst R. Reese

Kevin Atkinson wrote:
 
 On Mon, 18 Sep 2000, Garst R. Reese wrote:
 
  "Lars Gullik Bjønnes" wrote:
  
   "Garst R. Reese" [EMAIL PROTECTED] writes:
  spellchecker put up an error box saying "unable to load the aspell
  module."
 
 That is probably because you don't have Pspell compiled correctly.  If you
 are using Linux the easiest thing to do is compile Pspell with
 --enable-ltdl.  Otherwise you need to link in the Aspell code with Pspell
 like it says in the manual.
I recompiled pspell --enable-ltdl. That worked as far as getting aspell
to run,
But how do I pass options to LyX to select the canadian dictionary?
Garst



Re: Making patches: one possible solution.

2000-09-18 Thread Allan Rae

On Mon, 18 Sep 2000, Amir Karger wrote:

 On Mon, Sep 18, 2000 at 02:41:03PM +1000, Allan Rae wrote:
  On Fri, 15 Sep 2000, Amir Karger wrote:
  
   On Fri, Sep 15, 2000 at 01:48:42PM +1000, Allan Rae wrote:

I have modified GNU diff to be able to emulate `cvs diff`.  This allows
anyone with two copies of a cvs snapshot to modify one and generate a
diff between the two that is the equivalent of what they'd get if we gave
them cvs access.
   
   are you sure one couldn't hack
   together a perl script that would use diff as it already stands? 
  
  I tried this with a shell script.  It was ugly and it required me to
  identify which files were present in one directory and not the other
  (something diff already does for directory comparisons) and call diff on
  each individual pair of files like:
  diff -u dir1/file1 /dev/null
 
 Sounds like this worked, even if it was ugly.

Almost worked, but was getting hideously complicated and was duplicating
all the work done by diff.  In fact it was more complicated than diff
because it had to work around diff's shortcomings.

 Still not as ugly as requiring everyone to use a patched version of
 diffutils. (What happens when they upgrade diffultils? What if someone

Diffutils hasn't changed in five years -- at least according to the
ChangeLog.

 else wants them to use a differently patched version?) I guess if you
 sent in your patches to the diffutils folks, that might make more
 sense. And might even be useful since lots of people in the world who
 use diff also want to use it for cvs-diff-like things.

As I mentioned in an earlier post diffutils ChangeLog's last entry was in
1994.  I was hoping for some feedback while I figure out who to submit my
changes to.

  Either way required changes to diffutils sources so I wrote the one that
  made better sense to me.  
 
 I don't understand why the shell script you described required changing
 diffultils.

Okay, once more.  The '-N' option to diff only works when comparing
directories.  I can't compare directories with diff-2.7 and lyx-devel cvs
module because there are some patterns that match files that shouldn't be
excluded.  For example, "acinclude.m4" can be excluded in all directories
except lyx-devel/lib/reLyX/.  There are other patterns that should match
in only one subdirectory (see *.[cCh] discussion below).

Trying to work around the '-N' flag problem by figuring out which files
exist in one directory but not another was just duplicating all the work
already done in diff to build a list of files to diff (or to exclude) and
it added to the complexity.  The script was basically rewriting diff as a
shell script and only using the file-comparison code from diff to generate
the actual patches.  Stupid idea.  So I extended diff with a few lines of
code instead.

 Alternatively, if you're worrying about ignoring the right files, why not
 write a script that creates a file of files to ignore in the diff, and then
 call diff -X? 

Try:
man diff
or
info diff

diff only ignores files whose _basename_ matches a pattern in the exclude
file.  My first patch to diff was to make it possible to have a pattern
like:
*/src/frontends/xforms/forms/*.[Cch]

so that particular files in a particular subdirectory would be excluded.
That's when I thought it'd be more useful to just get diff to use a local
exclude file (local to each subdir: .cvsignore for example).

And before you ask, I did try building a list a patterns to exclude from
the .cvsignore files in my shell script and then tried to add the files
that were incorrectly matched¹.  You'll notice for example that
src/frontends/xforms/forms/.cvsignore contains patterns for *.[Cch] with
the result that all source files in every subdirectory would match.  I
tried four different ways² of building the patch and all had similar
limitations.  Ultimately, I decided diff needed extending to better cope
with these difficulties.

¹ Remember you can't remove a patch that shouldn't have been generated so
you have to err on the side of caution and then figure out which files
should have been included.

² I've just thought of a 5th way that might almost work but it involves
recursing through the directory structure and building exclude patterns to
be given to `diff -N -X` such that you only do one directory depth at any
time (ie. add the subdirectory names to the patterns so that diff won't
enter them).  This is also overly complicated. Extending diff is still the
nicest solution.

Allan. (ARRae)




Re: LyX/Literate programming bug report and fix

2000-09-18 Thread Allan Rae

On Mon, 18 Sep 2000, Juergen Vigna wrote:

 On 18-Sep-2000 Allan Rae wrote:
  
  I'm sure we've had more than occasion where contributors have forgotten
  those new files simply because they expected diff would add the new files
  to the patch for them.  Sure they should have checked their patch and
  noticed this but getting a patch and a tarball is a pain.  Why not get a
  patch from everyone?  The modified diffutils provides a way for that.
  
 
 I never said that I just said that we didn't reject patches with newfiles
 added as tar-ball, as you suggested in an earlier mail!

I didn't mention the "newfiles-in-a-tarball".  In fact you said something
to John about rejecting patches (although that may have simply been
referring to the lack of a ChangeLog) even though the discussion then led
to find a way for him to send a patch and no tarball.

  So we make life easier for some who've proven worthy and others just have
  to struggle.  I wish it was friday then I could leave out the smiley's ;-)
 
 NO, they can use your patched diffutils!!! (When they convice their system
 administrator that they are really the top of developement, secure and VERY
 handy ;)!

Or they can install them in their ~/bin.  256K compressed code and an
installed footprint of around 500KB if all diffutils components (docs,
executables etc.) are installed and are statically linked to the helper
code now shipped in glibc but also included in the diffutils distro (this
is the default and would require a patch to diffutils configure to fix
this).

  I understand that you want publicity for your patched diff-utils!
  
  Gotta start somewhere.
 
 BTW.: Could you provide a RPM for Redhat 6.x? I think I could use them, but
   I'm a bit lazy #:O)

Maybe.  What's wrong with just using the source tree (apart from a
misnamed link?).

Correct links are now at:
http://www.devel.lyx.org/~rae/code/

Allan. (ARRae)




Re: Dialogs only resizeable in one direction?

2000-09-18 Thread Allan Rae

On Sat, 16 Sep 2000, R. Lahaye wrote:

 
 Hi,
 
 I've just downloaded the latest CVS and I noticed
 that many dialogs have a limited "resizeability".
 I wonder wether that is on purpose, or that this
 is a "under-construction" feature.

On purpose.
 
 For example the Citation, Reference, Url etc. dialogs
 can only be resized in X direction. Others are even
 the resizeability disabled!

Most of these gain nothing by proving Y-resizing so we don't allow it.

 For some dialogs, that have a text field, it would be
 useful to stretch the text field with a resize of the
 dialog, while the buttons remain.

Good point.  All we need to do is figure out how.  Actually I think Angus
has worked this out.

Allan. (ARRae)




Re: Dialogs only resizeable in one direction?

2000-09-18 Thread Garst R. Reese

Allan Rae wrote:
 
 On Sat, 16 Sep 2000, R. Lahaye wrote:
 
 
  Hi,
 
  I've just downloaded the latest CVS and I noticed
  that many dialogs have a limited "resizeability".
  I wonder wether that is on purpose, or that this
  is a "under-construction" feature.
 
 On purpose.
 
  For example the Citation, Reference, Url etc. dialogs
  can only be resized in X direction. Others are even
  the resizeability disabled!
 
 Most of these gain nothing by proving Y-resizing so we don't allow it.
 
  For some dialogs, that have a text field, it would be
  useful to stretch the text field with a resize of the
  dialog, while the buttons remain.
 
 Good point.  All we need to do is figure out how.  Actually I think Angus
 has worked this out.
 
 Allan. (ARRae)
The Opions-Preferences dialog is too narrow on 800x600.
The tab that is labeled Pri??? is cutoff. Clicking on the part remaining
causes a
SIGSEV.
Garst



Re: "LyX/Literate programming" bug report and fix

2000-09-18 Thread Allan Rae

On Fri, 15 Sep 2000, Juergen Vigna wrote:

> > 
> > Nothing, as I said above in the first case.  The only problem comes with
> > the fact that new files are not included by the -N.  Only new files that
> > have been added to cvs using `cvs add` are actually caught by the -N flag.
> > To be able to run `cvs add` you have to be given at least read-only access
> > to cvs by Lars.  This just means extra work for Lars and extra hassle for
> > budding contributors just to be able to submit a patch that will be in an
> > acceptable form.  This is a silly requirement IMO as it is overly
> > restrictive.
> 
> Well I guess you didn't get enough sleep last night, isn't it!
> 
> I've never seen that a patch as John sends them with a cvs diff -u and
> the newfiles in a tar-ball are rejected,

I'm sure we've had more than occasion where contributors have forgotten
those new files simply because they expected diff would add the new files
to the patch for them.  Sure they should have checked their patch and
noticed this but getting a patch and a tarball is a pain.  Why not get a
patch from everyone?  The modified diffutils provides a way for that.

> I've only seen requests to Lars
> to add some of the "not fast disapearing developers" to the cvs-readonly
> list so that is is easier for them to do a cvs diff and include also the
> newfiles (and deleted files) in the patch!

So we make life easier for some who've proven worthy and others just have
to struggle.  I wish it was friday then I could leave out the smiley's ;-)

> I understand that you want publicity for your patched diff-utils!

Gotta start somewhere.

Allan. (ARRae)




Re: make dist oversight

2000-09-18 Thread Lars Gullik Bjønnes

"Kayvan A. Sylvan" <[EMAIL PROTECTED]> writes:

| Hi folk,
| 
| This small patch fixes a ``make dist'' oversight (and lets my auto-compile
| and RPM generation proceed). However, even with this, I get the following
| errors on compilation:
| 
| table.C:791: new declaration `class string LyXTable::GetPWidth(int)'
| table.h:173: ambiguates old declaration `const class string
| LyXTable::GetPWidth(

This is strange since table.C and table. are not part of the build
process anymore.

Lgb



Re: make dist oversight

2000-09-18 Thread Juergen Vigna


On 18-Sep-2000 Lars Gullik Bjønnes wrote:
> "Kayvan A. Sylvan" <[EMAIL PROTECTED]> writes:
> 
>| Hi folk,
>| 
>| This small patch fixes a ``make dist'' oversight (and lets my auto-compile
>| and RPM generation proceed). However, even with this, I get the following
>| errors on compilation:
>| 
>| table.C:791: new declaration `class string LyXTable::GetPWidth(int)'
>| table.h:173: ambiguates old declaration `const class string
>| LyXTable::GetPWidth(
> 
> This is strange since table.C and table. are not part of the build
> process anymore.
> 

Well he modified the Makefile and put them in again ;)

I fixed this and IMO they still should be part of the build process
at least until we remove all references to them in the LyX-Source and
remove them from the repository!

  Jürgen

--
-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._

Dr. Jürgen VignaE-Mail:  [EMAIL PROTECTED]
Italienallee 13/N   Tel/Fax: +39-0471-450260 / +39-0471-450253
I-39100 Bozen   Web: http://www.sad.it/~jug

-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._

With your bare hands?!?




Re: patch

2000-09-18 Thread Lars Gullik Bjønnes

Juergen Vigna <[EMAIL PROTECTED]> writes:

| On 15-Sep-2000 John Levon wrote:
| 
| > You are right, I have added the moc files to the make clean target in the
| > attached patch, incremental to my previous one today.
| > 
| 
| IMO moc files should only be removed on a distclean not on a simple
| clean, but if you insist that it should be done also on a make clean
| then I'll apply this patch!

distclean is certainly the right target for this, not clean.

| BTW.: I got a bit out of sync with KDE/GNOME patches as I had to wait
|   for Lars to have a look at one of them, could you please tell me
|   (John & Marko) which ones haven't been aplied?

Yes, please. I lost a bit of the oversight recently.

Lgb



Re: make dist oversight

2000-09-18 Thread Lars Gullik Bjønnes

Juergen Vigna <[EMAIL PROTECTED]> writes:

| On 18-Sep-2000 Lars Gullik Bjønnes wrote:
| > "Kayvan A. Sylvan" <[EMAIL PROTECTED]> writes:
| > 
| >| Hi folk,
| >| 
| >| This small patch fixes a ``make dist'' oversight (and lets my auto-compile
| >| and RPM generation proceed). However, even with this, I get the following
| >| errors on compilation:
| >| 
| >| table.C:791: new declaration `class string LyXTable::GetPWidth(int)'
| >| table.h:173: ambiguates old declaration `const class string
| >| LyXTable::GetPWidth(
| > 
| > This is strange since table.C and table. are not part of the build
| > process anymore.
| > 
| 
| Well he modified the Makefile and put them in again ;)
| 
| I fixed this and IMO they still should be part of the build process
| at least until we remove all references to them in the LyX-Source and
| remove them from the repository!

The only reason why I haven't done that already is that you might need
them a bit further. If you don't need those files I will remove them.

Lgb



Re: patch

2000-09-18 Thread Marko Vendelin



On Mon, 18 Sep 2000, Juergen Vigna wrote:

> BTW.: I got a bit out of sync with KDE/GNOME patches as I had to wait
>   for Lars to have a look at one of them, could you please tell me
>   (John & Marko) which ones haven't been aplied?

Gnome patches are

http://www.mail-archive.com/lyx-devel%40lists.lyx.org/msg14188.html
http://www.mail-archive.com/lyx-devel%40lists.lyx.org/msg14247.html

They both depend on TOC lists tracking implemented in the first patch.
These patches were results of the discussion on the GUI update policy of
LyX. Note, that in the present form the first patch may slow down LyX
considerably as indicated in the e-mail.

All other patches (Gnome frontend) were applied.

Marko




math-mode feature or bug?

2000-09-18 Thread Marko Vendelin


Hi!

I would like to report the following bug in the mathmode:

1. create a new document (File/New)
2. open math-mode by clicking on the toolbar or by selecting Math/Math
   Mode in menu
3. type "\alpha " and enjoy LyX dumping a core.

Marko

PS: I hope it is not only in my build (Gnome frontend)




[f95ts@efd.lth.se] [Lyx-feedback] Feedback from www.lyx.org

2000-09-18 Thread Lars Gullik Bjønnes



Tommy Sjögren ([EMAIL PROTECTED]) entered the 
following feedback message on the LyX home page:


Hi there,

I\'m trying to compile LyX, but I don\'t have Perl on my system, I guess
this is the main problemanyway
I thought it wasn\'t necessary to have Perl in order to comile the sourece
configure works fine, but when make enters relyx it doesn\'t find a
Makefile, wich is true as there isn\'t a Makefile.
I\'ve tried with \"export PERL=/bin/ls\" but didn\'t work either.

Do have to have Perl installed in order to compile?

Regards
Tommy
___
Lyx-feedback mailing list
[EMAIL PROTECTED]
http://www.lyx.org/mailman/listinfo/lyx-feedback





Re: make dist oversight

2000-09-18 Thread Juergen Vigna


On 18-Sep-2000 Lars Gullik Bjønnes wrote:

>| I fixed this and IMO they still should be part of the build process
>| at least until we remove all references to them in the LyX-Source and
>| remove them from the repository!
> 
> The only reason why I haven't done that already is that you might need
> them a bit further. If you don't need those files I will remove them.

I don't need them anymore (if I do I always could download them from
the cvs repository in an old version, couldn't I?

But what I meant is that before removing this files we have to remove
ALL NEW_TABULAR defines in the sources so that LyXTable is not mentioned
anywhere.

  Jürgen

--
-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._

Dr. Jürgen VignaE-Mail:  [EMAIL PROTECTED]
Italienallee 13/N   Tel/Fax: +39-0471-450260 / +39-0471-450253
I-39100 Bozen   Web: http://www.sad.it/~jug

-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._

Love is like the measles; we all have to go through it.
-- Jerome K. Jerome




Re: patch

2000-09-18 Thread Juergen Vigna


On 18-Sep-2000 Lars Gullik Bjønnes wrote:

>| http://www.mail-archive.com/lyx-devel%40lists.lyx.org/msg14247.html
> 
> I have not looked at this.

Well as this one depends on the first one IMO we cannot include this
too.

   Jürgen

--
-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._

Dr. Jürgen VignaE-Mail:  [EMAIL PROTECTED]
Italienallee 13/N   Tel/Fax: +39-0471-450260 / +39-0471-450253
I-39100 Bozen   Web: http://www.sad.it/~jug

-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._

Ours is a world of nuclear giants and ethical infants.
-- General Omar N. Bradley




Re: make dist oversight

2000-09-18 Thread Lars Gullik Bjønnes

Juergen Vigna <[EMAIL PROTECTED]> writes:

| On 18-Sep-2000 Lars Gullik Bjønnes wrote:
| 
| >| I fixed this and IMO they still should be part of the build process
| >| at least until we remove all references to them in the LyX-Source and
| >| remove them from the repository!
| > 
| > The only reason why I haven't done that already is that you might need
| > them a bit further. If you don't need those files I will remove them.
| 
| I don't need them anymore (if I do I always could download them from
| the cvs repository in an old version, couldn't I?
| 
| But what I meant is that before removing this files we have to remove
| ALL NEW_TABULAR defines in the sources so that LyXTable is not mentioned
| anywhere.

Sure... it is ok for you if I do this now?
And also remove table.C and table.h?

Lgb



Re: make dist oversight

2000-09-18 Thread Juergen Vigna


On 18-Sep-2000 Lars Gullik Bjønnes wrote:
> 
> Sure... it is ok for you if I do this now?
> And also remove table.C and table.h?
> 

I guess yes, but IMO we should ask the others.

We have seen that there ARE problems with the new tabular inset
(especially with the text-inset) so before removing the old code
and so definitively remove the old table-handling-code we should
also open our mind that this means making the textinset feature
complete (IMO this is not that much work, but all has to be tested!)

At least we have to redo the paragraph-layout and disable stuff
there if we are inside a text inset!

I need help for this as otherwise it might take some time!

I'm just now trying to fix Dekel's reported 'missing features' :)

 Jürgen

--
-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._

Dr. Jürgen VignaE-Mail:  [EMAIL PROTECTED]
Italienallee 13/N   Tel/Fax: +39-0471-450260 / +39-0471-450253
I-39100 Bozen   Web: http://www.sad.it/~jug

-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._

A straw vote only shows which way the hot air blows.
-- O'Henry




Re: patch

2000-09-18 Thread John Levon

On Mon, 18 Sep 2000, Juergen Vigna wrote:

> 
> On 15-Sep-2000 John Levon wrote:
> 
> > You are right, I have added the moc files to the make clean target in the
> > attached patch, incremental to my previous one today.
> > 
> 
> IMO moc files should only be removed on a distclean not on a simple
> clean, but if you insist that it should be done also on a make clean
> then I'll apply this patch!
>

I have no strong feelings either way, so it should be left as it is.
 
> BTW.: I got a bit out of sync with KDE/GNOME patches as I had to wait
>   for Lars to have a look at one of them, could you please tell me
>   (John & Marko) which ones haven't been aplied?
> 
>   Jürgen
> 

AFAICR I have no pending patches as Lars applied my last one

thanks
john

-- 
"Tragedy is if I cut my finger; comedy is if you walk into
 an open sewer and die."
- Mel Brooks





Re: make dist oversight

2000-09-18 Thread Juergen Vigna


On 18-Sep-2000 Juergen Vigna wrote:
> 
> I need help for this as otherwise it might take some time!
> 

Replying to myself ;)

There are a lot of functions in lyxfunc which modify or do something
with LyXText (bv->text!). Now all this functions should be handled
also by the text-inset. I now would need a nice way to do this for
all of this functions but without the need to copy their behaviour
into InsetText::LocalDispatch(...) so that 1 codebase could be used
for this functions.

One idea would be that we group all this functions so that we know
what they are and then make a LocalDispatch for them in LyXText.

A second is to copy the behaviour into InsetText::LocalDispatch(...)

What should I do?

  Jürgen

--
-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._

Dr. Jürgen VignaE-Mail:  [EMAIL PROTECTED]
Italienallee 13/N   Tel/Fax: +39-0471-450260 / +39-0471-450253
I-39100 Bozen   Web: http://www.sad.it/~jug

-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._-._

If all men were brothers, would you let one marry your sister?




Re: patch

2000-09-18 Thread Marko Vendelin



On 18 Sep 2000, Lars Gullik Bjønnes wrote:

> Marko Vendelin <[EMAIL PROTECTED]> writes:
> 
> | On Mon, 18 Sep 2000, Juergen Vigna wrote:
> | 
> | > BTW.: I got a bit out of sync with KDE/GNOME patches as I had to wait
> | >   for Lars to have a look at one of them, could you please tell me
> | >   (John & Marko) which ones haven't been aplied?
> | 
> | Gnome patches are
> | 
> | http://www.mail-archive.com/lyx-devel%40lists.lyx.org/msg14188.html
> 
> The idea is good, but the current way to update the lists is not good
> at all. I tink we will let this be suboptimal for the time beeing and
> leave everything as it is.

What do you mean by "suboptimal"? Is it the basic idea of updating the
list(s) by using buffer()->getTocList() (that means parsing LyX document
for headings etc.) or that it is done after every action and not
during "idle" time?

> Later we (or atleast I want to) have separate insets for headings,
> then it will be easy to emit the signals directly from the buffer when
> headings etc. change.

That would be nice. It should be the same for figures, tables, algorithms
and references too.

Marko




Re: patch

2000-09-18 Thread Lars Gullik Bjønnes

Marko Vendelin <[EMAIL PROTECTED]> writes:

| On 18 Sep 2000, Lars Gullik Bjønnes wrote:
| 
| > Marko Vendelin <[EMAIL PROTECTED]> writes:
| > 
| > | On Mon, 18 Sep 2000, Juergen Vigna wrote:
| > | 
| > | > BTW.: I got a bit out of sync with KDE/GNOME patches as I had to wait
| > | >   for Lars to have a look at one of them, could you please tell me
| > | >   (John & Marko) which ones haven't been aplied?
| > | 
| > | Gnome patches are
| > | 
| > | http://www.mail-archive.com/lyx-devel%40lists.lyx.org/msg14188.html
| > 
| > The idea is good, but the current way to update the lists is not good
| > at all. I tink we will let this be suboptimal for the time beeing and
| > leave everything as it is.
| 
| What do you mean by "suboptimal"? Is it the basic idea of updating the
| list(s) by using buffer()->getTocList() (that means parsing LyX document
| for headings etc.) or that it is done after every action and not
| during "idle" time?

More tat the support in gnome/kde will not be perfect from the
beginning.

| 
| > Later we (or atleast I want to) have separate insets for headings,
| > then it will be easy to emit the signals directly from the buffer when
| > headings etc. change.
| 
| That would be nice. It should be the same for figures, tables, algorithms
| and references too.

Yes, that is the idea.

Lgb



Re: Heretics and Perverts [was: Re: I'm back, take 2 :)

2000-09-18 Thread Amir Karger

On Fri, Sep 15, 2000 at 01:13:22PM -0400, [EMAIL PROTECTED] wrote:
> 
> 
> the vi pervert Andre audiated
> 
> >The problem is that there are quite a few things missing from the
> >LyX kernel to make such a thing viable. I don't think it is possible
> >at all to simulate vi's modes by fiddling with keybindings only.
> 
> I think we'd need to run from the mini-buffer to avoid an 
> escape-sequence for every command . . . hey, wait, there's only one 
> mode in vi.  Insert is a command, not a mode (at least until vim 
> blurred the distinciton . . .

Huh? vi has insert mode! It's entered by commands like i,a,o,s,c, and their
corresponding capitals. I'm not sure whether vi admits to replace mode being
separate from insert mode, though.

> >I believe Dvorak found that two hands is *better* than one. 
> 
> But dvorak has mostly been debunked.  His studies were run by a 
> relative of the person trying to sell the keyboards.

Except that the debunking of dvorak has mostly been debunked.

http://www.mwbrooks.com/dvorak/dissent.html

And, btw, even if you don't believe that web page, I'll bet that many
ergonomics folks would still agree with the statement that two hands are
better than one.

> I've also used vi on a number of occasions to edit lyx files (and emacs 
> a couple of times, too--I still haven't figured out how to include line 
> breaks in vi search/replace . . .)

C-v C-m

In general, you can use ^V to quote unquotables, like Esc. I think in Vim,
the ^V'ing is a bit more user friendly, but IIRC, it works just fine in
vanilla vi.

-Amir



slides-only in slide class

2000-09-18 Thread hawk



Has anyone tried to use slides-only or notes-only in the slides class?  
I've tried a few times, but never succeeded in getting it to work.

The errors are "undefined control sequence"  Since I can'tselect from 
the lyx merror windows, I've exported to latex:

...
(/usr/share/texmf/tex/latex/config/geometry.cfg))
! Undefined control sequence.
 Only Slides \slides 

l.51 \onlyslides{\slides}
 
? 
Only Slides 
No file random__variables.aux.
! Undefined control sequence.
\@doglslidelist ->\slides 
  ,99,99
l.55 \begin{document}
 
? 
--- Geometry parameters
mode: letterpaper 
h-parts: 0.75in, 505.89pt, 0.75in
v-parts: 0.75in, 686.56499pt, 0.75in
--- Page layout dimensions
\paperwidth  614.295pt
\paperheight 794.96999pt
\textwidth  505.89pt
\textheight 632.56499pt
\oddsidemargin  -18.06749pt
\evensidemargin -18.06749pt
\topmargin  -18.06749pt
\headheight 14.0pt
\headsep15.0pt
\footskip   25.0pt
\hoffset 0.0pt
\voffset 0.0pt
(1in=72.27pt, 1cm=28.45pt)
---
! Missing number, treated as zero.
 
   \relax 
l.58 \lyxnewslide{}
   
?  
)
Runaway argument?
\fi  
! File ended while scanning use of \@ggobtoend.
 
\par 
<*> random__variables.tex
 
? 


hawk

-- 





[gfox@chuma1.cas.usf.edu] [Lyx-feedback] Feedback from www.lyx.org

2000-09-18 Thread Lars Gullik Bjønnes



Gordon Fox ([EMAIL PROTECTED]) entered the 
following feedback message on the LyX home page:


Importing some LaTeX files fails on my machine, and I\'m not quite sure why.

The messages I get are:
 reLyX, the LaTeX to LyX translator. Revision date 2000/06/06

Reading LaTeX command syntax
(quiz1.tex: Splitting Preamble
Creating LyX preamble
Uncommented text before \\documentclass command ignored!
Use of uninitialized value at /usr/local/bin/../share/lyx/reLyX/MakePreamble.pm
line 182,  chunk 17.
Use of uninitialized value at /usr/local/bin/../share/lyx/reLyX/MakePreamble.pm
line 210,  chunk 17.
Cannot find layout file 2000/06/06.layout in dir(s) /home/gfox/.lyx/layouts at /
usr/local/bin/../share/lyx/reLyX/ReadCommands.pm line 267,  chunk 17.
Exited due to fatal Error!   

I\'m not really sure why it\'s looking for a file with this name -- 
obviously something isn\'t set up properly. Any suggestions as to
how to fix this?

Thanks.

Gordon Fox
___
Lyx-feedback mailing list
[EMAIL PROTECTED]
http://www.lyx.org/mailman/listinfo/lyx-feedback





Re: math-mode feature or bug?

2000-09-18 Thread Garst R. Reese

Marko Vendelin wrote:
> 
> Hi!
> 
> I would like to report the following bug in the mathmode:
> 
> 1. create a new document (File/New)
> 2. open math-mode by clicking on the toolbar or by selecting Math/Math
>Mode in menu
> 3. type "\alpha " and enjoy LyX dumping a core.
> 
> Marko
> 
> PS: I hope it is not only in my build (Gnome frontend)
It's not. xforms does the same.
Garst



Re: Making patches: one possible solution.

2000-09-18 Thread Amir Karger

On Mon, Sep 18, 2000 at 02:41:03PM +1000, Allan Rae wrote:
> On Fri, 15 Sep 2000, Amir Karger wrote:
> 
> > On Fri, Sep 15, 2000 at 01:48:42PM +1000, Allan Rae wrote:
> > > 
> > > I have modified GNU diff to be able to emulate `cvs diff`.  This allows
> > > anyone with two copies of a cvs snapshot to modify one and generate a
> > > diff between the two that is the equivalent of what they'd get if we gave
> > > them cvs access.
> > 
> > are you sure one couldn't hack
> > together a perl script that would use diff as it already stands? 
> 
> I tried this with a shell script.  It was ugly and it required me to
> identify which files were present in one directory and not the other
> (something diff already does for directory comparisons) and call diff on
> each individual pair of files like:
>   diff -u dir1/file1 /dev/null

Sounds like this worked, even if it was ugly. Still not as ugly as requiring
everyone to use a patched version of diffutils. (What happens when they
upgrade diffultils? What if someone else wants them to use a differently
patched version?) I guess if you sent in your patches to the diffutils
folks, that might make more sense. And might even be useful since lots of
people in the world who use diff also want to use it for cvs-diff-like
things.

> Either way required changes to diffutils sources so I wrote the one that
> made better sense to me.  

I don't understand why the shell script you described required changing
diffultils.

Alternatively, if you're worrying about ignoring the right files, why not
write a script that creates a file of files to ignore in the diff, and then
call diff -X? 

-Amir



Re: Making patches: one possible solution.

2000-09-18 Thread John Levon

On Mon, 18 Sep 2000, Amir Karger wrote:

> I don't understand why the shell script you described required changing
> diffultils.
> 
> Alternatively, if you're worrying about ignoring the right files, why not
> write a script that creates a file of files to ignore in the diff, and then
> call diff -X? 
> 
> -Amir
> 

I have a prototype dontdiff file for LyX at home if anyone wants it ...

it's not complete though

john

-- 
"Tragedy is if I cut my finger; comedy is if you walk into
 an open sewer and die."
- Mel Brooks




Re: [gfox@chuma1.cas.usf.edu] [Lyx-feedback] Feedback from www.lyx.org

2000-09-18 Thread Amir Karger

Hi.

On Mon, Sep 18, 2000 at 05:49:19PM +0200, Lars Gullik Bjønnes wrote:
> 
> Gordon Fox ([EMAIL PROTECTED]) entered the 
> following feedback message on the LyX home page:
> 
> 
> Reading LaTeX command syntax
> (quiz1.tex: Splitting Preamble
> Creating LyX preamble
> Uncommented text before \\documentclass command ignored!
> Use of uninitialized value at /usr/local/bin/../share/lyx/reLyX/MakePreamble.pm
> line 182,  chunk 17.
> Use of uninitialized value at /usr/local/bin/../share/lyx/reLyX/MakePreamble.pm
> line 210,  chunk 17.
> Cannot find layout file 2000/06/06.layout in dir(s) /home/gfox/.lyx/layouts at /
> usr/local/bin/../share/lyx/reLyX/ReadCommands.pm line 267,  chunk 17.
> Exited due to fatal Error!   
> 
> I\'m not really sure why it\'s looking for a file with this name -- 
> obviously something isn\'t set up properly. Any suggestions as to
> how to fix this?

I agree that something weird is going on.

Could you send a small file that exhibits this error to
[EMAIL PROTECTED]? (In fact, it looks like it's just the beginning of
your file that's confusing reLyX; it never even gets to most of the
translating part.)

-Amir Karger



Re: math-mode feature or bug?

2000-09-18 Thread Lior Silberman

On Mon, 18 Sep 2000, Marko Vendelin wrote:

> 
> Hi!
> 
> I would like to report the following bug in the mathmode:
> 
> 1. create a new document (File/New)
> 2. open math-mode by clicking on the toolbar or by selecting Math/Math
>Mode in menu
> 3. type "\alpha " and enjoy LyX dumping a core.
> 
> Marko
> 
> PS: I hope it is not only in my build (Gnome frontend)
> 
> 

This also happens in xforms. The culprit appears to be the following line
(line 787 of mathed/math_cursor.C):

imacro->SetName(0);

which is:

void MathedInset::SetName(string const & n) { name = n; }
[with: string name].

This activates basic_string's copy constructor with a null reference, gets
to a 'strlen(0)' (in /usr/include/g++-2/std/straits.h:125), and hence the
SEGV.

I think this should say one of:
imacro->SetName(string());
imacro->SetName(string(""));
imacro->SetName("");

but I don't know anything about the string template, so I don't know which
one is correct. The first one does solve the problem (tested). Is there a
global 'emptystring' instance?

Lior.




Re: math-mode feature or bug?

2000-09-18 Thread Lior Silberman

On Mon, 18 Sep 2000, Lior Silberman wrote:

> On Mon, 18 Sep 2000, Marko Vendelin wrote:
> 
> > 
> > Hi!
> > 
> > I would like to report the following bug in the mathmode:
> > 
> > 1. create a new document (File/New)
> > 2. open math-mode by clicking on the toolbar or by selecting Math/Math
> >Mode in menu
> > 3. type "\alpha " and enjoy LyX dumping a core.
> > 
> > Marko
> > 
> > PS: I hope it is not only in my build (Gnome frontend)
> > 
> > 
> 
> I think this should say one of:
>   imacro->SetName(string());
>   imacro->SetName(string(""));
>   imacro->SetName("");
> 
> but I don't know anything about the string template, so I don't know which
> one is correct. The first one does solve the problem (tested). Is there a
> global 'emptystring' instance?
> 
> Lior.
> 
> 

I must say didn't delve into the mathed, so this may break something, but
it seems to me that using the null pointer as a reference is wrong in most
cases.

Lior.




Re: math-mode feature or bug?

2000-09-18 Thread Lars Gullik Bjønnes

Lior Silberman <[EMAIL PROTECTED]> writes:

| This also happens in xforms. The culprit appears to be the following line
| (line 787 of mathed/math_cursor.C):
| 
|   imacro->SetName(0);

I really wonder why gcc accepts that.

| which is:
| 
| void MathedInset::SetName(string const & n) { name = n; }
| [with: string name].
| 
| This activates basic_string's copy constructor with a null reference, gets
| to a 'strlen(0)' (in /usr/include/g++-2/std/straits.h:125), and hence the
| SEGV.
| 
| I think this should say one of:
|   imacro->SetName(string());
|   imacro->SetName(string(""));
|   imacro->SetName("");

Either of the above should work.
(First or last looks best)

| but I don't know anything about the string template, so I don't know which
| one is correct. The first one does solve the problem (tested). Is there a
| global 'emptystring' instance?

Yes. Inside the std::string so you never have to worry about that.
(and if even if std::string does not have a shared empty string: do
not care about issues like that.)

Lgb




Re: math-mode feature or bug?

2000-09-18 Thread Lars Gullik Bjønnes

Lior Silberman <[EMAIL PROTECTED]> writes:

| I meant to ask whether some version of imacro->SetName(emptystring);
| is the correct syntax. Anyway: the following implements the first
| option.

I already did it with the last option. (+ another memory leak fix)

Lgb



InsetTabular tasks

2000-09-18 Thread Lars Gullik Bjønnes


Jürgen, can you make a task list on what needs to be fixed with
tabular?

Lgb



Re: cvs --with-pspell no go

2000-09-18 Thread Garst R. Reese

Juergen Vigna wrote:
> 
> On 16-Sep-2000 Garst R. Reese wrote:
> > pspell-.11.2
> > aspell-.32.5
> > gzipped log attached.
> > cvs 1900h ADT Sep 15
> > Garst
> 
> Fixed as Kevin said (word.c_str())!
> 
>   Jürgen
> 
Did you commit that? I don't see any changes to spellchecker.C in 4PM
cvs update.
Garst



Re: cvs --with-pspell no go

2000-09-18 Thread Lars Gullik Bjønnes

"Garst R. Reese" <[EMAIL PROTECTED]> writes:

| > Fixed as Kevin said (word.c_str())!
| > 
| >   Jürgen
| > 
| Did you commit that? I don't see any changes to spellchecker.C in 4PM
| cvs update.

I'll commit some stuff to fix this right away.

btw. Have you tried the current cvs with xforms 0.89 and C-space?

Lgb



Re: cvs --with-pspell no go

2000-09-18 Thread Garst R. Reese

"Lars Gullik Bjønnes" wrote:
> 
> "Garst R. Reese" <[EMAIL PROTECTED]> writes:
> 
> | > Fixed as Kevin said (word.c_str())!
> | >
> | >   Jürgen
> | >
> | Did you commit that? I don't see any changes to spellchecker.C in 4PM
> | cvs update.
> 
> I'll commit some stuff to fix this right away.
Thanks
> btw. Have you tried the current cvs with xforms 0.89 and C-space?
> 
> Lgb
No, but I'll do that and let you know in the AM.
Garst



Re: cvs --with-pspell no go

2000-09-18 Thread Garst R. Reese

"Lars Gullik Bjønnes" wrote:
> 
> "Garst R. Reese" <[EMAIL PROTECTED]> writes:
> 
> | > Fixed as Kevin said (word.c_str())!
> | >
> | >   Jürgen
> | >
> | Did you commit that? I don't see any changes to spellchecker.C in 4PM
> | cvs update.
> 
> I'll commit some stuff to fix this right away.
> 
> btw. Have you tried the current cvs with xforms 0.89 and C-space?
> 
> Lgb
\alpha worked, C-space worked, xforms 0.89
compiled --with-pspell
spellchecker put up an error box saying "unable to load the aspell
module."
That's probably a matter of reading the docs, but that sucks. I went to
the aspell docs and get referred to the pspell docs. They are copius and
probably tell me what I need to know, but there should be a default
config. 
Three Thumbs Down. No Smiley.
If I open the Options menu, there is a tab at the far right that starts
with Pri,
but the box isn't big enough to show it all. If I click on the partial
tab,
LyX sigsevs.
Garst



Re: cvs --with-pspell no go

2000-09-18 Thread Kevin Atkinson

On Mon, 18 Sep 2000, Garst R. Reese wrote:

> "Lars Gullik Bjønnes" wrote:
> > 
> > "Garst R. Reese" <[EMAIL PROTECTED]> writes:
> > 
> > | > Fixed as Kevin said (word.c_str())!
> > | >
> > | >   Jürgen
> > | >
> > | Did you commit that? I don't see any changes to spellchecker.C in 4PM
> > | cvs update.
> > 
> > I'll commit some stuff to fix this right away.
> > 
> > btw. Have you tried the current cvs with xforms 0.89 and C-space?
> > 
> > Lgb
> \alpha worked, C-space worked, xforms 0.89
> compiled --with-pspell
> spellchecker put up an error box saying "unable to load the aspell
> module."

That is probably because you don't have Pspell compiled correctly.  If you
are using Linux the easiest thing to do is compile Pspell with
--enable-ltdl.  Otherwise you need to link in the Aspell code with Pspell
like it says in the manual.

--- 
Kevin Atkinson
kevina at users sourceforge net
http://metalab.unc.edu/kevina/




Re: cvs --with-pspell no go

2000-09-18 Thread Garst R. Reese

Kevin Atkinson wrote:
> 
> On Mon, 18 Sep 2000, Garst R. Reese wrote:
> 
> > spellchecker put up an error box saying "unable to load the aspell
> > module."
> 
> That is probably because you don't have Pspell compiled correctly.  If you
> are using Linux the easiest thing to do is compile Pspell with
> --enable-ltdl.  Otherwise you need to link in the Aspell code with Pspell
> like it says in the manual.
That should be in the top level README.



Re: cvs --with-pspell no go

2000-09-18 Thread Garst R. Reese

Kevin Atkinson wrote:
> 
> On Mon, 18 Sep 2000, Garst R. Reese wrote:
> 
> > "Lars Gullik Bjønnes" wrote:
> > >
> > > "Garst R. Reese" <[EMAIL PROTECTED]> writes:
> > spellchecker put up an error box saying "unable to load the aspell
> > module."
> 
> That is probably because you don't have Pspell compiled correctly.  If you
> are using Linux the easiest thing to do is compile Pspell with
> --enable-ltdl.  Otherwise you need to link in the Aspell code with Pspell
> like it says in the manual.
I recompiled pspell --enable-ltdl. That worked as far as getting aspell
to run,
But how do I pass options to LyX to select the canadian dictionary?
Garst



Re: Making patches: one possible solution.

2000-09-18 Thread Allan Rae

On Mon, 18 Sep 2000, Amir Karger wrote:

> On Mon, Sep 18, 2000 at 02:41:03PM +1000, Allan Rae wrote:
> > On Fri, 15 Sep 2000, Amir Karger wrote:
> > 
> > > On Fri, Sep 15, 2000 at 01:48:42PM +1000, Allan Rae wrote:
> > > > 
> > > > I have modified GNU diff to be able to emulate `cvs diff`.  This allows
> > > > anyone with two copies of a cvs snapshot to modify one and generate a
> > > > diff between the two that is the equivalent of what they'd get if we gave
> > > > them cvs access.
> > > 
> > > are you sure one couldn't hack
> > > together a perl script that would use diff as it already stands? 
> > 
> > I tried this with a shell script.  It was ugly and it required me to
> > identify which files were present in one directory and not the other
> > (something diff already does for directory comparisons) and call diff on
> > each individual pair of files like:
> > diff -u dir1/file1 /dev/null
> 
> Sounds like this worked, even if it was ugly.

Almost worked, but was getting hideously complicated and was duplicating
all the work done by diff.  In fact it was more complicated than diff
because it had to work around diff's shortcomings.

> Still not as ugly as requiring everyone to use a patched version of
> diffutils. (What happens when they upgrade diffultils? What if someone

Diffutils hasn't changed in five years -- at least according to the
ChangeLog.

> else wants them to use a differently patched version?) I guess if you
> sent in your patches to the diffutils folks, that might make more
> sense. And might even be useful since lots of people in the world who
> use diff also want to use it for cvs-diff-like things.

As I mentioned in an earlier post diffutils ChangeLog's last entry was in
1994.  I was hoping for some feedback while I figure out who to submit my
changes to.

> > Either way required changes to diffutils sources so I wrote the one that
> > made better sense to me.  
> 
> I don't understand why the shell script you described required changing
> diffultils.

Okay, once more.  The '-N' option to diff only works when comparing
directories.  I can't compare directories with diff-2.7 and lyx-devel cvs
module because there are some patterns that match files that shouldn't be
excluded.  For example, "acinclude.m4" can be excluded in all directories
except lyx-devel/lib/reLyX/.  There are other patterns that should match
in only one subdirectory (see *.[cCh] discussion below).

Trying to work around the '-N' flag problem by figuring out which files
exist in one directory but not another was just duplicating all the work
already done in diff to build a list of files to diff (or to exclude) and
it added to the complexity.  The script was basically rewriting diff as a
shell script and only using the file-comparison code from diff to generate
the actual patches.  Stupid idea.  So I extended diff with a few lines of
code instead.

> Alternatively, if you're worrying about ignoring the right files, why not
> write a script that creates a file of files to ignore in the diff, and then
> call diff -X? 

Try:
man diff
or
info diff

diff only ignores files whose _basename_ matches a pattern in the exclude
file.  My first patch to diff was to make it possible to have a pattern
like:
*/src/frontends/xforms/forms/*.[Cch]

so that particular files in a particular subdirectory would be excluded.
That's when I thought it'd be more useful to just get diff to use a local
exclude file (local to each subdir: .cvsignore for example).

And before you ask, I did try building a list a patterns to exclude from
the .cvsignore files in my shell script and then tried to add the files
that were incorrectly matched¹.  You'll notice for example that
src/frontends/xforms/forms/.cvsignore contains patterns for *.[Cch] with
the result that all source files in every subdirectory would match.  I
tried four different ways² of building the patch and all had similar
limitations.  Ultimately, I decided diff needed extending to better cope
with these difficulties.

¹ Remember you can't remove a patch that shouldn't have been generated so
you have to err on the side of caution and then figure out which files
should have been included.

² I've just thought of a 5th way that might almost work but it involves
recursing through the directory structure and building exclude patterns to
be given to `diff -N -X` such that you only do one directory depth at any
time (ie. add the subdirectory names to the patterns so that diff won't
enter them).  This is also overly complicated. Extending diff is still the
nicest solution.

Allan. (ARRae)




Re: "LyX/Literate programming" bug report and fix

2000-09-18 Thread Allan Rae

On Mon, 18 Sep 2000, Juergen Vigna wrote:

> On 18-Sep-2000 Allan Rae wrote:
> > 
> > I'm sure we've had more than occasion where contributors have forgotten
> > those new files simply because they expected diff would add the new files
> > to the patch for them.  Sure they should have checked their patch and
> > noticed this but getting a patch and a tarball is a pain.  Why not get a
> > patch from everyone?  The modified diffutils provides a way for that.
> > 
> 
> I never said that I just said that we didn't reject patches with newfiles
> added as tar-ball, as you suggested in an earlier mail!

I didn't mention the "newfiles-in-a-tarball".  In fact you said something
to John about rejecting patches (although that may have simply been
referring to the lack of a ChangeLog) even though the discussion then led
to find a way for him to send a patch and no tarball.

> > So we make life easier for some who've proven worthy and others just have
> > to struggle.  I wish it was friday then I could leave out the smiley's ;-)
> 
> NO, they can use your patched diffutils!!! (When they convice their system
> administrator that they are really the top of developement, secure and VERY
> handy ;)!

Or they can install them in their ~/bin.  256K compressed code and an
installed footprint of around 500KB if all diffutils components (docs,
executables etc.) are installed and are statically linked to the helper
code now shipped in glibc but also included in the diffutils distro (this
is the default and would require a patch to diffutils configure to fix
this).

> >> I understand that you want publicity for your patched diff-utils!
> > 
> > Gotta start somewhere.
> 
> BTW.: Could you provide a RPM for Redhat 6.x? I think I could use them, but
>   I'm a bit lazy #:O)

Maybe.  What's wrong with just using the source tree (apart from a
misnamed link?).

Correct links are now at:
http://www.devel.lyx.org/~rae/code/

Allan. (ARRae)




Re: Dialogs only resizeable in one direction?

2000-09-18 Thread Allan Rae

On Sat, 16 Sep 2000, R. Lahaye wrote:

> 
> Hi,
> 
> I've just downloaded the latest CVS and I noticed
> that many dialogs have a limited "resizeability".
> I wonder wether that is on purpose, or that this
> is a "under-construction" feature.

On purpose.
 
> For example the Citation, Reference, Url etc. dialogs
> can only be resized in X direction. Others are even
> the resizeability disabled!

Most of these gain nothing by proving Y-resizing so we don't allow it.

> For some dialogs, that have a text field, it would be
> useful to stretch the text field with a resize of the
> dialog, while the buttons remain.

Good point.  All we need to do is figure out how.  Actually I think Angus
has worked this out.

Allan. (ARRae)




Re: Dialogs only resizeable in one direction?

2000-09-18 Thread Garst R. Reese

Allan Rae wrote:
> 
> On Sat, 16 Sep 2000, R. Lahaye wrote:
> 
> >
> > Hi,
> >
> > I've just downloaded the latest CVS and I noticed
> > that many dialogs have a limited "resizeability".
> > I wonder wether that is on purpose, or that this
> > is a "under-construction" feature.
> 
> On purpose.
> 
> > For example the Citation, Reference, Url etc. dialogs
> > can only be resized in X direction. Others are even
> > the resizeability disabled!
> 
> Most of these gain nothing by proving Y-resizing so we don't allow it.
> 
> > For some dialogs, that have a text field, it would be
> > useful to stretch the text field with a resize of the
> > dialog, while the buttons remain.
> 
> Good point.  All we need to do is figure out how.  Actually I think Angus
> has worked this out.
> 
> Allan. (ARRae)
The Opions->Preferences dialog is too narrow on 800x600.
The tab that is labeled Pri??? is cutoff. Clicking on the part remaining
causes a
SIGSEV.
Garst