In D12504#313996, @filipesaraiva wrote:

@sirgienko thanks, it fix the current problem with Scilab backend (I could to reproduce it). I think while we don't implement T5382, we need to fix the problem with this backend.

@sirgienko thanks, it fix the current problem with Scilab backend (I could to reproduce it). I think while we don't implement T5382, we need to fix the problem with this backend.

+1

Given that v1.0 was released recently with a lot of advertisement around this and that we support this version without any know issues, I'm fine with this. But maybe somebody else has some concerns here.

Seems to be possible in the current code already. Closing this task

Review in progress at D14738.

In D14780#308005, @sirgienko wrote:

@filipesaraiva, after 90b001d1b01556cc5d5b60791e9b43e6dfcff0b4, do you still have build errors?

@filipesaraiva, after 90b001d1b01556cc5d5b60791e9b43e6dfcff0b4, do you still have build errors?

In D14780#307604, @filipesaraiva wrote:

In D14780#307590, @asemke wrote:

There is no jl_string_ptr in Julia 0.4.7. Are you using older includes with newer libraries?

Maybe it is a very old piece of code. I am using 0.6.4 version. Before my change, the code was using jl_string_ptr if minor version > 5, but this function doesn't exist for that version. Which version are you using?

In D14780#307590, @asemke wrote:

There is no jl_string_ptr in Julia 0.4.7. Are you using older includes with newer libraries?

There is no jl_string_ptr in Julia 0.4.7. Are you using older includes with newer libraries?

Add forgotten changes and fix build error

@sirgienko can we completely get rid of Python3Session and Python2Session and use one single class PythonSession only? We can have PythonSession with an additional parameter for python version and depending on this parameter we'll start the proper python_server.

In D14680#306377, @sirgienko wrote:

Hi @filipesaraiva, in this moment Python 3 use dbus for communication with server. And Python 2 don't use server, so we have bug 363606. In this commit I apply python server code from Python 3 also for Python 2. In future we plan remove dbus code and use QProcess (or KProcess) for this.

In D14680#306373, @filipesaraiva wrote:

Hi @sirgienko, what about the support for this backend in other OS? In past we throught about drop dbus because it is unfortunately not supported in Windows and OS X.

Hi @sirgienko, what about the support for this backend in other OS? In past we throught about drop dbus because it is unfortunately not supported in Windows and OS X.

Merge PythonDbusSession to PythonSession

Merge work branch realise-T4540 in master

Add compatibility for julia 0.4.7

This doesn't compile with julia 0.4.7

Add @asemke suggesteons.

Close as outdated without merging.

Update

Update

Change store code

In D13688#282047, @sirgienko wrote:

This code works, but there are two problems

1. I can't save, what right tab was choosen before user changes worksheet
2. This code not good integrate in existing code @asemke, I hope, you could help with second problem.

Hi, @filipesaraiva, currect master branch have all this features (there are some bugs, but not critical, I think we fix it coming soon).
So, I think, this task is done.
We still use dbus in R backend, but we can changes it later.
So, I will close this task as done on 7 July, if you don't reply until this date.

Hi, @filipesaraiva, @rishabhg, as I see qprocess_port branch isn't finished and was in a worse state than currect master branch.
On master branch we have working plots and correct highlighting and I see, that qprocess_port haven't these features, so merging qprocess_port with master will be regression.
That more important, last commit in this branch was 14 Feb 2018, so, as I see, developing on this branch stopped.
So, I will close this task as outdated on 7 July, if you don't reply until this date.

This code works, but there are two problems

1. I can't save, what right tab was choosen before user changes worksheet
2. This code not good integrate in existing code

@asemke, I hope, you could help with second problem.

Update comments

Update error message

Set versions 8.1 and 8.2 as recommends

Hello @sirgienko and @asemke, thank you for all your work in order to improve Cantor. I am following the commits and reviews, please keep the good work and go ahead!

After D13550(54f93b4b949e6b8e8222776f4681eb4bba3aa147) we can use Cantor without specify QT_PLUGIN_PATH.
But for tests it's not true, so for running tests we need use QT_PLUGIN_PATH.

Add logic for situation, when tempory file don't exist

Add const

Fix comment messages

Move code from backend.cpp to main.cpp: it guaranteeses, that plugin path will be modified before using

I have tested this changes with installation path, which contains utf8 symbols (I have used japan language). Also I checked installation in user directory (without sudo)
As I see, all works fine, but Cantor don't install correctly, if path to install is set: we loose Qalculate icon and buttons like "Worksheet evaluate", "Interapt Calculation", etc.
I think, I fix it soon.