bugGNU Octave - Bugs: bug #29266, qhull code mixing C and C++...

 
 

bug #29266: qhull code mixing C and C++ linkage options

Submitter:  None
Submitted:  Fri 19 Mar 2010 02:37:47 PM UTC
   
 
Category:  Configuration and Build System Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Build Failure
Status:  Works For Me Assigned to:  highegg
Originator Name:  John Swensen Originator Email:  -email is unavailable-
Open/Closed:  * Closed Release:  * dev
Operating System:  * Mac OS Fixed Release:  None
Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Mon 10 May 2010 04:39:12 PM UTC, comment #2: 

Closed after 1 month without response to request for re-testing.

Rik <rik5>
Group administrator
Thu 01 Apr 2010 09:48:45 AM UTC, comment #1: 

Builds normally for me on GNU/Linux and qhull 2010.1. Please update both qhull and Octave. If the problem persists, it's probably Mac-specific.

Jaroslav Hajek <highegg>
Fri 19 Mar 2010 02:37:47 PM UTC, original submission:  

When trying to compile DLD-FUNCTIONS/convhullncc (or any other file that involves QHull, for that matter) I am getting an error about the wrong kind of linkage.  It claims that it is declared with C linkage with previous declaration of C++ linkage.  The thing that seems weird is that it has what I think is the appropriate #ifdef's to detect whether C++ is being used and used 'extern C' appropriately.

Also, when I tried to disable qhull using '--without-qhull' the problem still exists.  This seems to imply that this flag does not affect whether qhull is excluded.

Excerpt
-------
In file included from /opt/local-native/include/qhull/qhull_a.h:29,
                 from DLD-FUNCTIONS/convhulln.cc:45:
/opt/local-native/include/qhull/libqhull.h:1089: error: previous declaration of ‘void qh_collectstatistics()’ with ‘C++’ linkage
/opt/local-native/include/qhull/stat.h:523: error: conflicts with new declaration with ‘C’ linkage
/opt/local-native/include/qhull/libqhull.h:1090: error: previous declaration of ‘void qh_printallstatistics(FILE*, const char*)’ with ‘C++’ linkage
/opt/local-native/include/qhull/stat.h:528: error: conflicts with new declaration with ‘C’ linkage

Anonymous

 

(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 rik5 (Posted a comment)
  • -email is unavailable- added by highegg (Posted a comment)
  • -email is unavailable- added by None (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
    2010-05-10 rik5 Open/ClosedOpen Closed
    2010-04-01 highegg Item GroupIncorrect Result Build Failure
        StatusNone Works For Me
        Assigned toNone highegg

    Back to the top

    Powered by Savane 3.13-caa5.
    Corresponding source code