Re: [LyX/master] Copy documentation of new features of Additional.lyx in de, es, fr and ja documentation files as change tracking and remove chnage tracking in Additional.lyx

[Guenter Milde]

On 2017-07-23, Jean-Pierre Chrétien wrote:
> Le 16/07/2017 à 04:23, Scott Kostyshak a écrit :
>> On Wed, Jul 12, 2017 at 10:34:53PM +0200, jpc wrote:
>>> commit c9f398d7c516f9e9f3df71e74231a87e639180b9
>>> Author: jpc 
>>> Date:   Wed Jul 12 22:31:30 2017 +0200

>>>Copy documentation of new features of Additional.lyx in de, es, fr 
>>> and ja documentation files
>>>as change tracking and remove chnage tracking in Additional.lyx


> The 'Missing glyphs' error with system fonts and xelatex was present at 
> 94479240, before this commit. 

> FYI, I get the missing glyph error with 
> Additional.lyx in system fonts mode whatever the language. 

This should be fixed by Jürgen's patch for ZWNJ
[a40868510d36ccdcf3c431cde].

> In French, I get supplementary errors wirh emdash and endash
> characters.

The EM- and EN-DASH are present in LatinModern. 

After a split and try orgy, I found that the errors are due to missing
2015HORIZONTAL BAR = quotation dash
* long dash introducing quoted text).

This may be masked in the export tests by Kornels script-based replacement
of the default non-TeX fonts with FreeSerif/-Sans/-Mono.

Grep points to:

Additional.lyx:1918:―
Additional.lyx:10468:― 
Additional.lyx:10552: ― 
Additional.lyx:21842:― 
Additional.lyx:21856:― 

Jean-Pierre,
could you please have a look, whether this is intended or an em-dash
could/should be used at these places?

Merci,
Günter

Re: [LyX/master] Copy documentation of new features of Additional.lyx in de, es, fr and ja documentation files as change tracking and remove chnage tracking in Additional.lyx

[Jean-Pierre Chrétien]

Le 16/07/2017 à 04:23, Scott Kostyshak a écrit :

On Wed, Jul 12, 2017 at 10:34:53PM +0200, jpc wrote:

commit c9f398d7c516f9e9f3df71e74231a87e639180b9
Author: jpc 
Date:   Wed Jul 12 22:31:30 2017 +0200

   Copy documentation of new features of Additional.lyx in de, es, fr and 
ja documentation files
   as change tracking and remove chnage tracking in Additional.lyx

 lib/doc/Additional.lyx|  484 ++--
 lib/doc/de/Additional.lyx |  994 ++-
 lib/doc/es/Additional.lyx |  995 ++-
 lib/doc/fr/Additional.lyx | 1009 ++--
 lib/doc/ja/Additional.lyx | 1137 -
 5 files changed, 4080 insertions(+), 539 deletions(-)


This commit broke the following tests:

export/doc/de/Additional_pdf4_systemF
export/doc/de/Additional_pdf5_systemF

I just wanted to document which commit broke the test, in case anyone
wanted to take a look (it's possible it's due to a bug in our language
nesting/switching) and so that when others see the failing test they are
aware.


The 'Missing glyphs' error with system fonts and xelatex was present at 
94479240, before this commit. FYI, I get the missing glyph error with 
Additional.lyx in system fonts mode whatever the language. In French, I get 
supplementary errors wirh emdash and endash characters.


--
Jean-Pierre

Re: [LyX/master] Copy documentation of new features of Additional.lyx in de, es, fr and ja documentation files as change tracking and remove chnage tracking in Additional.lyx

[Kornel Benko]

Am Sonntag, 16. Juli 2017 um 16:56:48, schrieb Kornel Benko 
> Am Sonntag, 16. Juli 2017 um 04:16:22, schrieb Scott Kostyshak 
> 
> > On Sat, Jul 15, 2017 at 10:23:33PM -0400, Scott Kostyshak wrote:
> > > On Wed, Jul 12, 2017 at 10:34:53PM +0200, jpc wrote:
> > > > commit c9f398d7c516f9e9f3df71e74231a87e639180b9
> > > > Author: jpc 
> > > > Date:   Wed Jul 12 22:31:30 2017 +0200
> > > > 
> > > >Copy documentation of new features of Additional.lyx in de, es, 
> > > > fr and ja documentation files
> > > >as change tracking and remove chnage tracking in Additional.lyx
> > > > 
> > > >  lib/doc/Additional.lyx|  484 ++--
> > > >  lib/doc/de/Additional.lyx |  994 
> > > > ++-
> > > >  lib/doc/es/Additional.lyx |  995 
> > > > ++-
> > > >  lib/doc/fr/Additional.lyx | 1009 
> > > > ++--
> > > >  lib/doc/ja/Additional.lyx | 1137 
> > > > -
> > > >  5 files changed, 4080 insertions(+), 539 deletions(-)
> > > 
> > > This commit broke the following tests:
> > > 
> > > export/doc/de/Additional_pdf4_systemF
> > > export/doc/de/Additional_pdf5_systemF
> > > 
> > > I just wanted to document which commit broke the test, in case anyone
> > > wanted to take a look (it's possible it's due to a bug in our language
> > > nesting/switching) and so that when others see the failing test they are
> > > aware.
> > 
> > Actually the errors I see are about missing glyphs. I don't think there
> > is a regression here.
> > 
> > Scott
> 
> Looks like we cannot use \texcompmark macro in xelatex and lualatex.
> 

Just retested with
TL2015: Previously attached works with xelatex + lualatex
TL2016: works with lualatex but not with xelatex (missing glyphs)
TL2017: does not work for both (missing glyphs)

Kornel

signature.asc
Description: This is a digitally signed message part.

Re: [LyX/master] Copy documentation of new features of Additional.lyx in de, es, fr and ja documentation files as change tracking and remove chnage tracking in Additional.lyx

[Kornel Benko]

Am Sonntag, 16. Juli 2017 um 04:16:22, schrieb Scott Kostyshak 

> On Sat, Jul 15, 2017 at 10:23:33PM -0400, Scott Kostyshak wrote:
> > On Wed, Jul 12, 2017 at 10:34:53PM +0200, jpc wrote:
> > > commit c9f398d7c516f9e9f3df71e74231a87e639180b9
> > > Author: jpc 
> > > Date:   Wed Jul 12 22:31:30 2017 +0200
> > >
> > >Copy documentation of new features of Additional.lyx in de, es, fr 
> > > and ja documentation files
> > >as change tracking and remove chnage tracking in Additional.lyx
> > >
> > >  lib/doc/Additional.lyx|  484 ++--
> > >  lib/doc/de/Additional.lyx |  994 ++-
> > >  lib/doc/es/Additional.lyx |  995 ++-
> > >  lib/doc/fr/Additional.lyx | 1009 ++--
> > >  lib/doc/ja/Additional.lyx | 1137 
> > > -
> > >  5 files changed, 4080 insertions(+), 539 deletions(-)
> >
> > This commit broke the following tests:
> >
> > export/doc/de/Additional_pdf4_systemF
> > export/doc/de/Additional_pdf5_systemF
> >
> > I just wanted to document which commit broke the test, in case anyone
> > wanted to take a look (it's possible it's due to a bug in our language
> > nesting/switching) and so that when others see the failing test they are
> > aware.
>
> Actually the errors I see are about missing glyphs. I don't think there
> is a regression here.
>
> Scott

Looks like we cannot use \texcompmark macro in xelatex and lualatex.

Kornel

textcompwordmark.lyx
Description: application/lyx


signature.asc
Description: This is a digitally signed message part.

Re: [LyX/master] Copy documentation of new features of Additional.lyx in de, es, fr and ja documentation files as change tracking and remove chnage tracking in Additional.lyx

[Scott Kostyshak]

On Sat, Jul 15, 2017 at 10:23:33PM -0400, Scott Kostyshak wrote:
> On Wed, Jul 12, 2017 at 10:34:53PM +0200, jpc wrote:
> > commit c9f398d7c516f9e9f3df71e74231a87e639180b9
> > Author: jpc 
> > Date:   Wed Jul 12 22:31:30 2017 +0200
> > 
> >Copy documentation of new features of Additional.lyx in de, es, fr 
> > and ja documentation files
> >as change tracking and remove chnage tracking in Additional.lyx
> > 
> >  lib/doc/Additional.lyx|  484 ++--
> >  lib/doc/de/Additional.lyx |  994 ++-
> >  lib/doc/es/Additional.lyx |  995 ++-
> >  lib/doc/fr/Additional.lyx | 1009 ++--
> >  lib/doc/ja/Additional.lyx | 1137 
> > -
> >  5 files changed, 4080 insertions(+), 539 deletions(-)
> 
> This commit broke the following tests:
> 
> export/doc/de/Additional_pdf4_systemF
> export/doc/de/Additional_pdf5_systemF
> 
> I just wanted to document which commit broke the test, in case anyone
> wanted to take a look (it's possible it's due to a bug in our language
> nesting/switching) and so that when others see the failing test they are
> aware.

Actually the errors I see are about missing glyphs. I don't think there
is a regression here.

Scott


signature.asc
Description: PGP signature

Re: [LyX/master] Copy documentation of new features of Additional.lyx in de, es, fr and ja documentation files as change tracking and remove chnage tracking in Additional.lyx

[Scott Kostyshak]

On Wed, Jul 12, 2017 at 10:34:53PM +0200, jpc wrote:
> commit c9f398d7c516f9e9f3df71e74231a87e639180b9
> Author: jpc 
> Date:   Wed Jul 12 22:31:30 2017 +0200
> 
>Copy documentation of new features of Additional.lyx in de, es, fr and 
> ja documentation files
>as change tracking and remove chnage tracking in Additional.lyx
> 
>  lib/doc/Additional.lyx|  484 ++--
>  lib/doc/de/Additional.lyx |  994 ++-
>  lib/doc/es/Additional.lyx |  995 ++-
>  lib/doc/fr/Additional.lyx | 1009 ++--
>  lib/doc/ja/Additional.lyx | 1137 
> -
>  5 files changed, 4080 insertions(+), 539 deletions(-)

This commit broke the following tests:

export/doc/de/Additional_pdf4_systemF
export/doc/de/Additional_pdf5_systemF

I just wanted to document which commit broke the test, in case anyone
wanted to take a look (it's possible it's due to a bug in our language
nesting/switching) and so that when others see the failing test they are
aware.

Scott


signature.asc
Description: PGP signature