bugGNU Octave - Bugs: bug #59343, [octave forge] (symbolic) `pkg...

 
 

bug #59343: [octave forge] (symbolic) `pkg test symbolic` fails in "sympref.m"

Submitter:  Markus Mützel <mmuetzel>
Submitted:  Sat 24 Oct 2020 08:59:18 PM UTC
   
 
Category:  Octave Package Severity:  3 - Normal
Priority:  5 - Normal Item Group:  None
Status:  Fixed Assigned to:  None
Originator Name:  Open/Closed:  * Closed
Release:  * other Operating System:  * Microsoft Windows
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Tue 21 Jun 2022 07:28:05 PM UTC, comment #2: 

Thanks for the fix.

IIUC, that is this change:
https://github.com/cbm755/octsympy/commit/263e8a1091f0855800f614d568ea2c555bc8529d

Closing report.

Markus Mützel <mmuetzel>
Group administrator
Tue 21 Jun 2022 07:21:39 PM UTC, comment #1: 

This can be closed, it should be fixed in "main" branch and upcoming 3.0.0

Colin Macdonald <cbm>
Sat 24 Oct 2020 08:59:18 PM UTC, original submission:  

When running the BISTs of symbolic 2.9.0 in Octave 6.0.92, the tests for "sympref.m" fail with the following error:

>>>>> processing C:\Program Files\GNU Octave\Octave-6.0.92\mingw64\share\octave\packages\symbolic-2.9.0\sympref.m
***** test
 sympref('ipc', 'systmpfile');
 syms x
 delete('tmp_python_cmd.py')
!!!!! test failed
fprintf: invalid stream number = -1
shared variables   scalar structure containing the fields:

    sympref_orig =

      scalar structure containing the fields:

        ipc = default
        display = ascii
        digits = 32
        quiet = 0

***** error <line 1> pycall_sympy__ ('raise ValueError')
!!!!! error failed.
Expected <line 1>, but got <fprintf: invalid stream number = -1>

***** error <line 1> pycall_sympy__ ('raise ValueError', struct('a', 1, 'b', 'word'))
!!!!! error failed.
Expected <line 1>, but got <fprintf: invalid stream number = -1>


This is probably because the current directory during the test is the directory of the .m file. The user might not have write permission for that directory (without elevated or superuser privileges). Could the current directory be changed to `tempdir ()` for these tests?

I've marked as Microsoft Windows because I've tested on that OS. But I believe the error would also happen on Linux if the package is installed in /usr and Octave runs without superuser privileges.

CC'ing Colin Macdonald as the package maintainer.

Markus Mützel <mmuetzel>
Group administrator

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by cbm (Posted a comment)
  • -email is unavailable- added by jwe (Updated the item)
  • -email is unavailable- added by mmuetzel (Submitted the item)
  • -email is unavailable- added by mmuetzel
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only group members can vote.

     

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2022-06-21 mmuetzel StatusNone Fixed
        Open/ClosedOpen Closed
    2020-11-27 jwe Release6.0.92 other
    2020-10-24 mmuetzel Carbon-Copy- Added cbm

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code