bugGNU Octave - Bugs: bug #58698, pkg install fails in current 6.0.1...

 
 

bug #58698: pkg install fails in current 6.0.1 octave

Submitter:  John Donoghue <lostbard>
Submitted:  Fri 03 Jul 2020 07:49:07 PM UTC
   
 
Category:  Interpreter Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Unexpected Error or Warning
Status:  Fixed Assigned to:  siko1056
Originator Name:  JohnD Open/Closed:  * Closed
Release:  * 6.0.90 Operating System:  * Any
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Sun 05 Jul 2020 01:51:08 PM UTC, comment #4: 

Thank you for the feedback John.

Kai Torben Ohlhus <siko1056>
Group Member
Sat 04 Jul 2020 08:04:00 PM UTC, comment #3: 

Looks good to me!

John Donoghue <lostbard>
Group Member
Sat 04 Jul 2020 12:38:13 PM UTC, comment #2: 

@John: Sorry, I probably broke it.  My change is reverted, I tried you examples and they worked again.  Can you check if things are working for you again?

Kai Torben Ohlhus <siko1056>
Group Member
Sat 04 Jul 2020 07:31:31 AM UTC, comment #1: 

That is probably a regression from bug #58686.
See also the discussion there.

Markus Mützel <mmuetzel>
Group administrator
Fri 03 Jul 2020 07:49:07 PM UTC, original submission:  

Not sure what commit broke it but on running an pkg install with a package that contains a src folder with oct files but not .m or mex files, the install fails.


error: Element 1 in argument list is not defined
error: called from
    install>copy_built_files at line 426 column 15
    install at line 197 column 7
    pkg at line 566 column 9


The code at that point:


    m = dir (fullfile (src, "*.m"))
    oct = dir (fullfile (src, "*.oct"))
    mex = dir (fullfile (src, "*.mex"))

    filenames = cellfun (@(x) fullfile (src, x),
                         {m.name, oct.name, mex.name},
                         "uniformoutput", false);


The issue is that m and mex (in my test package at least) are empty structs and {m.name} now generates an error rather than returning a empty cell.

Simpler illustration:


m = dir("*.noextmatch");
{m.name}


In octave 6.0.1, the error will be "error: Element 1 in argument list is not defined.". In octave 5.1 it returns an empty cell {}.







John Donoghue <lostbard>
Group Member

 

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

Attach Files:
   
   
Comment:
   

No files currently attached

 

Digest:
   bug dependencies.

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by siko1056 (Posted a comment)
  • -email is unavailable- added by mmuetzel (Posted a comment)
  • -email is unavailable- added by lostbard (Submitted the item)
  •  

    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 8 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2020-07-05 siko1056 StatusReady For Test Fixed
        Open/ClosedOpen Closed
    2020-07-04 siko1056 StatusConfirmed Ready For Test
        Assigned toNone siko1056
    2020-07-04 mmuetzel CategoryNone Interpreter
        StatusNone Confirmed
        Releasedev 6.0.90
        Dependencies- Depends on bugs #58686

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code