Opened 8 years ago

Last modified 8 years ago

#51437 closed defect

Should qscintilla force C++ linkage? — at Version 1

Reported by: MarcusCalhoun-Lopez (Marcus Calhoun-Lopez) Owned by: michaelld (Michael Dickens)
Priority: Normal Milestone:
Component: ports Version: 2.3.4
Keywords: Cc:
Port: qscintilla

Description (last modified by ryandesign (Ryan Carsten Schmidt))

Many of the header files force C++ linkages QScintilla but only on Macs

#ifdef __APPLE__                                                                                                                                                                                           
extern "C++" {                                                                                                                                                                                             
#endif 

This causes a problems when trying to build octave with GCC:

Why forced C++ linkage is needed (but only on Macs) is not clear to me.
I have posted a question on the upstream mailing list, but so far, no response.

I have attached a patch that seems to fix the problem, but it seems a little dangerous without knowing why it was done in the first place.

Change History (3)

Changed 8 years ago by MarcusCalhoun-Lopez (Marcus Calhoun-Lopez)

Attachment: Portfile.diff added

Changed 8 years ago by MarcusCalhoun-Lopez (Marcus Calhoun-Lopez)

Attachment: patch-remove_extern.diff added

comment:1 Changed 8 years ago by ryandesign (Ryan Carsten Schmidt)

Description: modified (diff)
Note: See TracTickets for help on using tickets.