https://bugs.kde.org/show_bug.cgi?id=411645
--- Comment #2 from pcpa <paulo.cesar.pereira.de.andr...@gmail.com> ---
(In reply to Nikita Sirgienko from comment #1)
> Hi
>
> First, if you are familiar with development, then there is
> phabricator.kde.org for suggestions like this.
> Second, there is tests for Cantor sage backend, and on my machine (SageMath
> 8.7 based on python2) one of them is failed (TestSage::testPlot(),
> testsage.cpp::92).
>
> So, on this moment, I can't accept the patch, because it breaks appearing of
> plots appearing for SageMath, based on python2.
>
> But, anyway, this problem is important, so, if this patch won't be improved
> for working with Python2, I think, I just will add a option "SageMath python
> 3 compatibility".
Many thanks for checking it. I am not much familiar with phabricator.kde.org
For some reason, while working on parallel with several issues related
to the sagemath rpm package, when submitting the report I though the patch
could be simplified, but it cannot.
The proper line for two arguments would be:
print("%s %s" % ('____TMP_DIR____', sage.misc.misc.SAGE_TMP))
python2 would generate different output. For example:
$ python2 -c 'print("foo", "bar")'
('foo', 'bar')
$ python3 -c 'print("foo", "bar")'
foo bar
$ python2 -c 'print("%s %s" % ("foo", "bar"))'
foo bar
I was actually testing a patch with the later version, but got confused and
for some reason thought it could be simplified.
--
You are receiving this mail because:
You are watching all bug changes.
