Skip to content

fix macos CI test, add SQLites to coatjava build#1328

Merged
c-dilks merged 14 commits into
developmentfrom
ci-macos
Jul 1, 2026
Merged

fix macos CI test, add SQLites to coatjava build#1328
c-dilks merged 14 commits into
developmentfrom
ci-macos

Conversation

@baltzell

@baltzell baltzell commented Jun 30, 2026

Copy link
Copy Markdown
Collaborator

@baltzell baltzell changed the title cleanup fix macos CI test Jun 30, 2026
@baltzell baltzell marked this pull request as ready for review June 30, 2026 21:04

@c-dilks c-dilks left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

build-coatjava.sh now downloads the SQLite files by default, which could be out of date with respect to the "official" databases. However, these downloaded SQLite files are only used if $CCDB_CONNECTION or $RCDB_CONNECTION point to them (as you do in the CI here); otherwise, the databases that are used by default remain unchanged.

Do I understand that correctly?

@c-dilks

c-dilks commented Jul 1, 2026

Copy link
Copy Markdown
Member

Could we have a more descriptive PR title (for the release notes)?

@baltzell

baltzell commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator Author

Yes, the SQLite snapshots are now included by default, but only used if one figures out how to set $CCDB/RCDB_CONNECTION to point to them.

@baltzell baltzell changed the title fix macos CI test fix macos CI test, add SQLites to coatjava build Jul 1, 2026
@c-dilks c-dilks enabled auto-merge (squash) July 1, 2026 17:16
@c-dilks c-dilks merged commit abe1839 into development Jul 1, 2026
22 checks passed
@c-dilks c-dilks deleted the ci-macos branch July 1, 2026 19:11
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