SIGSEGV when copy/pasting a child doc

2018-04-23 Thread Scott Kostyshak
I can reproduce on master and on 2.3.x. To reproduce: 1. download the attached 3 documents and put them in the same directory. 2. open master_copy_from.lyx in one LyX instance 3. in the dialog that comes up, click "cancel" 4. open master_copy_to.lyx in a *separate* LyX instance 5. highlight the c

Re: [LyX/master] Add basic support for cprotect

2018-04-23 Thread Scott Kostyshak
On Tue, Apr 24, 2018 at 01:55:31AM +, Scott Kostyshak wrote: > On Mon, Apr 23, 2018 at 02:56:53PM +, Jürgen Spitzmüller wrote: > > 2018-04-23 16:38 GMT+02:00 Scott Kostyshak : > > > > > That could be. I really don't know. Is there a LyX method to check if > > > the context is fragile? Or s

Re: [LyX/master] Add basic support for cprotect

2018-04-23 Thread Scott Kostyshak
On Mon, Apr 23, 2018 at 02:56:53PM +, Jürgen Spitzmüller wrote: > 2018-04-23 16:38 GMT+02:00 Scott Kostyshak : > > > That could be. I really don't know. Is there a LyX method to check if > > the context is fragile? Or should I hardcode the names of the > > environments that seem to need it (fr

Re: [LyX/2.3.x] Add bindings for inset-settings.

2018-04-23 Thread Richard Kimberly Heck
On 04/23/2018 05:38 AM, Pavel Sanda wrote: > Richard Kimberly Heck wrote: >> diff --git a/status.23x b/status.23x >> index 0ca83fd..38395b2 100644 >> --- a/status.23x >> +++ b/status.23x >> @@ -91,6 +91,8 @@ What's new >> >> - Allow LFUN_UNICODE_INSERT to take multiple arguments (bug 11084). >>

Re: [LyX/master] Add bindings for inset-settings.

2018-04-23 Thread Richard Kimberly Heck
On 04/23/2018 03:01 AM, Kornel Benko wrote: > > Am Montag, 23. April 2018 05:56:38 CEST schrieb Richard Kimberly Heck > : > > > commit 8da025014dda1ff2d3cc1409778d5df68d06bf1a > > > Author: Richard Kimberly Heck > > > Date: Sun Apr 22 23:56:17 2018 -0400 > > > > > > Add bindings for inset-settings

Re: New export errors after the latest commits

2018-04-23 Thread Kornel Benko
Am Montag, 23. April 2018 18:36:38 CEST schrieb Jürgen Spitzmüller : > 2018-04-23 18:10 GMT+02:00 Kornel Benko : > > > > Thanks, yes, the Japanese tests pass now. > > > > Can you try again with the Polish ones now? > All pl compilations pass now too. > Jürgen Kornel signature.asc Des

Re: New export errors after the latest commits

2018-04-23 Thread Jürgen Spitzmüller
2018-04-23 18:10 GMT+02:00 Kornel Benko : > > Thanks, yes, the Japanese tests pass now. > Can you try again with the Polish ones now? Jürgen > > > Kornel > > >

Re: New export errors after the latest commits

2018-04-23 Thread Kornel Benko
Am Montag, 23. April 2018 16:22:20 CEST schrieb Jürgen Spitzmüller : > 2018-04-23 14:56 GMT+02:00 Kornel Benko : > > > < export/doc/attic/pl_Additional_dvi3_texF > > > > < export/doc/attic/pl_Additional_pdf2 > > > > < export/doc/attic/pl_Additional_pdf4_texF > > > > < export/doc/attic/pl_Addition

Re: [LyX/master] Add basic support for cprotect

2018-04-23 Thread Jürgen Spitzmüller
2018-04-23 16:38 GMT+02:00 Scott Kostyshak : > That could be. I really don't know. Is there a LyX method to check if > the context is fragile? Or should I hardcode the names of the > environments that seem to need it (from testing)? > The NeedProtect flag of layouts. > > Alternatively, I'm also

Re: [LyX/master] Add basic support for cprotect

2018-04-23 Thread Jürgen Spitzmüller
2018-04-23 16:38 GMT+02:00 Scott Kostyshak : > I misunderstood the comment > >// Environments and "no latex" types (e.g., knitr chunks) >// need cprotection regardless the content > > Reading it again, it was my mistake to interpret it as "every command > including an environment should be

Re: cprotect loaded unconditionally if e.g. verbatim

2018-04-23 Thread Scott Kostyshak
On Mon, Apr 23, 2018 at 09:05:17AM +, Jürgen Spitzmüller wrote: > Am Sonntag, den 22.04.2018, 15:47 -0400 schrieb Scott Kostyshak: > > That works for the example I had attached. But open the knitr example > > file, and you will see that cprotect is loaded (even though it is not > > needed). >

Re: [LyX/master] Add basic support for cprotect

2018-04-23 Thread Scott Kostyshak
On Mon, Apr 23, 2018 at 08:19:12AM +, Jürgen Spitzmüller wrote: > Am Montag, den 23.04.2018, 09:30 +0200 schrieb Jürgen Spitzmüller: > > AFAIU this would simply overshoot the cprotecting. It would cprotect > > each single command that has an environment in its argument, but > > AFAIU > > only s

Re: [LyX/master] Add basic support for cprotect

2018-04-23 Thread Scott Kostyshak
On Mon, Apr 23, 2018 at 07:30:01AM +, Jürgen Spitzmüller wrote: > Am Sonntag, den 22.04.2018, 15:48 -0400 schrieb Scott Kostyshak: > > Could you please give an example (e.g. steps to reproduce a problem > > with the patch)? > > AFAIU this would simply overshoot the cprotecting. It would cprote

Re: New export errors after the latest commits

2018-04-23 Thread Jürgen Spitzmüller
2018-04-23 14:56 GMT+02:00 Kornel Benko : > < export/doc/attic/pl_Additional_dvi3_texF > > < export/doc/attic/pl_Additional_pdf2 > > < export/doc/attic/pl_Additional_pdf4_texF > > < export/doc/attic/pl_Additional_pdf5_texF > > < export/doc/ja/Math_dvi > > < export/doc/ja/Math_pdf > > < export/doc/

New export errors after the latest commits

2018-04-23 Thread Kornel Benko
< export/doc/attic/pl_Additional_dvi3_texF < export/doc/attic/pl_Additional_pdf2 < export/doc/attic/pl_Additional_pdf4_texF < export/doc/attic/pl_Additional_pdf5_texF < export/doc/ja/Math_dvi < export/doc/ja/Math_pdf < export/doc/ja/Math_pdf3 < export/examples/ja/beamer_dvi < export/examples/ja/bea

Re: [LyX/2.3.x] Add bindings for inset-settings.

2018-04-23 Thread Pavel Sanda
Richard Kimberly Heck wrote: > diff --git a/status.23x b/status.23x > index 0ca83fd..38395b2 100644 > --- a/status.23x > +++ b/status.23x > @@ -91,6 +91,8 @@ What's new > > - Allow LFUN_UNICODE_INSERT to take multiple arguments (bug 11084). > > +- Added C-M-i as a shortcut for LFUN_INSET_SETTI

Re: cprotect loaded unconditionally if e.g. verbatim

2018-04-23 Thread Jürgen Spitzmüller
Am Sonntag, den 22.04.2018, 15:47 -0400 schrieb Scott Kostyshak: > That works for the example I had attached. But open the knitr example > file, and you will see that cprotect is loaded (even though it is not > needed). Better now? Jürgen > > Scott signature.asc Description: This is a digitall

Re: [LyX/master] Add basic support for cprotect

2018-04-23 Thread Jürgen Spitzmüller
Am Montag, den 23.04.2018, 09:30 +0200 schrieb Jürgen Spitzmüller: > AFAIU this would simply overshoot the cprotecting. It would cprotect > each single command that has an environment in its argument, but > AFAIU > only some commands need the cprotecting (I suppose those who are not > \long). With

Re: [LyX/master] Add basic support for cprotect

2018-04-23 Thread Jürgen Spitzmüller
Am Sonntag, den 22.04.2018, 15:48 -0400 schrieb Scott Kostyshak: > Could you please give an example (e.g. steps to reproduce a problem > with the patch)? AFAIU this would simply overshoot the cprotecting. It would cprotect each single command that has an environment in its argument, but AFAIU only

Re: [LyX/master] Add bindings for inset-settings.

2018-04-23 Thread Kornel Benko
Am Montag, 23. April 2018 05:56:38 CEST schrieb Richard Kimberly Heck : > commit 8da025014dda1ff2d3cc1409778d5df68d06bf1a > Author: Richard Kimberly Heck > Date: Sun Apr 22 23:56:17 2018 -0400 > > Add bindings for inset-settings. > > Fixes #7762. I propose following to emacs.bind