Skip to content

build fixes for alpha tag#1759

Merged
vithikashah001 merged 1 commit into
NOAA-GFDL:mainfrom
rem1776:alpha-fixes
Aug 25, 2025
Merged

build fixes for alpha tag#1759
vithikashah001 merged 1 commit into
NOAA-GFDL:mainfrom
rem1776:alpha-fixes

Conversation

@rem1776
Copy link
Copy Markdown
Contributor

@rem1776 rem1776 commented Aug 25, 2025

Description
Fixes for the builds for some stuff that broke with the recent cmake updates. There can't be duplicates script names with how I set up ctest, so most of the changes are renaming some test files that were added in with another pr.

Also fixes a test kind macro that snuck in and was breaking the autotools CI.

How Has This Been Tested?
cmake and autotools builds on the amd box with gcc

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules
  • New check tests, if applicable, are included
  • make distcheck passes

Copy link
Copy Markdown
Contributor

@uramirez8707 uramirez8707 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing this!

@vithikashah001 vithikashah001 merged commit 4fdd530 into NOAA-GFDL:main Aug 25, 2025
28 of 29 checks passed
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.

3 participants