D12987: Fix creating temporary directory for R backend

Alexander Semke noreply at phabricator.kde.org
Sun May 20 08:15:20 UTC 2018


asemke added a comment.


  In D12987#265161 <https://phabricator.kde.org/D12987#265161>, @sirgienko wrote:
  
  > @asemke, Could you confirm this R backend bug <https://bugs.kde.org/show_bug.cgi?id=394018> after applying and testing this patch with images? I have found a solution, that fix this for me, but the solution changes mime-types and I begin to doubt, that this is bug of Cantor and not mimetype errors in my system.
  
  
  I can confirm that BUG 394018 and also the issues with the plots is fixed with your change. However, I'm seeing yet another inconsistency in Cantor's code with respect to the names of the temporary folders and files. Check for example maximaexpression.cpp:72 and similar for juliaexpressions and maybe for other backends. Should we remove the subfolder structure in rserver and go for a simple
  
    m_tmpDir = QDir::tempPath() + QString::fromLatin1("/cantor_rserver-%1").arg(getpid());
    QDir dir;
    dir.mkdir(m_tmpDir);
  
  ?

REPOSITORY
  R55 Cantor

REVISION DETAIL
  https://phabricator.kde.org/D12987

To: sirgienko, asemke
Cc: kde-edu, asemke, narvaez, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20180520/c8fbf81f/attachment.html>


More information about the kde-edu mailing list