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

 
 

bug #46081: [octave forge] (nurbs) fails to build in win32 with --enable-64

Submitter:  John Donoghue <lostbard>
Submitted:  Wed 30 Sep 2015 05:22:45 PM UTC
   
 
Category:  Octave Package Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Build Failure
Status:  Fixed Assigned to:  None
Originator Name:  JohnD Open/Closed:  * Closed
Release:  * dev Operating System:  * Microsoft Windows
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Tue 30 Mar 2021 07:08:33 AM UTC, comment #8: 

Closed as fixed with nurbs-1.4.3 (released today) by private request of package maintainer.

Kai Torben Ohlhus <siko1056>
Group Member
Tue 15 Aug 2017 12:33:25 PM UTC, comment #7: 

With --enable-windows-64 --enable-64, also passes 91 tests

John Donoghue <lostbard>
Group Member
Wed 19 Apr 2017 01:19:12 PM UTC, comment #6: 

passes 91 tests with windows64, with patch

Trying --enable-64 next

John Donoghue <lostbard>
Group Member
Tue 18 Apr 2017 01:57:26 PM UTC, comment #5: 

relooking at it, it appears it is actually when compiling for win64 rather than enable-64

I'm recompiling all then will rerun the tests

John Donoghue <lostbard>
Group Member
Tue 18 Apr 2017 09:18:11 AM UTC, comment #4: 

Hi,

The patch looks OK to me, but does it actually fix the package for Octave built with --enable-64 ?

I see a lot of instances of indices declared as 'int' in the package.

I have never built with '--enable-64' so cannot test myself, do all the tests work for you with this patch applied?

@Rafa, I believe geopdes too might need a few changes to work with 64 bit indices, which will be enabled by default in 4.4 ...

c.



Carlo de Falco <cdf>
Group Member
Tue 18 Apr 2017 09:11:29 AM UTC, comment #3: 

Hi, even though I seem to be in the CC list
I had missed the original tracker item.

I'll have a look at the issue ASAP, thanks for the report.

c.

Carlo de Falco <cdf>
Group Member
Mon 17 Apr 2017 06:27:05 PM UTC, comment #2: 

Still an issue with 1.3.13

John Donoghue <lostbard>
Group Member
Wed 30 Sep 2015 05:30:08 PM UTC, comment #1: 

Patch included

(file #35018)

John Donoghue <lostbard>
Group Member
Wed 30 Sep 2015 05:22:45 PM UTC, original submission:  

Attempting to install mxe-octave build octave compiled with --enable-64.

Fails with a  number of errors similar to:


low_level_functions.cc: In function 'int surfderivcpts(octave_idx_type, octave_idx_type, const RowVector&, octave_idx_type, octave_idx_type, const RowVector&, const Matrix&, octave_idx_type, octave_idx_type, octave_idx_type, octave_idx_type, octave_idx_type, NDArray&)':
low_level_functions.cc:309:55: error: call of overloaded 'idx_vector(int)' is ambiguous
    idxva (0) = idx_vector(k); idxva (1) = idx_vector(0);


John Donoghue <lostbard>
Group Member

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #35018:  of-nurbs-1-fixes.patch added by lostbard (4KiB - application/octet-stream)

 

Depends on the following items: None found

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 jwe (Updated the item)
  • -email is unavailable- added by cdf (Posted a comment)
  • -email is unavailable- added by cdf
  • -email is unavailable- added by lostbard (Adding package maintainer)
  • -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
    2021-03-30 siko1056 StatusPatch Submitted Fixed
        Open/ClosedOpen Closed
    2018-07-02 jwe Summary[octave-forge] (nurbs) fails to build in win32 with --enable-64 [octave forge] (nurbs) fails to build in win32 with --enable-64
    2017-08-11 jwe Summary[octave-forge] nurbs fails to build in win32 with --enable-64 [octave-forge] (nurbs) fails to build in win32 with --enable-64
    2017-04-18 cdf Carbon-Copy- Added -email is unavailable-
    2017-04-17 lostbard Carbon-Copy- Added cdf
    2015-09-30 lostbard StatusNone Patch Submitted
    2015-09-30 lostbard Attached File- Added of-nurbs-1-fixes.patch, #35018

    Back to the top

    Powered by Savane 3.13-4448.
    Corresponding source code