Re: [patch] support for fontspec options

2017-04-10 Thread Jürgen Spitzmüller
Am Dienstag, den 11.04.2017, 08:30 +0200 schrieb Jürgen Spitzmüller:
> Am Montag, den 10.04.2017, 00:44 +0200 schrieb Uwe Stöhr:
> > I don't understand. We are talking about 3 simple line edits. 
> 
> And I tried to argue that I think these line edits are not a suitable
> UI.
> 
> > As you can 
> > see in the patch, this not a massive change in anything. It is just
> > to 
> > provide a field to enter package options. I mean we also provide
> > such
> > a 
> > field where the user can enter "T1" or a font encoding of his
> > choice.
> > I 
> > don't see any difference to the 3 edit fields I proposed.
> 
> It is much more different, since the line edit are supposed complex
> key-value pairs.

Supposed _to take_ complex kv-pairs. And besides that, you cannot
simply add a fontend line to the preamble, since fontenc needs to be
loaded early (and we use the fontenc value at other places when
encodings are changed).

Jürgen

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


Re: [patch] support for fontspec options

2017-04-10 Thread Jürgen Spitzmüller
Am Montag, den 10.04.2017, 01:47 +0200 schrieb Uwe Stöhr:
> It is like with bug
> http://www.lyx.org/trac/ticket/8034
> , we wait for years now and I don't see a reason why we don't start
> to 
> implement the possibilities to add options to fontspec and also to 
> polyglossia now.

Yes, but putting in a line edit that just pretends native support and
actually does not any benefit over the preamble is not a real solution
to this. It just hides the actual problem.

Jürgen

> 
> Our Arabic translator would need the polyglossia options feature too:
> http://www.lyx.org/trac/ticket/9577
> 
> regards Uwe

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


Re: [patch] support for fontspec options

2017-04-10 Thread Jürgen Spitzmüller
Am Montag, den 10.04.2017, 00:44 +0200 schrieb Uwe Stöhr:
> I don't understand. We are talking about 3 simple line edits. 

And I tried to argue that I think these line edits are not a suitable
UI.

> As you can 
> see in the patch, this not a massive change in anything. It is just
> to 
> provide a field to enter package options. I mean we also provide such
> a 
> field where the user can enter "T1" or a font encoding of his choice.
> I 
> don't see any difference to the 3 edit fields I proposed.

It is much more different, since the line edit are supposed complex
key-value pairs.

> Concerning the cycles. I have not contributed much for months due to 
> lack of time. now I had a week in late shift. That sucks, but gives
> time 
> at night that can be used. In fact I cannot plan my life for
> development 
> cycles. As I said, unless anything is announced that new features are
> no 
> longer allowed, I think I can send patches.

You can also store your work and submit it when the new cycle starts.

> 
> > > \setmainfont[Script=Devanagari
> > > is sometimes necessary. Inputting the Script option is currently
> > > almost
> > > impossible if you are not a LaTeX master.
> > 
> > You need the same level of LaTeX knowledge (or knowledge of the
> > fontspec package, for that matter) in order to use your proposed
> > UI. It
> > does not make things easier at all. It will rather confuse users
> > who do
> > not know what to enter there (and if they know, adding a line to
> > the
> > preamble is in no way more difficult).
> 
> I see it differently. A good friend of mine is from India and he
> once 
> asked me about LyX. I created a Wiki page:
> https://wiki.lyx.org/Windows/Devanagari
> And that way I googled around. Googling brings you quickly to the 
> "Script=Devanagari option. So one doesn't need to be a TeXpert to
> find 
> this. 

But googling also brings you to the whole macro. This is really no step
forward.

> To be constructive. Why don't you propose a better UI? 

I did.


> It cannot be that 
> complicated to decide where 3 edit fields belong to.

Certainly not. But the point is that these 3 edit fields are a wrong
design decision to begin with.

> And as you can see in my patch, I have not increased the width of
> the 
> dialog.

I see an increasement.

> > I plan to implement some generic key-value dialog widget in the 2.4
> > cycle. This would translate key-value option to either checkboxes
> > (booleans), combos (closed lists), editable combos (open lists) or
> > line
> > widgets (text input). The UI could be oriented for instance at the
> > UI
> > of the Qt designer. It occurred to me that such a thing is needed
> > to
> > implement the rest of biblatex, but it will also be useful for
> > class
> > and package options, and it can be used to implement a proper font
> > features UI.
> 
> That would be a massive change. Since I cannot see much difference
> to 
> the line edit for the font encoding I don't see why we must wait for
> the 
> 2.4. cycle to add the new edit fields. As you plan to rearrange them 
> anyway soon, you will in any case shift everything around.
> I mean when you e.g manufacture a car and an engineer proposes to add
> a 
> second inner mirror, you are able to do this even as you know that
> will 
> will redesign the car for in future. Then the future car will have
> also 
> 2 inner mirrors, but maybe with another shape and in another
> position.

I repeat what I wrote in the last mail: going "let's implement the
feature and fix the UI later" is always a bad idea.

> > > I implemented that the fields are only active when fontspec is
> > > actually
> > > used. If you see here mistakes in my patch, please tell me.
> > 
> > Yet they are still visible (and hence clutter the dialog) when
> > fontspec
> > is not used.
> 
> That is easy to change. So if we would find a suitable position for
> the 
> fields I will add this to the patch.

"let's implement the feature and fix the UI later" is always a bad
idea.

> > That's what I do. But you seem to always take criticism personally,
> > whereas I am criticizing the implementation proposal, not you.
> 
> It depends on how you say things. I took it personally. But it is OK
> now.

Good.

Jürgen

> 
> regards Uwe

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


Re: [patch] fix float label (bug 10618)

2017-04-10 Thread Jean-Marc Lasgouttes

Le 11/04/2017 à 02:53, Uwe Stöhr a écrit :

The attached simple patch fixes
http://www.lyx.org/trac/ticket/10618

The bug is that on opening /saving a file only the float type is written
to the label. But there exists a detailed label writing function that is
just not called (only on buffer update).

OK to go in or have I overseen anything?
If it goes in, this would be a candidate for LyX 2.2.3 as well.


This looks OK to me.

JMarc



Re: Tentative schedule for 2.3.0 release

2017-04-10 Thread Andrew Parsloe



On 11/04/2017 2:45 p.m., Scott Kostyshak wrote:



A second reason to keep the alpha is that very few Windows and Mac users
can compile a development version. An alpha is nice in situations where
we cannot reproduce a bug. We can ask users if they can still reproduce
e.g. a serious crash that they reported. This can let us know if that
issue is something we need to focus on fixing before a beta.


On Mon, Apr 10, 2017 at 11:58:51AM +0200, Jean-Marc Lasgouttes wrote:


Seriously, I am not sure about the alpha release. It is serious work, and I
am not sure what we will gain from it (how many user would try it, but not
try nightlies, for example).


Have we produced nightlies before? Doesn't that require an automatic
build system for Mac and Windows?

I hope others join this conversation. To alpha or not to alpha?

Scott



For what it's worth, both alpha1 and alpha2 of the LyX 2.2 release had 
significant bugs on windows 
(http://marc.info/?l=lyx-users&m=144833725611421&w=2 and 
http://marc.info/?l=lyx-devel&m=144878562018319&w=2).


Andrew

---
This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/antivirus



Re: LyX docs: cleaning up math options

2017-04-10 Thread Scott Kostyshak
On Mon, Nov 21, 2016 at 10:00:19PM -0500, Scott Kostyshak wrote:
> On Mon, Nov 21, 2016 at 05:13:55PM +, Jean-Pierre Chrétien wrote:
> 
> > 88% tests passed, 669 tests failed out of 5553
> > Does this sound correct? I have TL2016 with all language collections and
> > fonts installed.
> 
> Doesn't seem too strange. If I run all the tests, I get 148 failing out
> of 6469, but this is because I have spent a lot of time satisfying the
> dependencies of the tests so that is expected.
> 
> > I won't touch the files with option 2 (always load).
> 
> Why not? I would prefer to change those to automatic also.
> 
> > Any further idea ? In particular, should I run ctest with other options 
> > than export?
> 
> I think your plan is good. I will also save my ctest results
> before/after your changes so I will double-check that no tests go from
> passing to failing.
> 
> Thanks,
> 
> Scott

Jean-Pierre,

Any update on this?

Scott


signature.asc
Description: PGP signature


Re: [LyX/master] Add support to cross out characters

2017-04-10 Thread Scott Kostyshak
On Thu, Apr 06, 2017 at 09:41:57AM +0100, José Abílio Matos wrote:
> On Thursday, 6 April 2017 02.56.51 WEST Scott Kostyshak wrote:
> > I see it's used in other places in lyx2lyx (grep for '== True'), so
> > perhaps this is convention in Python?
> 
> Nope.
> 
> The convention is the former:
> 
> if changed:
>   ...

I was going to make this change, but then I checked more globally and
there are instances of the "== true" in our C++ code.

If someone wants me to change this for the instances in our C++ and our
Python code, I will put this on my TODO list after the release of 2.3.0
(we might as well make this change at the beginning of a cycle).
Otherwise, I will forget about it.

To see instances in other parts of the code, run

  git grep -i "== true"

Scott


signature.asc
Description: PGP signature


Re: Tentative schedule for 2.3.0 release

2017-04-10 Thread Scott Kostyshak
On Mon, Apr 10, 2017 at 09:46:45AM +0200, Jean-Marc Lasgouttes wrote:

> It was proably discussed last time, but I do not remember: what is the
> rationale for having an alpha before feature freeze? Shouldn't it be
> the opposite?

The requirements and expectations of an alpha are low. I think it is
understood that there should not be any big features right before the
feature freeze. But doing it this way allows us to get an alpha out
quickly and get testing on 99% of the features that will go into the
final release.


On Mon, Apr 10, 2017 at 11:23:16AM +0100, José Abílio Matos wrote:

> OTHO I remember that as the release manager of one of the previous releases 
> the alpha release was the first moment to the test and lubricate the release 
> procedure. To guarantee that tools like "make distcheck" work as intended. By 
> decoupling this from the beta stage where we should by then be in feature 
> freeze the testing was easier.
> Note that this is my personal recollection, I can be wrong. :-)

Indeed, this is nice. I think it was more important last time when it
was the first time I was release manager. It was good for me to practice
the fun process of signing the files, make sure upload to FTP works,
etc.

> With that said we have improved the test and maintenance of the release tools 
> and since the last version was released less than one year ago and Scott is 
> the release manager I agree that we can skip this version. But clearly this 
> is 
> Scott's call.

I am inclined to keep the alpha but am open to more discussion.

One reason for keeping the alpha is that we do not have strong
requirements to get it out. It is a way to start the ball rolling.
Requirements to release a beta are more strict, and it is nice to break
up a big task into smaller more achievable tasks.

A second reason to keep the alpha is that very few Windows and Mac users
can compile a development version. An alpha is nice in situations where
we cannot reproduce a bug. We can ask users if they can still reproduce
e.g. a serious crash that they reported. This can let us know if that
issue is something we need to focus on fixing before a beta.


On Mon, Apr 10, 2017 at 11:58:51AM +0200, Jean-Marc Lasgouttes wrote:

> Seriously, I am not sure about the alpha release. It is serious work, and I
> am not sure what we will gain from it (how many user would try it, but not
> try nightlies, for example).

Have we produced nightlies before? Doesn't that require an automatic
build system for Mac and Windows?

I hope others join this conversation. To alpha or not to alpha?

Scott


signature.asc
Description: PGP signature


Re: Use Qt 5.9.0 for Mac/Win binaries?

2017-04-10 Thread Scott Kostyshak
On Mon, Apr 10, 2017 at 09:41:56AM +0200, Jean-Marc Lasgouttes wrote:

> I would expect LTS to be a safe, conservative choice, but I may be
> wrong.

I think you are right.

Scott


signature.asc
Description: PGP signature


Re: Use Qt 5.9.0 for Mac/Win binaries?

2017-04-10 Thread Scott Kostyshak
On Mon, Apr 10, 2017 at 11:47:40PM +0200, Uwe Stöhr wrote:
> El 10.04.2017 a las 09:41, Jean-Marc Lasgouttes escribió:
> 
> > I have not followed the recent Qt development, but I find it depressing
> > that we have to use the latst and greatest because anything older has
> > significant issues. I would expect LTS to be a safe, conservative
> > choice, but I may be wrong.
> 
> My experience with Qt is that .0 releases often had bugs. Therefore I waited
> for 4.8.1 etc. I also see that bugs are often not fixed for older releases.
> With the LTS versions QT went another way. Now they try to backport the
> major bugfixes.
> 
> Therefore I would like to use the latest Qt LTS version. This is still
> version 5.6 (just a year old). They plan to release a new bugfix version of
> 5.6 soon.

OK 5.6 sounds like a good and safe idea. Unless we know of bugs (I
don't) that affect LyX that are fixed, we can stick with that.

Thanks for the comments,

Scott


signature.asc
Description: PGP signature


Re: [LyX/master] Add support to cross out characters

2017-04-10 Thread Scott Kostyshak
On Mon, Apr 10, 2017 at 11:20:51PM +0200, Uwe Stöhr wrote:
> El 10.04.2017 a las 10:43, Jean-Marc Lasgouttes escribió:
> 
> > > I have not heard about this rule.
> > 
> > This is why there is the word "propose" in the sentence above.
> 
> OK. Nevertheless I support this rule. Btw. where can I find accepted rules?

The only place I know is Development.lyx.

Scott


signature.asc
Description: PGP signature


Re: [LyX/master] Length.cpp: add new unit representing \baselineskip

2017-04-10 Thread Scott Kostyshak
On Tue, Apr 11, 2017 at 02:58:08AM +0200, Uwe Stöhr wrote:
> El 10.04.2017 a las 05:35, Scott Kostyshak escribió:
> 
> > This commit broke the test
> > 
> >check_Length
> > 
> > I'm guessing that there is no regression and that the test just needs to
> > be updated. Uwe, can you please take a look?
> 
> How is this test executed? What problems do you get and where is the test
> defined?

If you use CMake, you can run:

  ctest -R "check_Length"

For autotools, I think this test is run when you do

  make check
  make distcheck

But I think the test has been fixed.

Scott


signature.asc
Description: PGP signature


Re: [LyX/master] Length.cpp: add new unit representing \baselineskip

2017-04-10 Thread Uwe Stöhr

El 10.04.2017 a las 05:35, Scott Kostyshak escribió:


This commit broke the test

   check_Length

I'm guessing that there is no regression and that the test just needs to
be updated. Uwe, can you please take a look?


How is this test executed? What problems do you get and where is the 
test defined?


I only find check_Length.cpp which compiles here without errors or warnings.

regards Uwe


[patch] fix float label (bug 10618)

2017-04-10 Thread Uwe Stöhr

The attached simple patch fixes
http://www.lyx.org/trac/ticket/10618

The bug is that on opening /saving a file only the float type is written 
to the label. But there exists a detailed label writing function that is 
just not called (only on buffer update).


OK to go in or have I overseen anything?
If it goes in, this would be a candidate for LyX 2.2.3 as well.

reards Uwe


Re: [patch] fix bug 10270 - allow float placements for rotated floats

2017-04-10 Thread Uwe Stöhr

El 10.04.2017 a las 11:05, Jean-Marc Lasgouttes escribió:

These environment always put the result its own page for technical 
reasons. All the original reporter said is "I think the 'Here if 
possible' and 'Here definitely' GUI options should still remain available".


I think we have here the same case as with \baselineskip. LaTeX allows 
all placement options. They might not be sensible but if the user wants 
to use them we should not forbid them like we do not forbid 
\baselineskip as horizontal length.


I rethought the patch and think a fileformat is necessary. Attached is 
an updated patch where I for now disabled the top and bottom placement 
ans you proposed.


thanks and regards
Uwe
 development/FORMAT   |  4 +++
 lib/lyx2lyx/lyx_2_3.py   | 49 +++-
 src/frontends/qt4/FloatPlacement.cpp |  9 ---
 src/insets/InsetFloat.cpp|  2 +-
 src/tex2lyx/text.cpp |  5 
 src/version.h|  4 +--
 6 files changed, 65 insertions(+), 8 deletions(-)

diff --git a/development/FORMAT b/development/FORMAT
index 13dad66442..0a2de0370d 100644
--- a/development/FORMAT
+++ b/development/FORMAT
@@ -7,6 +7,10 @@ changes happened in particular if possible. A good example 
would be
 
 ---
 
+2017-04-11 Uwe Stöhr 
+   * Format incremented to 540: support for rotated float placements
+ - no new LFUN or buffer parameters
+
 2017-04-08 Uwe Stöhr 
* Format incremented to 539: support for \baselineskip.
  - new length unit BLS
diff --git a/lib/lyx2lyx/lyx_2_3.py b/lib/lyx2lyx/lyx_2_3.py
index 392f5e5dbc..23eee11d11 100644
--- a/lib/lyx2lyx/lyx_2_3.py
+++ b/lib/lyx2lyx/lyx_2_3.py
@@ -2063,6 +2063,51 @@ def revert_baselineskip(document):
 i = i + 1
 
 
+def revert_rotfloat(document):
+  " Revert placement options for rotated floats "
+  i = 0
+  j = 0
+  k = 0
+  while True:
+i = find_token(document.body, "sideways true", i)
+if i != -1:
+  regexp = re.compile(r'^.*placement.*$')
+  j = find_re(document.body, regexp, i-2)
+  if j == -1:
+  return
+  if j != i-2:
+  i = i + 1
+  continue
+else:
+  return
+# we found a sideways float with placement options
+# at first store the placement
+beg = document.body[i-2].rfind(" ");
+placement = document.body[i-2][beg+1:]
+# check if the option'H' is used
+if placement.find("H") != -1:
+  add_to_preamble(document, ["\\usepackage{float}"])  
+# now check if it is a starred type
+if document.body[i-1].find("wide true") != -1:
+  star = '*'
+else:
+  star = ''
+# store the float type
+beg = document.body[i-3].rfind(" ");
+fType = document.body[i-3][beg+1:]
+# now output TeX code
+endInset = find_end_of_inset(document.body, i-3)
+if endInset == -1:
+  document.warning("Malformed LyX document: Missing '\\end_inset' of Float 
inset.")
+  return
+else:
+  document.body[endInset-2: endInset+1] = put_cmd_in_ert("\\end{sideways" 
+ fType + star + '}')
+  document.body[i-3: i+2] = put_cmd_in_ert("\\begin{sideways" + fType + 
star + "}[" + placement + ']')
+  add_to_preamble(document, ["\\usepackage{rotfloat}"])  
+
+i = i + 1
+
+
 ##
 # Conversion hub
 #
@@ -2099,10 +2144,12 @@ convert = [
[536, []],
[537, []],
[538, [convert_mathindent]],
-   [539, []]
+   [539, []],
+   [540, []]
   ]
 
 revert =  [
+   [539, [revert_rotfloat]],
[538, [revert_baselineskip]],
[537, [revert_mathindent]],
[536, [revert_xout]],
diff --git a/src/frontends/qt4/FloatPlacement.cpp 
b/src/frontends/qt4/FloatPlacement.cpp
index b446c7c057..d0c7a58bcf 100644
--- a/src/frontends/qt4/FloatPlacement.cpp
+++ b/src/frontends/qt4/FloatPlacement.cpp
@@ -244,18 +244,19 @@ void FloatPlacement::checkAllowed() const
if (spanCB->isVisible()) {
bool const span = spanCB->isChecked();
bool const sideways = sidewaysCB->isChecked();
-   defaultsCB->setEnabled(!sideways);
topCB->setEnabled(!sideways && !defaults && !heredefinitely
  && contains(allowed_placement_, 't'));
bottomCB->setEnabled(!sideways && !defaults && !span && 
!heredefinitely
 && contains(allowed_placement_, 'b'));
pageCB->setEnabled(!sideways && !defaults && !heredefinitely
   && contains(allowed_placement_, 'p'));
-   herepossiblyCB->setEnabled(!sideways && !defaults && !span && 
!heredefinitely
+   if (!pageCB->isChecked())
+   pageCB->setChecked(sideways && 
contains(allowed_placement_, 'p'));
+   herepossiblyCB->setEnabled(!defaults && !span && !heredefinitely
   

Re: Use Qt 5.9.0 for Mac/Win binaries?

2017-04-10 Thread Uwe Stöhr

El 10.04.2017 a las 09:41, Jean-Marc Lasgouttes escribió:

I have not followed the recent Qt development, but I find it depressing 
that we have to use the latst and greatest because anything older has 
significant issues. I would expect LTS to be a safe, conservative 
choice, but I may be wrong.


My experience with Qt is that .0 releases often had bugs. Therefore I 
waited for 4.8.1 etc. I also see that bugs are often not fixed for older 
releases. With the LTS versions QT went another way. Now they try to 
backport the major bugfixes.


Therefore I would like to use the latest Qt LTS version. This is still 
version 5.6 (just a year old). They plan to release a new bugfix version 
of 5.6 soon.


regards Uwe


Re: [patch] fix for bug 10440 (LyX.exe does no longer work on command line)

2017-04-10 Thread Uwe Stöhr

El 10.04.2017 a las 05:33, Scott Kostyshak escribió:


5 months ago the user "backbone" presented a patch that fixes the bug for me
(see attached).
I don't understand what is wrong wit this patch and all further patches do
not work.


See comment 15. backbone replied to this question.


I don't understand this comment. Enrico replied that what backbone 
understood in the comment was not what he meant.


However, I don't know anything here concerning the code, but I can tell 
that I had the patch often and at least on Windows it works fine.



I agree that this issue is important. We are waiting for input from
Georg. If Georg does not have time to take a look before alpha, then we
will need to make a decision without him.


This bug is also in LyX 2.2.x so we should act independent of LyX 2.3.

regards Uwe


Re: [LyX/master] Add support to cross out characters

2017-04-10 Thread Uwe Stöhr

El 10.04.2017 a las 23:20, Uwe Stöhr escribió:


You mean that formulas are now also indented within LyX. I'll have a look but.


I had a look but failed as often: I cannot find a start point where 
displayed formulas are painted. I often have to stop at this point. LyX 
needs like a sitemap where what is done.


In this case I know where the different math features are painted but 
not where the math inset itself is painted.


regards Uwe


Re: [LyX/master] Add support to cross out characters

2017-04-10 Thread Uwe Stöhr

El 10.04.2017 a las 10:43, Jean-Marc Lasgouttes escribió:


I have not heard about this rule.


This is why there is the word "propose" in the sentence above.


OK. Nevertheless I support this rule. Btw. where can I find accepted rules?


I could not find a better solution that I proposed. When zooming out a
lot one got no stroke at all.


I implemented this for you now at 94114fd1. It is even a little bit 
shorter than the original code, I do not think that the price to pay was 
so high for something correct.


Thanks. It works better but still the number of strokes depends on the 
zoom level. For example


This is \xout{xout text}.

gives me 10 or 11 strokes (slashes) depending on the zoom level.
But more important, the problem reported by Scott is gone.

WYSIWYM is not about "I have the right to give an ugly screen output", 
it is about having a visually pleasant output that tells you what your 
output in a meaningful way. The only difference with WYSIWYG  is that 
some things (labels, footnotes, line breaks) are not represented 
visually, but semantically.


Hmm, I did not design ugly screen output. Here it was an acceptable 
solution. I did not see the problem of Scott. Of course for Scott it was 
then indeed ugly.


So, what I mean is that mimicking LaTeX is not a requirement, it is a 
trick to help you.


OK, I understand.


Now, what I would like to see is some action on the lefteqn patch ;)


What exactly? I added now a tooltip that I forgot ;-)
You mean that formulas are now also indented within LyX. I indeed never 
thought about this. (I simply use fleqn for so long that I did not miss 
the on-screen representation.) I'll have a look but.


regards Uwe


Re: [patch] Re: LyX master still not ready for Python 3

2017-04-10 Thread Enrico Forestieri
On Mon, Apr 10, 2017 at 03:07:34PM +0200, Jean-Marc Lasgouttes wrote:

> Le 10/04/2017 à 12:27, José Abílio Matos a écrit :
> > Since this fixes a bug I think that it should be committed.
> > For the final version I intend to uniform all the changes that have been 
> > done
> > to support python 2 and 3 to make the code more maintainable.
> 
> OK, I did that.

Thanks. Short on time lately.

-- 
Enrico


Re: Tentative schedule for 2.3.0 release

2017-04-10 Thread Stephan Witt
Am 10.04.2017 um 05:40 schrieb Scott Kostyshak :
> 
> Dear all,
> 
> I think there is agreement that master is pretty stable. Besides just a
> feeling, I think that this can be confirmed by looking at the trac
> tickets with "2.3.0" milestone and tickets with the "regression"
> keyword.
> 
> I propose the following schedule for the 2.3.0 release process:
> 
>alpha:   22 April
>feature freeze:  19 May
>beta:09 June
>RC:  12 July
>final:   01 August
> 
> Uwe and Stephan, do you know if you will be available around these dates
> to produce binaries?

Yes. These dates are no problem for me. I’m in France when final is planned.
But I cannot think of a hotel without network access in Europe. 

Stephan

> 
> As usual, this schedule is tentative and likely to change (dates could
> be earlier or later).
> 
> Please feel free to propose changes to this schedule, or to ask for me
> to explain why I chose a certain date (or rather a certain duration
> between stages).
> 
> Scott
> 



Re: Ticket #10455

2017-04-10 Thread Guillaume MM

Le 10/04/2017 à 21:05, racoon a écrit :

Hi!

Can someone check

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

and put it into master for testing?

Best,
Daniel




Hello Daniel and the list,

you also have
http://www.lyx.org/trac/ticket/10483
http://www.lyx.org/trac/ticket/10379
http://www.lyx.org/trac/ticket/10283

they are (and remain) on my to-do list for 2.3, but anybody else please
feel free to review the patches since it takes so much time for me to
get back to it.

Sincerely,
Guillaume



Ticket #10455

2017-04-10 Thread racoon

Hi!

Can someone check

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

and put it into master for testing?

Best,
Daniel



Re: Inconsistency in "Advices for using the LyX bug tracker"

2017-04-10 Thread racoon

On 10.04.2017 11:14, Jean-Marc Lasgouttes wrote:

Le 08/04/2017 à 09:13, racoon a écrit :

In "Advices for using the LyX bug tracker"

http://www.lyx.org/trac/wiki/reportingBugs

It is stated: "When reporting a bug, indicate your LyX version and
*platform* (the latter via TicketKeywords)."


I changed the page. Is it better now?


Yes, thanks!

Daniel




Re: [patch] Re: LyX master still not ready for Python 3

2017-04-10 Thread Jean-Marc Lasgouttes

Le 10/04/2017 à 12:27, José Abílio Matos a écrit :

Since this fixes a bug I think that it should be committed.
For the final version I intend to uniform all the changes that have been done
to support python 2 and 3 to make the code more maintainable.


OK, I did that.

JMarc



Re: [patch] Re: LyX master still not ready for Python 3

2017-04-10 Thread José Abílio Matos
On Monday, 10 April 2017 11.02.21 WEST Jean-Marc Lasgouttes wrote:
> As I wrote, it fixes my problem. Do you want me to apply it, or are
> there reasons to hold it?
> 
> JMarc

Since this fixes a bug I think that it should be committed.
For the final version I intend to uniform all the changes that have been done 
to support python 2 and 3 to make the code more maintainable.
-- 
José Abílio


Re: Tentative schedule for 2.3.0 release

2017-04-10 Thread José Abílio Matos
On Monday, 10 April 2017 10.58.51 WEST Jean-Marc Lasgouttes wrote:
> No, because doing everything by opposition to others is like doing doing
> it like them: it is following their whim 

:-)
 
> Seriously, I am not sure about the alpha release. It is serious work,
> and I am not sure what we will gain from it (how many user would try it,
> but not try nightlies, for example).
> 
> JMarc

That is a reasonable objection. In essence that was the reason to have it 
dropped from Fedora.

OTHO I remember that as the release manager of one of the previous releases 
the alpha release was the first moment to the test and lubricate the release 
procedure. To guarantee that tools like "make distcheck" work as intended. By 
decoupling this from the beta stage where we should by then be in feature 
freeze the testing was easier.
Note that this is my personal recollection, I can be wrong. :-)

With that said we have improved the test and maintenance of the release tools 
and since the last version was released less than one year ago and Scott is 
the release manager I agree that we can skip this version. But clearly this is 
Scott's call.

-- 
José Abílio


Jenkins build is back to normal : Build branch "master" » ubuntu-xenial-qt4-autotools-extended #148

2017-04-10 Thread ci-lyx
https://ci.inria.fr/lyx/job/build-master-head/job/ubuntu-xenial-qt4-autotools-extended/148/


Re: [patch] Re: LyX master still not ready for Python 3

2017-04-10 Thread Jean-Marc Lasgouttes

Le 05/04/2017 à 22:08, Enrico Forestieri a écrit :

Most probably you have some non-ascii characters in a \DeclareLaTeXClass
line in some of your layout files. Now those files are explicitly read as
utf-8 encoded. If this is so, the attached patch should help.

Another consequence of the recent changes is that if someone has old
layouts containing latin1 characters then configure will now misteriously
fail.


As I wrote, it fixes my problem. Do you want me to apply it, or are 
there reasons to hold it?


JMarc



Re: How far is 2.3.0?

2017-04-10 Thread Jean-Marc Lasgouttes

Le 10/04/2017 à 11:49, José Abílio Matos a écrit :

FWIW, IMHO it all comes to:

What is the event that we think is relevant to jump to the 3.x series?

If none then the first number in the 2.x.y moniker is irrelevant and it should
be dropped.


I agree with what you write, but it does not seem important enough to me 
to actually act on it :)


Either way, I do not really care. The only numbering that makes a real 
difference to me is Year.Month, but it is not adapted to our workflow.


JMarc



Re: Tentative schedule for 2.3.0 release

2017-04-10 Thread Jean-Marc Lasgouttes

Le 10/04/2017 à 11:27, José Abílio Matos a écrit :

Again following your reasoning since everyone is abandoning the alpha release
we should keep it. ;-)


No, because doing everything by opposition to others is like doing doing 
it like them: it is following their whim :)


Seriously, I am not sure about the alpha release. It is serious work, 
and I am not sure what we will gain from it (how many user would try it, 
but not try nightlies, for example).


JMarc



Re: How far is 2.3.0?

2017-04-10 Thread José Abílio Matos
On Monday, 10 April 2017 08.50.32 WEST Jean-Marc Lasgouttes wrote:
> Version numbers are just version numbers, I do not think that they make
> as much of a difference than getting versions out fast. In the case of
> firefox and gcc, the important part of the new release scheme was speed.
> The rest is just marketing.
> 
> JMarc

My rationale is always the same we have basically major and minor versions. 
For major versions we introduce changes that are incompatible with older 
versions, in the sense that older versions can not read the new file formats 
(lyx file, layouts, etc).

Minor versions are strictly compatible and represent incremental improvements 
in the same stable series.

So if our development only has two types of releases we should use two 
numbers, the major and the minor.

I remember this type of discussion that we had to jump to the 1.0 release, to 
jump to the 2.0 release and so on. :-)

In the case of gcc btw the idea was not to get faster versions, since the 
release cycle was kept at one year, but to convey the message that each new 
major version is really a major version.
The marketing even in free software projects is also important, as you know. 
:-)

Also as a developer that has to type the version number often when comparing 
previous formats I would appreciate to have to type less meaningless numbers. 
This is a very objective argument.


FWIW, IMHO it all comes to:

What is the event that we think is relevant to jump to the 3.x series?

If none then the first number in the 2.x.y moniker is irrelevant and it should 
be dropped.
-- 
José Abílio


Re: Tentative schedule for 2.3.0 release

2017-04-10 Thread José Abílio Matos
On Monday, 10 April 2017 08.46.45 WEST Jean-Marc Lasgouttes wrote:
> It was proably discussed last time, but I do not remember: what is the
> rationale for having an alpha before feature freeze? Shouldn't it be the
> opposite?

I think that this is the usual procedure. Take Fedora as an example, the alpha 
release is done before the feature freeze.

The rationale, as far as I can remember, is to invite the users to test and 
get an idea where the project is coming and how do the new features integrate 
in the project. So it could be possible that some feature is deemed unready 
and dropped before feature freeze.

On the other hand there is a proposal to drop the alpha release starting from 
Fedora 27 (to be released in October/November). So that means that Fedora 26 
alpha, that was released last Tuesday, is the last alpha release for Fedora.

Again following your reasoning since everyone is abandoning the alpha release 
we should keep it. ;-)
-- 
José Abílio


Re: Inconsistency in "Advices for using the LyX bug tracker"

2017-04-10 Thread Jean-Marc Lasgouttes

Le 08/04/2017 à 09:13, racoon a écrit :

In "Advices for using the LyX bug tracker"

http://www.lyx.org/trac/wiki/reportingBugs

It is stated: "When reporting a bug, indicate your LyX version and
*platform* (the latter via TicketKeywords)."


I changed the page. Is it better now?

JMarc



Re: [patch] fix bug 10270 - allow float placements for rotated floats

2017-04-10 Thread Jean-Marc Lasgouttes

Le 10/04/2017 à 02:21, Uwe Stöhr a écrit :

As reported in
http://www.lyx.org/trac/ticket/10270
LyX forbids incorrectly the float placement options.

The attached patch fixed this.


These environment always put the result its own page for technical 
reasons. All the original reporter said is "I think the 'Here if 
possible' and 'Here definitely' GUI options should still remain available".


This comment is correct as far as I understand from my cursory google 
search. Nobody 'forgot' to implement sideways for sidewyas, things like 
'top' or 'bottom' are correctly disabled (and page should be on but 
disabled IMO).


JMarc


Re: [LyX/master] support to indent formulas

2017-04-10 Thread Jean-Marc Lasgouttes

Le 08/04/2017 à 02:13, Uwe Stöhr a écrit :

But OK, I moved it now as you requested.


Thanks Uwe.

JMarc



Re: [patch] support for \baselineskip

2017-04-10 Thread Jean-Marc Lasgouttes

Le 08/04/2017 à 04:58, Uwe Stöhr a écrit :

El 07.04.2017 a las 10:51, Jean-Marc Lasgouttes escribió:


Maybe more like:...


Thanks.
The patch is now in including this hint and the new BLS unit is
available for all lengths.


Thanks.

JMarc



Re: [LyX/master] Add support to cross out characters

2017-04-10 Thread Jean-Marc Lasgouttes

Le 10/04/2017 à 01:00, Uwe Stöhr a écrit :

El 06.04.2017 a las 11:10, Jean-Marc Lasgouttes escribió:


That is not how it works. Actually, I propose to add a rule to our
coding rules that says that no code which does not take in account
zoom and DPI when drawing should be accepted (with the usual
exceptions, of course).


I have not heard about this rule.


This is why there is the word "propose" in the sentence above.


I also sent the patch to the list for
some days before I put it in. I haven't got this info from you.


This is why I did not complain about that. But it is not because a patch 
is in that it is immune of of code review (especially when there are 
user-visible issues).



I don't get it. Seems that I have been away for too long. At first, yes
I can read TeX code, yes I experimented with painting including the
zoom.
I could not find a better solution that I proposed. When zooming out a
lot one got no stroke at all.


I implemented this for you now at 94114fd1. It is even a little bit 
shorter than the original code, I do not think that the price to pay was 
so high for something correct.



However, I still think that we should not leave the WYSIWYM track in
favor of WYSIWYG. The number of strokes is not important within LyX. The
user should see that there will be strokes and that is it. That is the
WYSIWYM concept.


WYSIWYM is not about "I have the right to give an ugly screen output", 
it is about having a visually pleasant output that tells you what your 
output in a meaningful way. The only difference with WYSIWYG  is that 
some things (labels, footnotes, line breaks) are not represented 
visually, but semantically.



As I understand you, you want me to use the currently
selected screen font to draw a '/' character over the text that is
repeated by 0.35em (in pixels). That is quite complicated (at least for
me) and I don't see the benefit. That would be WYSIWYG. Is this really
necessary?


You got it wrong. What I say is a _requirement_ is to have a good output 
at a high zoom level (this is a good way to see what HiDPI users will 
see). Now, how do I find a nice way to have good output at high DPI? 
Wait, LaTeX does output at high DPI !! Let's see what it does and do the 
same if it is not too complicated!


So, what I mean is that mimicking LaTeX is not a requirement, it is a 
trick to help you. And don't come tell me that the new code is more 
complicated than the old one, because it is not.


Now, what I would like to see is some action on the lefteqn patch ;)

JMarc



Re: Build failed in Jenkins: Build branch "master" » ubuntu-xenial-qt4-autotools-extended #145

2017-04-10 Thread Kornel Benko
Am Montag, 10. April 2017 um 09:43:15, schrieb Jean-Marc Lasgouttes 

> Le 10/04/2017 à 09:23, Kornel Benko a écrit :
> > The first number is value in pixels(250) for 2342baselineskip%,
> > the second is value in Big Points for 2342baselineskip%
> >
> > I propose more readable output for check_Length.cpp.
> > For instance the attached.
> 
> Yes, please put it in and fix the test.
> 
> Thanks,
> JMarc

OK, done at 560ebc1

Kornel

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


Re: How far is 2.3.0?

2017-04-10 Thread Jean-Marc Lasgouttes

Le 10/04/2017 à 05:36, Scott Kostyshak a écrit :

First the obvious conclusion, those number are completely arbitrary. Each of
those versions has been a major version on its own. And as usual I propose to
go the way of gcc (not necessarily dropping the first .0 ;-) ). And use simply
a major number and a minor number for the version.

An yearly version on the other hand seems a good compromise. Again gcc is a
good model.


I don't have a strong opinion on this. Does anyone else?


Well, everybody is doing that. It looks like a good reason for _not_ 
doing it :)


Version numbers are just version numbers, I do not think that they make 
as much of a difference than getting versions out fast. In the case of 
firefox and gcc, the important part of the new release scheme was speed. 
The rest is just marketing.


JMarc



Re: Tentative schedule for 2.3.0 release

2017-04-10 Thread Jean-Marc Lasgouttes

Le 10/04/2017 à 05:40, Scott Kostyshak a écrit :

I think there is agreement that master is pretty stable. Besides just a
feeling, I think that this can be confirmed by looking at the trac
tickets with "2.3.0" milestone and tickets with the "regression"
keyword.


Yes, I think it is time to think seriously about it.


I propose the following schedule for the 2.3.0 release process:

alpha:   22 April
feature freeze:  19 May


It was proably discussed last time, but I do not remember: what is the 
rationale for having an alpha before feature freeze? Shouldn't it be the 
opposite?



beta:09 June
RC:  12 July
final:   01 August


1st of august will probably be a slow period. But there may not be a 
better solution.


JMarc


Re: Build failed in Jenkins: Build branch "master" » ubuntu-xenial-qt4-autotools-extended #145

2017-04-10 Thread Jean-Marc Lasgouttes

Le 10/04/2017 à 09:23, Kornel Benko a écrit :

The first number is value in pixels(250) for 2342baselineskip%,
the second is value in Big Points for 2342baselineskip%

I propose more readable output for check_Length.cpp.
For instance the attached.


Yes, please put it in and fix the test.

Thanks,
JMarc



Re: Use Qt 5.9.0 for Mac/Win binaries?

2017-04-10 Thread Jean-Marc Lasgouttes

Le 10/04/2017 à 06:39, Scott Kostyshak a écrit :

Dear all,

I would like to propose that we release our LyX 2.3.0 alpha binaries for
Mac and Windows with Qt 5.9.0beta1 and that we release our final LyX
2.3.0 binaries with the final release of Qt 5.9.0.


Hello,

A question: do we have compelling reasons to use this instead of Qt 5.6 LTS?

I have not followed the recent Qt development, but I find it depressing 
that we have to use the latst and greatest because anything older has 
significant issues. I would expect LTS to be a safe, conservative 
choice, but I may be wrong.


JMarc


Re: Build failed in Jenkins: Build branch "master" » ubuntu-xenial-qt4-autotools-extended #145

2017-04-10 Thread Kornel Benko
Am Samstag, 8. April 2017 um 22:49:48, schrieb Jean-Marc Lasgouttes 

> Le 08/04/2017 à 16:24, Kornel Benko a écrit :
> >> I think this is real breakage and not due to me testing things. Perhaps
> >> related to Uwe's recent changes?
> >
> > Fails under cmake too (e.g. #ctest -R check_Length)
> >
> > Diff for expected and created data
> > #diff /src/tests/regfiles/Length /src/tests/Length_data
> > 18a19
> > > 279990
> > 37a39
> > > 280
> >
> > But these are new length values, so nothing old changed.
>
> Probably related to the fact that a new unit got added, but this
> deserves to be checked by hand.
>
> JMarc

The first number is value in pixels(250) for 2342baselineskip%,
the second is value in Big Points for 2342baselineskip%

I propose more readable output for check_Length.cpp.
For instance the attached.

Kornel

signature.asc
Description: This is a digitally signed message part.
diff --git a/src/tests/check_Length.cpp b/src/tests/check_Length.cpp
index 8311e45..37286e5 100644
--- a/src/tests/check_Length.cpp
+++ b/src/tests/check_Length.cpp
@@ -18,7 +18,7 @@ void test_inPixels()
 	lyxrc.dpi = 72;
 	for (int i = Length::BP; i <= Length::UNIT_NONE; ++i) {
 		Length const l(2342, static_cast(i));
-		cout << l.inPixels(250) << endl;
+		cout << l.inPixels(250) << " pix(250) = " << l.asString() << endl;
 	}
 }
 
@@ -27,7 +27,7 @@ void test_inBP()
 {
 	for (int i = Length::BP; i <= Length::UNIT_NONE; ++i) {
 		Length const l(2342, static_cast(i));
-		cout << l.inBP() << endl;
+		cout << l.inBP() << " BP = " << l.asString() << endl;
 	}
 }