Opened 7 years ago

Closed 7 years ago

#1036 closed defect (fixed)

[with patch] optional macaulay2 package does not install

Reported by: malb Owned by: was
Priority: minor Milestone: sage-2.9
Component: packages: standard Keywords:
Cc: Merged in:
Authors: Reviewers:
Report Upstream: Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description

first bison is required and after installing that, this happens:

In file included from ../comp.hpp:9,
                 from ../comp_gb.hpp:6,
                 from lingb.hpp:8,
                 from lingb.cpp:1:
../comp.hpp:7: error: previous declaration of ‘int gbTrace’ with ‘C++’ linkage
../engine.h:1530: error: conflicts with new declaration with ‘C’ linkage

This is with GCC 4.2.3 on 64-bit Debian/testing.

Attachments (1)

macaulay2_gbTrace_linkage.diff (637 bytes) - added by wjp 7 years ago.
gbTrace C++-linkage patch

Download all attachments as: .zip

Change History (5)

comment:1 Changed 7 years ago by mabshoff

Hello,

Macaulay2 not building is a known issue and #10 should take care of that. But after initial ativity about a month back the work on a new Macaulay2 release seems to have slowed down.

Cheers,

Michael

comment:2 Changed 7 years ago by wjp

Attached a small patch. The 'extern int gbTrace' in engine.h is in an extern "C" {} block, while 'int gbTrace' is defined in a .cpp file.

Since it doesn't appear to be patched in Macaulay2 svn, attaching a small patch.

Changed 7 years ago by wjp

gbTrace C++-linkage patch

comment:3 Changed 7 years ago by was

  • Milestone changed from sage-2.10 to sage-2.9
  • Summary changed from optional macaulay2 package does not install to [with patch] optional macaulay2 package does not install

comment:4 Changed 7 years ago by was

  • Resolution set to fixed
  • Status changed from new to closed

I've put this into sage.

Note: See TracTickets for help on using tickets.