Re: Test results for 2.2.0dev on Mac

2016-01-31 Thread Stephan Witt
Am 29.01.2016 um 22:15 schrieb Georg Baum : > > Stephan Witt wrote: > >> Am 24.01.2016 um 19:08 schrieb Georg Baum >> : >>> >>> This is the idea, yes, but before implementing it I'd like to know >>> whether it works in principle.

Re: Test results for 2.2.0dev on Mac

2016-01-31 Thread Georg Baum
Stephan Witt wrote: > After applying the patches I get: > > $ (cd lyx-build/cmake/2.2.0dev;ctest -C debug -R tex2lyx) > Test project /Users/stephan/git/lyx-build/cmake/2.2.0dev > > … > > 86% tests passed, 4 tests failed out of 28 > > Label Time Summary: > cmplyx = 21.45 sec (14 tests)

Re: Test results for 2.2.0dev on Mac

2016-01-29 Thread Georg Baum
Stephan Witt wrote: > Am 24.01.2016 um 19:08 schrieb Georg Baum > : >> >> This is the idea, yes, but before implementing it I'd like to know >> whether it works in principle. This would require a two step approach for >> the case that no command line option is

Re: Test results for 2.2.0dev on Mac

2016-01-24 Thread Stephan Witt
Am 24.01.2016 um 14:50 schrieb Georg Baum : > > Stephan Witt wrote: > >> Am 24.01.2016 um 13:00 schrieb Stephan Witt : >> I’ve tried to follow the tokenizing and parsing program flow. >> >> While processing the „latin1“ option of „inputenc“ the

Re: Test results for 2.2.0dev on Mac

2016-01-24 Thread Georg Baum
Stephan Witt wrote: > For latin1 documents (e.g. algo2e.tex) it works better. The test now > passed. > > For CJK.lex it’s obviously a problem: > $ /Users/stephan/git/lyx-build/cmake/2.2.0dev/bin/Debug/tex2lyx -f > -roundtrip /Users/stephan/git/lyx/src/tex2lyx/test/CJK.tex >

Re: Test results for 2.2.0dev on Mac

2016-01-24 Thread Georg Baum
Stephan Witt wrote: > You are asking for running tex2lyx in the debugger? Yes, I’m able to do > so. What should I try exactly? > > I started tex2lyx in debugger with these options: > -f > -roundtrip > /Users/stephan/git/lyx/src/tex2lyx/test/algo2e.tex > /Users/stephan/git/lyx-build/algo2e.lyx >

Re: Test results for 2.2.0dev on Mac

2016-01-24 Thread Stephan Witt
Am 24.01.2016 um 13:00 schrieb Stephan Witt : > > Am 24.01.2016 um 12:12 schrieb Georg Baum : >> >> Stephan Witt wrote: >> >>> You are asking for running tex2lyx in the debugger? Yes, I’m able to do >>> so. What should I try exactly? >>> >>> I

Re: Test results for 2.2.0dev on Mac

2016-01-24 Thread Georg Baum
Stephan Witt wrote: > Am 24.01.2016 um 13:00 schrieb Stephan Witt : > I’ve tried to follow the tokenizing and parsing program flow. > > While processing the „latin1“ option of „inputenc“ the encoding of > the idocstream is changed. (Line 395 of support/docstream.cpp is called) >

Re: Test results for 2.2.0dev on Mac

2016-01-24 Thread Georg Baum
Stephan Witt wrote: > Yes, we shouldn’t change that on the fly. If we change it we have to > provide a migration path to copy the current contents to the new location. > Not very difficult but a change in configure.py would be required. > > Since we don’t want to use different settings for Lyx,

Re: Test results for 2.2.0dev on Mac

2016-01-24 Thread Stephan Witt
Am 24.01.2016 um 12:12 schrieb Georg Baum : > > Stephan Witt wrote: > >> You are asking for running tex2lyx in the debugger? Yes, I’m able to do >> so. What should I try exactly? >> >> I started tex2lyx in debugger with these options: >> -f >> -roundtrip >>

Re: Test results for 2.2.0dev on Mac

2016-01-24 Thread Stephan Witt
Am 24.01.2016 um 15:07 schrieb Georg Baum : > > Stephan Witt wrote: > >> Yes, we shouldn’t change that on the fly. If we change it we have to >> provide a migration path to copy the current contents to the new location. >> Not very difficult but a change in

Re: Test results for 2.2.0dev on Mac

2016-01-24 Thread Georg Baum
Stephan Witt wrote: > ATM, the directory LyX is using is "$HOME/Library/Application > Support/PACKAGE“ > > PACKAGE is LyX-2.2 on Mac when version suffix „-2.2“ is used. Looks like QtApplication does something different here than QtCoreApplication. >> Stephan, does it work? Note that this is

Re: Test results for 2.2.0dev on Mac

2016-01-24 Thread Stephan Witt
Am 24.01.2016 um 12:02 schrieb Georg Baum : > > Stephan Witt wrote: > >> ATM, the directory LyX is using is "$HOME/Library/Application >> Support/PACKAGE“ >> >> PACKAGE is LyX-2.2 on Mac when version suffix „-2.2“ is used. > > Looks like QtApplication does

Re: Test results for 2.2.0dev on Mac

2016-01-24 Thread Georg Baum
Scott Kostyshak wrote: > Do I understand correctly that this issue is no longer a beta blocker or > is there still something that needs to be done here before beta? The original reason why I viewed it as beta blocker (general user or sytem directory problems) turned out to be wrong. I would

Re: Test results for 2.2.0dev on Mac

2016-01-24 Thread Stephan Witt
Am 24.01.2016 um 18:17 schrieb Georg Baum : > > Scott Kostyshak wrote: > >> Do I understand correctly that this issue is no longer a beta blocker or >> is there still something that needs to be done here before beta? > > The original reason why I viewed it as

Re: Test results for 2.2.0dev on Mac

2016-01-24 Thread Scott Kostyshak
On Sun, Jan 24, 2016 at 03:36:09PM +0100, Georg Baum wrote: > OK, but I don't think it is too important to know whether the old compiler > did handle that. There is a FIXME near the imbue call anyway, since we do > not know whether it is standard conforming what we do there. > > I'll have a

Re: Test results for 2.2.0dev on Mac

2016-01-24 Thread Stephan Witt
Am 24.01.2016 um 19:08 schrieb Georg Baum : > > Stephan Witt wrote: > >> Am 24.01.2016 um 18:17 schrieb Georg Baum >> : >>> >>> Stephan, does the algo2e.tex test succeed if you apply the attached patch >>> on a clean tree? >> >>

Re: Test results for 2.2.0dev on Mac

2016-01-24 Thread Georg Baum
Stephan Witt wrote: > Am 24.01.2016 um 18:17 schrieb Georg Baum > : >> >> Stephan, does the algo2e.tex test succeed if you apply the attached patch >> on a clean tree? > > No, it doesn’t work. With the patch I get: It turned out that we used two different names

Re: Test results for 2.2.0dev on Mac

2016-01-24 Thread Scott Kostyshak
On Sun, Jan 24, 2016 at 06:17:18PM +0100, Georg Baum wrote: > Scott Kostyshak wrote: > > > Do I understand correctly that this issue is no longer a beta blocker or > > is there still something that needs to be done here before beta? > > The original reason why I viewed it as beta blocker

Re: Test results for 2.2.0dev on Mac

2016-01-23 Thread Stephan Witt
Am 22.01.2016 um 22:32 schrieb Georg Baum : > > Stephan Witt wrote: > >> No. It’s not wrong. You’re right with the use of the _own_ userdir >> for tests. But the real life matters too. It’s the wrong userdir >> for standard orpeation. And this is Georg message,

Re: Test results for 2.2.0dev on Mac

2016-01-23 Thread Stephan Witt
> Am 22.01.2016 um 21:54 schrieb Georg Baum : > > Stephan Witt wrote: > >> $ lyx-build/cmake/2.2.0dev/bin/Debug/tex2lyx >> tex2lyx: Not enough arguments. >> Usage: tex2lyx [options] infile.tex [outfile.lyx] >> … >> -sysdir SYSDIR Set system directory to

Re: Test results for 2.2.0dev on Mac

2016-01-22 Thread Georg Baum
Stephan Witt wrote: > There is some change required because tex2lyx has the same problem itself. > I’ve changed this for the lyx binary here: > > http://www.lyx.org/trac/changeset/6e9bd23/lyxgit > > Attached is a similar change to correct it for tex2lyx. Ok to commit? I do not understand why

Re: Test results for 2.2.0dev on Mac

2016-01-22 Thread Stephan Witt
Am 22.01.2016 um 21:01 schrieb Georg Baum : > > Stephan Witt wrote: > >> There is some change required because tex2lyx has the same problem itself. >> I’ve changed this for the lyx binary here: >> >> http://www.lyx.org/trac/changeset/6e9bd23/lyxgit >> >>

Re: Test results for 2.2.0dev on Mac

2016-01-22 Thread Georg Baum
Stephan Witt wrote: > $ lyx-build/cmake/2.2.0dev/bin/Debug/tex2lyx > tex2lyx: Not enough arguments. > Usage: tex2lyx [options] infile.tex [outfile.lyx] > … > -sysdir SYSDIR Set system directory to SYSDIR. > Default: /Users/stephan/git/lyx/lib/ > -userdir USERDIR Set user directory to

Re: Test results for 2.2.0dev on Mac

2016-01-22 Thread Kornel Benko
Am Freitag, 22. Januar 2016 um 21:01:42, schrieb Georg Baum > Stephan Witt wrote: > > > There is some change required because tex2lyx has the same problem itself. > > I’ve changed this for the lyx binary here: > > > >

Re: Test results for 2.2.0dev on Mac

2016-01-22 Thread Georg Baum
Stephan Witt wrote: >> Am 22.01.2016 um 05:43 schrieb Scott Kostyshak : >> >> Note that if this is a regression and if the tex2lyx test used to work >> before, it would be very easy to do a 'git bisect'. It could be done >> automatically by doing 'git bisect run' (When a ctest

Re: Test results for 2.2.0dev on Mac

2016-01-22 Thread Georg Baum
Stephan Witt wrote: > Am 22.01.2016 um 21:01 schrieb Georg Baum > : >> >> Stephan Witt wrote: >> >>> There is some change required because tex2lyx has the same problem >>> itself. I’ve changed this for the lyx binary here: >>> >>>

Re: Test results for 2.2.0dev on Mac

2016-01-22 Thread Georg Baum
Stephan Witt wrote: > No. It’s not wrong. You’re right with the use of the _own_ userdir > for tests. But the real life matters too. It’s the wrong userdir > for standard orpeation. And this is Georg message, IMO. Yes. Also, depending on the reaason for the wrong default user dir, also the

Re: Test results for 2.2.0dev on Mac

2016-01-22 Thread Stephan Witt
Am 21.01.2016 um 22:24 schrieb Georg Baum : > > Stephan Witt wrote: > >> $ lyx-build/cmake/2.2.0dev/bin/Debug/tex2lyx >> tex2lyx: Not enough arguments. >> Usage: tex2lyx [options] infile.tex [outfile.lyx] >> … >> -sysdir SYSDIR Set system directory to SYSDIR.

Re: Test results for 2.2.0dev on Mac

2016-01-22 Thread Stephan Witt
Am 22.01.2016 um 12:16 schrieb Kornel Benko : > > Am Donnerstag, 21. Januar 2016 um 22:24:58, schrieb Georg Baum > >> Stephan Witt wrote: >> >>> $ lyx-build/cmake/2.2.0dev/bin/Debug/tex2lyx >>> tex2lyx: Not enough arguments. >>> Usage: tex2lyx

Re: Test results for 2.2.0dev on Mac

2016-01-22 Thread Kornel Benko
Am Donnerstag, 21. Januar 2016 um 22:24:58, schrieb Georg Baum > Stephan Witt wrote: > > > $ lyx-build/cmake/2.2.0dev/bin/Debug/tex2lyx > > tex2lyx: Not enough arguments. > > Usage: tex2lyx [options] infile.tex [outfile.lyx] > > … > > -sysdir SYSDIR Set

Re: Test results for 2.2.0dev on Mac

2016-01-22 Thread Kornel Benko
Am Freitag, 22. Januar 2016 um 13:34:41, schrieb Stephan Witt > >> Ah, now we are getting somewhere. The systemdir is fine, the userdir is > >> wrong. It should be > >> > >> /Users/stephan/Library/Application Support/LyX/ > > > > Wrong. For tests we use _own_ userdir

Re: Test results for 2.2.0dev on Mac

2016-01-21 Thread Georg Baum
Stephan Witt wrote: > $ lyx-build/cmake/2.2.0dev/bin/Debug/tex2lyx > tex2lyx: Not enough arguments. > Usage: tex2lyx [options] infile.tex [outfile.lyx] > … > -sysdir SYSDIR Set system directory to SYSDIR. > Default: /Users/stephan/git/lyx/lib/ > -userdir USERDIR Set user directory to

Re: Test results for 2.2.0dev on Mac

2016-01-21 Thread Stephan Witt
> Am 22.01.2016 um 05:43 schrieb Scott Kostyshak : > > On Thu, Jan 21, 2016 at 10:24:58PM +0100, Georg Baum wrote: > >> Does LyX itself tell the correct userdir for both cases? I am now pretty >> sure that support::Package or some related code is broken on OS X, at least >>

Re: Test results for 2.2.0dev on Mac

2016-01-21 Thread Scott Kostyshak
On Thu, Jan 21, 2016 at 10:24:58PM +0100, Georg Baum wrote: > Does LyX itself tell the correct userdir for both cases? I am now pretty > sure that support::Package or some related code is broken on OS X, at least > for tex2lyx, which means that this is not only a minor test issue, but a > beta

Re: Test results for 2.2.0dev on Mac

2016-01-20 Thread Stephan Witt
> Am 20.01.2016 um 22:31 schrieb Georg Baum : > > Kornel Benko wrote: > >> Am Mittwoch, 20. Januar 2016 um 06:58:20, schrieb Georg Baum >> >>> Stephan Witt wrote: >>> Me too - I really meant *somewhere* - perhaps

Re: Test results for 2.2.0dev on Mac

2016-01-20 Thread Georg Baum
Kornel Benko wrote: > Am Mittwoch, 20. Januar 2016 um 06:58:20, schrieb Georg Baum > >> Stephan Witt wrote: >> >> > Me too - I really meant *somewhere* - >> > perhaps tex2lyx reads some file from outside the build directory. >> >> No. It reads from the system

Re: Test results for 2.2.0dev on Mac

2016-01-20 Thread Kornel Benko
Am Dienstag, 19. Januar 2016 um 14:10:09, schrieb Scott Kostyshak > On Tue, Jan 19, 2016 at 08:10:59AM +0100, Stephan Witt wrote: > > > I have this: > > > > $ cat > > /Users/stephan/git/lyx-build/cmake/2.2.0dev/Testing/.lyx/layouts/testalgorithm2e.layout > > > > #% Do not

Re: Test results for 2.2.0dev on Mac

2016-01-20 Thread Kornel Benko
Am Dienstag, 19. Januar 2016 um 08:10:59, schrieb Stephan Witt > Am 19.01.2016 um 05:13 schrieb Scott Kostyshak : > > > > On Mon, Jan 18, 2016 at 09:53:26PM +0100, Stephan Witt wrote: > >> Am 18.01.2016 um 21:48 schrieb Kornel Benko : > >>> >

Re: Test results for 2.2.0dev on Mac

2016-01-20 Thread Kornel Benko
Am Mittwoch, 20. Januar 2016 um 06:58:20, schrieb Georg Baum > Stephan Witt wrote: > > > Am 19.01.2016 um 20:10 schrieb Scott Kostyshak : > >> > >> On Tue, Jan 19, 2016 at 08:10:59AM +0100, Stephan Witt wrote: > >> > >>> BTW, when debugging

Re: Test results for 2.2.0dev on Mac

2016-01-19 Thread Stephan Witt
Am 19.01.2016 um 20:10 schrieb Scott Kostyshak : > > On Tue, Jan 19, 2016 at 08:10:59AM +0100, Stephan Witt wrote: > >> I have this: >> >> $ cat >> /Users/stephan/git/lyx-build/cmake/2.2.0dev/Testing/.lyx/layouts/testalgorithm2e.layout >> >> #% Do not delete the line below;

Re: Test results for 2.2.0dev on Mac

2016-01-19 Thread Scott Kostyshak
On Tue, Jan 19, 2016 at 08:10:59AM +0100, Stephan Witt wrote: > I have this: > > $ cat > /Users/stephan/git/lyx-build/cmake/2.2.0dev/Testing/.lyx/layouts/testalgorithm2e.layout > > #% Do not delete the line below; configure depends on this > # \DeclareLaTeXClass{testalgorithm2e} > Format 55

Re: Test results for 2.2.0dev on Mac

2016-01-19 Thread Georg Baum
Stephan Witt wrote: > Am 19.01.2016 um 20:10 schrieb Scott Kostyshak : >> >> On Tue, Jan 19, 2016 at 08:10:59AM +0100, Stephan Witt wrote: >> >>> BTW, when debugging tex2lyx it complains about missing USERDIR and I >>> have to add -userdir >>>

Re: Test results for 2.2.0dev on Mac

2016-01-18 Thread Scott Kostyshak
On Mon, Jan 18, 2016 at 09:53:26PM +0100, Stephan Witt wrote: > Am 18.01.2016 um 21:48 schrieb Kornel Benko : > > > > Am Montag, 18. Januar 2016 um 15:26:06, schrieb Scott Kostyshak > > > >> On Mon, Jan 18, 2016 at 09:23:47PM +0100, Stephan Witt wrote: > >>>

Re: Test results for 2.2.0dev on Mac

2016-01-18 Thread Stephan Witt
Am 19.01.2016 um 05:13 schrieb Scott Kostyshak : > > On Mon, Jan 18, 2016 at 09:53:26PM +0100, Stephan Witt wrote: >> Am 18.01.2016 um 21:48 schrieb Kornel Benko : >>> >>> Am Montag, 18. Januar 2016 um 15:26:06, schrieb Scott Kostyshak >>>

Re: Test results for 2.2.0dev on Mac

2016-01-18 Thread Stephan Witt
Am 18.01.2016 um 21:48 schrieb Kornel Benko : > > Am Montag, 18. Januar 2016 um 15:26:06, schrieb Scott Kostyshak > >> On Mon, Jan 18, 2016 at 09:23:47PM +0100, Stephan Witt wrote: >>> Running ctest there are 16 failed tests in tex2lyx. >>> Is this to be

Re: Test results for 2.2.0dev on Mac

2016-01-18 Thread Scott Kostyshak
On Mon, Jan 18, 2016 at 09:23:47PM +0100, Stephan Witt wrote: > Running ctest there are 16 failed tests in tex2lyx. > Is this to be expected? I don't think so. All of the tex2lyx tests pass for me. > Where are the details to analyze it further? In the CMake build directory, look at

Test results for 2.2.0dev on Mac

2016-01-18 Thread Stephan Witt
Running ctest there are 16 failed tests in tex2lyx. Is this to be expected? Where are the details to analyze it further? Stephan $ (cd lyx-build/cmake/2.2.0dev;ctest -C debug ) ... 92% tests passed, 16 tests failed out of 202 Label Time Summary: cmplyx = 21.99 sec (14 tests)

Re: Test results for 2.2.0dev on Mac

2016-01-18 Thread Kornel Benko
Am Montag, 18. Januar 2016 um 15:26:06, schrieb Scott Kostyshak > On Mon, Jan 18, 2016 at 09:23:47PM +0100, Stephan Witt wrote: > > Running ctest there are 16 failed tests in tex2lyx. > > Is this to be expected? > > I don't think so. All of the tex2lyx tests pass for me.