bugGNU Octave - Bugs: bug #57392, [octave forge] (image) fails to...

 
 

bug #57392: [octave forge] (image) fails to install on dev octave due to function handle changes

Submitter:  John Donoghue <lostbard>
Submitted:  Tue 10 Dec 2019 06:16:20 PM UTC
   
 
Category:  Octave Package Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Installation Failure
Status:  Fixed Assigned to:  None
Originator Name:  JohnD Open/Closed:  * Closed
Release:  * dev Operating System:  * Any
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Mon 23 Dec 2019 11:24:03 AM UTC, comment #6: 
Andreas Weber <andy1978>
Group Member
Mon 23 Dec 2019 09:43:45 AM UTC, comment #5: 

Wow, this was a fast reaction.

Thanks for the hint, Andreas. Would it be helpful to mention this change somewhere in the NEWS file? (Or would this be too much detail for there?)

Thanks for the fast code fix, Carne!

Hartmut <hardy>
Mon 23 Dec 2019 02:07:02 AM UTC, comment #4: 

Thank you for bringing this up. I have fixed this so that there is no longer an increment operator used within anonymous function.  I would expect the code to also have better performance.

http://hg.code.sf.net/p/octave/image/rev/0b3500b23026

Carnë Draug <carandraug>
Group Member
Sun 22 Dec 2019 09:48:08 PM UTC, comment #3: 

I think this change was made intentionally, see https://savannah.gnu.org/bugs/?57255#comment5

Andreas Weber <andy1978>
Group Member
Sun 22 Dec 2019 09:35:10 PM UTC, comment #2: 

Help! Does anyone know if this error (see comment #0) is due to an INTENTIONAL change in core Octave?

If yes, what is the actual change? What are the new "rules" for anonymous functions?

If not, the bug title should maybe be adapted to mark it as a regression bug of core Octave (instead of a bug in the image package as it is labelled now). And it might be fixed before the upcoming new Octave 6 release.

I would very much appreciate any hint in this respect. Because leaving this bug un-fixed might make the image package disappear from any further Octave release :(

Hartmut <hardy>
Tue 10 Dec 2019 09:08:46 PM UTC, comment #1: 

add people to cc

Avinoam Kalma <avinoam>
Group Member
Tue 10 Dec 2019 06:16:20 PM UTC, original submission:  

When attempting to install the image package in dev octave 6, I get an error:

  • install image failed...parse error near line 49 of file C:\Octave\Octave-6.0.0\mingw64\share\octave\pack

ages\image-2.10.0\labelmatrix.m

  invalid use of operator ++ in anonymous function

>>>   labels = cell2mat (cellfun (@(ind) repmat (n++, 1, numel(ind)),




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

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by andy1978 (Posted a comment)
  • -email is unavailable- added by hardy (Posted a comment)
  • -email is unavailable- added by avinoam (Posted a comment)
  • -email is unavailable- added by avinoam
  • -email is unavailable- added by avinoam
  • -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 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2019-12-23 carandraug StatusNone Fixed
        Open/ClosedOpen Closed
    2019-12-10 avinoam Carbon-Copy- Added carandraug
        Carbon-Copy- Added hardy

    Back to the top

    Powered by Savane 3.13-bb6a.
    Corresponding source code