Re: ctests: uninvert the failing inverted unicode tests?

2016-11-23 Thread Scott Kostyshak
On Tue, Nov 22, 2016 at 03:16:00PM -0500, Scott Kostyshak wrote: > On Tue, Nov 22, 2016 at 10:49:47AM +, Guenter Milde wrote: > > I did a "blind update" to only keep the inversions not listed in your > > posting in eadc55b..7bb07f1. Could you please correct by adding/removing > > patterns to

Remove templatization from bformat() ?

2016-11-23 Thread Tommaso Cucinotta
Hi, while playing with strings, I've just run into a mysterious linking error [1] which, after a bit of digging, turned out to be decrypted as [2]. Basically, [2] is obtained by removing all templatization code for bformat() in lstrings.h. All template implementations are completely

Re: #10481: Hardening LyX against potential misuse

2016-11-23 Thread Enrico Forestieri
On Wed, Nov 23, 2016 at 10:27:03PM +0100, Tommaso Cucinotta wrote: > > One note: one way to avoid the [auth session] section growing unbounded, > might be to have an expiry timestamp, so that e.g., authorizations would > expire in ~1y or so. This might be done with a section syntax like: > >

Re: #10481: Hardening LyX against potential misuse

2016-11-23 Thread Richard Heck
On 11/23/2016 04:27 PM, Tommaso Cucinotta wrote: > On 23/11/2016 01:11, Enrico Forestieri wrote: >>> 1. versioning of the preferences file...what else is needed? >> >> Have a look at c2a18fc1 to get an idea. > > ok, thanks, do we need also anything else on the prefs2prefs side? If > not, do we

Re: #10481: Hardening LyX against potential misuse

2016-11-23 Thread Tommaso Cucinotta
On 23/11/2016 01:11, Enrico Forestieri wrote: 1. versioning of the preferences file...what else is needed? Have a look at c2a18fc1 to get an idea. ok, thanks, do we need also anything else on the prefs2prefs side? If not, do we actually need to bump up the prefs file version? What if a

LyX might still load hyperref if "use hyperref support" is unchecked

2016-11-23 Thread Scott Kostyshak
If there is a hyperlink inset, LyX seems to use hyperref regardless of whether "use hyperref support" is checked in document settings. I was initially surprised by this (I did not know that the particular document used hyperlink insets), although when I realized that hyperlink insets were used the

Re: ctests: how to classify 2 failing Arabic tests

2016-11-23 Thread Scott Kostyshak
On Tue, Nov 22, 2016 at 10:53:29AM +, Guenter Milde wrote: > Maybe [Show full LaTeX log] has more hints? Attached. I googled the error and it seems related to hyperref. I tried unchecking "use hyperref support" in document setting but even after doing that hyperref is still used, I suppose

Re: child's non-TeX fonts setting affects compilation from master

2016-11-23 Thread Scott Kostyshak
On Tue, Nov 22, 2016 at 08:54:05AM +, Guenter Milde wrote: > Therefore: > > * Warn the user when including a child document with different > "use-non-TeX-fonts" setting. +1. It would be simliar to what we do if the document class of the child is different from master. Scott

Re: Add groups for table option

2016-11-23 Thread Scott Kostyshak
On Wed, Nov 23, 2016 at 12:19:14PM +0300, edu Gpl wrote: > Dear lyx team > Thank you > > Sugg: > I hope you add groups in table options, like the groups in figures option, > this let us control and change the table font size and bourders automatecly. This is an interesting idea. I think the hard

Re: [LyX/master] Add the icon-size to the toolbar menu

2016-11-23 Thread Jürgen Spitzmüller
2016-11-22 10:02 GMT+01:00 Jean-Marc Lasgouttes : > commit e91572a00bd61d56b5386519b52d316a32974497 > Author: Daniel Ramöller > Date: Sun Oct 9 09:15:20 2016 +0200 > > Add the icon-size to the toolbar menu > > - Adds LFUNs for setting the icon size >

Re: pasted non-acceptable symbol

2016-11-23 Thread Jean-Marc Lasgouttes
Le 10/11/2016 à 10:19, Jean-Marc Lasgouttes a écrit : Le 09/11/2016 à 21:45, Stephan Witt a écrit : Am 09.11.2016 um 12:10 schrieb Jean-Marc Lasgouttes : Le 08/11/2016 à 14:56, Stephan Witt a écrit : I tested your patch and it works for me. The change of

Re: [LyX/master] Add 'needauth' option to converters that need explicit user authorization.

2016-11-23 Thread Jean-Marc Lasgouttes
Le 23/11/2016 à 10:05, Tommaso Cucinotta a écrit : indeed, the use_needauth option can be enabled/disabled in the GUI only when it's not forbidden in the other option, and if forbidden then use_needauth is ignored in the code. I know, might be turned into a 3-opt radio or combo :-)... Yes,

Re: [LyX/master] Cmake build: Prepare for hardening use of external programs

2016-11-23 Thread Kornel Benko
Am Dienstag, 22. November 2016 um 20:35:36, schrieb Tommaso Cucinotta > unless I'm mistaken, this is going to break compilation, as lyxwrap.cpp is > not (yet) included? > > T. See the following: Here nothing happens if usr.bin.lyxwrap.in is missing ... > > + > >

Add groups for table option

2016-11-23 Thread edu Gpl
Dear lyx team Thank you Sugg: I hope you add groups in table options, like the groups in figures option, this let us control and change the table font size and bourders automatecly. Best regards

Re: [LyX/master] Add 'needauth' option to converters that need explicit user authorization.

2016-11-23 Thread Tommaso Cucinotta
On 23/11/2016 10:00, Jean-Marc Lasgouttes wrote: Sorry to intervene so late in the game (and for not following the thread), but does it make sense to have two options? I am not sure what happens when the first one is on and the second one off. Wouldn't 3 radio buttons be more appropriate?

Re: [LyX/master] Add 'needauth' option to converters that need explicit user authorization.

2016-11-23 Thread Jean-Marc Lasgouttes
Le 22/11/2016 à 23:55, Tommaso Cucinotta a écrit : Two preference options are added: lyxrc.use_converter_needauth_forbidden disables any converter with the 'needauth' option, which is meant to force user to an explicit action via the preferences pane, before being able to use

Re: [LyX/master] Add the icon-size to the toolbar menu

2016-11-23 Thread racoon
On 22.11.2016 11:43, Enrico Forestieri wrote: On Tue, Nov 22, 2016 at 10:02:09AM +0100, Jean-Marc Lasgouttes wrote: commit e91572a00bd61d56b5386519b52d316a32974497 Author: Daniel Ramöller Date: Sun Oct 9 09:15:20 2016 +0200 Add the icon-size to the toolbar menu -