Skip to content

basemap: Fix -A in modern mode.#8980

Merged
joa-quim merged 1 commit intomasterfrom
fix-8962
Apr 15, 2026
Merged

basemap: Fix -A in modern mode.#8980
joa-quim merged 1 commit intomasterfrom
fix-8962

Conversation

@joa-quim
Copy link
Copy Markdown
Member

I didn't intend to consume resources (both organic and silicium) with this issue that, as discussed in not really a bug) but since I asked Claude by mistake to address it, and it did it ...

Fix #8962

Assisted: Claude code, Sonnet 4.6

Fix #8962

Assisted: Claude code, Sonnet 4.6
@joa-quim joa-quim requested a review from a team April 15, 2026 15:14
@joa-quim joa-quim merged commit c069488 into master Apr 15, 2026
11 of 14 checks passed
@joa-quim joa-quim deleted the fix-8962 branch April 15, 2026 21:16
@mkononets
Copy link
Copy Markdown

a minor thing but the comment on line 352 refers to the classic mode, while it's the modern mode.

		/* Allow -A (dump map boundary coordinates) in classic mode since it writes a dataset, not PostScript */

@joa-quim
Copy link
Copy Markdown
Member Author

Well, it's inside a condition

if (API->GMT->current.setting.run_mode == GMT_CLASSIC ...

-A is now both classic and modern.

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.

gmt basemap -R... -J... -A fails as a modern-mode one-liner

2 participants