Skip to content

Modify the src/gmt/CmakeList.txt to build a mbsystem library that better integrated in GMT#1588

Open
joa-quim wants to merge 2 commits into
dwcaress:masterfrom
GenericMappingTools:master
Open

Modify the src/gmt/CmakeList.txt to build a mbsystem library that better integrated in GMT#1588
joa-quim wants to merge 2 commits into
dwcaress:masterfrom
GenericMappingTools:master

Conversation

@joa-quim
Copy link
Copy Markdown
Contributor

Background: I made MB be able to let build on Windows with MSVC (no cygwin or mingw). The non graphical tools for now.

I than realized that the ported-to-be-GMT-modules do not show up when one do gmt --show-modules and trying to access them from GMT.jl (the GMT Julia wrapper) wan not working well. I then asked help to Claude to investigate the problem and it came out with this solution that makes it prefectly fine. Not only it now shows the modules

gmt --show-modules
...
x2sys_put
x2sys_report
x2sys_solve
mbcontour
mbgrd2obj
mbgrdtiff
mbswath

and it also let communicate in-memory data between between MB and Julia environment. Cannot show and example here as I have not write yet a GMT.jl wraper to to these modules and they likely need change to full use the GMT I/O in order that it works. But I have a couple of other written by be that works (Dave, I demonstrated you that 2 years ago at the AGU)

joa-quim added 2 commits May 22, 2026 01:12
Previous version was not displaying the MB ported modules with gmt --show-modules. Now with this change (and a new folder that will follow) it does
@anbj
Copy link
Copy Markdown
Contributor

anbj commented May 22, 2026

Sorry to bring this up here, but: don't you get into problems with #1484?

@joa-quim
Copy link
Copy Markdown
Contributor Author

Nope, that would be a GMT building issue and I never had it.

@anbj
Copy link
Copy Markdown
Contributor

anbj commented May 22, 2026

At my side, building GMT works fine - it's when builing MB-System that this error occurs. But let's take that in the other thread - at some point.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants