Bug #876

gmt-5.2.1 universal build failure

Added by Takeshi about 1 year ago. Updated 12 months ago.

Status:ClosedStart date:2016-03-19
Priority:NormalDue date:
Assignee:Paul% Done:

100%

Category:-
Target version:Candidate for next bugfix release
Affected version:other Platform:Mac OS X

Description

Because of inconsistent defis of dsyev_ build fails with gmt_vector.c. dsyev_ is already defined in Accelerate/Accelerate.h for APPLE, included in gmt_fft.h and gmt_grdio.h. See the patch.

patch-src-gmt_vector.c.diff Magnifier (563 Bytes) Takeshi, 2016-03-19 23:20

Associated revisions

Revision 15908
Added by Paul about 1 year ago

Address issue #876

History

#1 Updated by Paul about 1 year ago

  • Status changed from New to In Progress

Thanks. Curious about this since we are all using OS X and not getting this problem. Are you doing something different, e.g., another compiler or special flags?

#2 Updated by Takeshi about 1 year ago

I have not been aware of the problem either. The problem was found by the report by one of the users who tried to compile for both x86_64 and i386 in MacPorts. This is possible by adding +universal. When -arch i386 is specified, LP64 symbol is not defined and integers in cblas.h become long int, which is now 64-bit. Even if this problem does not occur it is a good idea to avoid redefinition.

#3 Updated by Paul about 1 year ago

  • Status changed from In Progress to Resolved
  • Assignee set to Paul
  • Target version set to Candidate for next bugfix release
  • % Done changed from 0 to 100

Agreed, I have applied the fix in 5.2.2 (svn) and trunk. Thanks for the patch. In r15909.

#4 Updated by Paul 12 months ago

  • Status changed from Resolved to Closed

Fixed, closing the issue.

Also available in: Atom PDF