Re: Bug #10295

2017-06-05 Thread Richard Heck
On 06/01/2017 01:04 PM, Scott Kostyshak wrote:
> On Tue, May 30, 2017 at 03:13:30PM -0400, Richard Heck wrote:
>> On 05/29/2017 12:05 PM, Scott Kostyshak wrote:
>>> On Tue, May 16, 2017 at 10:18:05PM -0400, Richard Heck wrote:
 Can anyone test the fix for this bug posted at

 http://www.lyx.org/trac/ticket/10295

 It'd be nice to have a decent version for beta1.
>>> I can't reproduce the bug. Can anyone reproduce and then check that
>>> Richard's patch makes it go away?
>> JMarc confirmed it. We should try to get this into beta, but I won't be
>> able to get to it for a day or two.
> I agree.

This is now in.

Richard



Re: Gnuplot format & converter script

2017-06-05 Thread Tommaso Cucinotta

from the help message this sounds interesting, I also thought
we could have had various terminal types, and get .gp to .pdf,
to .eps, etc, albeit it's sufficient to convert to one
vectorial format, and LyX knows how to get to the others.
Perhaps among the most useful things, is that 1) you can use this
script to render on screen through the .gp interactive capabilities,
using the same unmodified .gp script that is included in .lyx;
2) the terminal options, but theburden is that each document
(even each use of .gp scripts) might need slightly different options...

not really much time in these days, but this script might easily
replace the gnuplot2pdf.py one, with proper invokation in configure.py.
(-t pdf). However, I can't get at a glance the why of all of this
lines' parsing & mangling. guess it's a try to be robust w.r.t. various
different contents of the .gp script.

T.

On 25/05/2017 19:36, Scott Kostyshak wrote:

On Fri, May 05, 2017 at 08:03:02PM -0400, Scott Kostyshak wrote:


Tommaso,

I found an old script from Koji for gnuplot. Is it useful now?


Tommaso,

No worries if you don't have time, but I just wanted to make sure you
saw this.

Scott





Re: [LyX/master] skip graphics conversion when runparams.dryrun is true

2017-06-05 Thread Tommaso Cucinotta

On 04/06/2017 03:40, Enrico Forestieri wrote:

On Fri, Jun 02, 2017 at 12:55:47AM +0200, Enrico Forestieri wrote:


So, output_file is always empty, whatever op.dryrun. I think that
the following change was instead meant:

-   string const output_file = prepareHTMLFile(op);
+   string const output_file = op.dryrun ? string() : prepareHTMLFile(op);


Good catch. I agree. Please commit.


Done at 8ae652eb.


thx Enrico !

T.



Re: Update: polyglossia in 2.2.3

2017-06-05 Thread N. Andrew Walsh
Hi Scott,

thank you for your message.

On Mon, Jun 5, 2017 at 3:08 AM, Scott Kostyshak  wrote:

> On Fri, May 26, 2017 at 12:35:09PM +0200, Jürgen Spitzmüller wrote:
> > Am Freitag, den 26.05.2017, 12:23 +0200 schrieb N. Andrew Walsh:
> >
> > > Or rather, the file will compile with 2.2.2, but not with 2.2.3
> > > (giving back the error about deleting a closing bracket as previously
> > > described).
> >
> > You mean 2.2.3 not 2.3alpha, right? In any case, the file compiles fine
> > for me both with recent stable (what will become 2.2.4) and master
> > (what will become 2.3.0).
> >
> > Could you also post the LaTeX export (to LaTeX (XeTeX)) of this MWE?
>
> Andrew, I just wanted to make sure you saw the above message.
>
Yes, I saw. I'm copied on the bug report, and have since submitted several
updates. There's a patch on the bug report that should resolve the issue
(and is going into 2.3.0) that resolves it for other users.

Thanks for keeping me informed.

Sincerely,

Andrew


Re: [LyX/master] Fix bugs #10650 and #9598

2017-06-05 Thread Enrico Forestieri
On Mon, Jun 05, 2017 at 07:37:08AM +0200, Kornel Benko wrote:

> Am Montag, 5. Juni 2017 um 03:50:47, schrieb Enrico Forestieri 
> 
> > On Sun, Jun 04, 2017 at 01:08:06PM +0200, Guillaume MM wrote:
> > 
> > > Le 04/06/2017 à 02:27, Enrico Forestieri a écrit :
> > > > On Sat, Jun 03, 2017 at 11:27:25PM +0200, Guillaume MM wrote:
> > > > 
> > > > > I am wondering since this results in a different vertical spacing.
> > > > 
> > > > No, this will not change the vertical spacing.
> > > It actually does, as shown with diffpdf.
> > 
> > I don't see any difference,
> 
> Have you used diffpdf?
> Here the snapshot of 'diffpdf -a'

The old code was wrong.

-- 
Enrico


Re: dash patch for stable

2017-06-05 Thread Guenter Milde
On 2017-06-03, Enrico Forestieri wrote:

> [-- Type: text/plain, Encoding:  --]

> On Sat, Jun 03, 2017 at 01:08:07AM +0200, Enrico Forestieri wrote:

>> On Fri, Jun 02, 2017 at 07:30:34PM +, Guenter Milde wrote:
>> > 
>> > Note that the "ERT patch" is only for *stable*, i.e. 2.2.x.
>> > For 2.2, ERT is the only way to ensure full backwards compatibility.

>> A solution should have been thought and done before 2.2.0 was released.
>> Now it's too late.

> Anyway, the attached patch should ensure backwards compatibility
> in stable without resorting to ERT.

This is the "90% compatible" workaround also suggested in 
>>   http://www.lyx.org/trac/ticket/10543
as a possible alternative. Mind, that it does not prevent all line-break
changes: with ---, hyphenation in adjacent words is suppressed, with
\textemdash+ZWSP, hyphenation in adjacent words is allowed.
See the attachments to  http://www.lyx.org/trac/ticket/10543
for examples where this leads to different output.

I don't have a preference regarding the two alternative patches for stable.


Günter