Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Separate Linux and MacOS CI release build jobs #738

Merged
merged 1 commit into from
Feb 15, 2021
Merged

Separate Linux and MacOS CI release build jobs #738

merged 1 commit into from
Feb 15, 2021

Conversation

flother
Copy link
Contributor

@flother flother commented Feb 13, 2021

Compiling MacOS release builds on Linux — as the GitHub Actions CI job does now — means cgo isn't enabled, which means the go-sqlite3 package isn't compiled, which means no GeoPackage support in the MacOS builds.

This pull request separates the jobs so that Linux builds are compiled on the ubuntu-latest runner while MacOS builds are compiled on the macos-latest runner. This means no cross-compilation is required and cgo can run happily.

Resolves #736

@coveralls
Copy link

coveralls commented Feb 13, 2021

Pull Request Test Coverage Report for Build 8d0445ef3-PR-738

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 45.599%

Totals Coverage Status
Change from base Build c1da823d9: 0.0%
Covered Lines: 5382
Relevant Lines: 11803

💛 - Coveralls

Copy link
Member

@ARolek ARolek left a comment

Choose a reason for hiding this comment

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

LGTM! Could you please squash your commits into a single one?

Thanks for the contribution!

Compiling MacOS release builds on Linux, as the GitHub Actions CI job
does now, means cgo isn't enabled, which means the go-sqlite3 package
isn't compiled, which means no GeoPackage support in the MacOS builds.

This commit separates the jobs so that Linux builds are compiled on the
ubuntu-latest runner while MacOS builds are compiled on the macos-latest
runner. This means no cross-compilation is required and cgo can run
happily. For further details, see the discussion at
#736.
@flother
Copy link
Contributor Author

flother commented Feb 14, 2021

Could you please squash your commits into a single one?

Done!

Copy link
Member

@gdey gdey left a comment

Choose a reason for hiding this comment

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

LGTM

@ARolek ARolek merged commit 55d79c4 into go-spatial:v0.13.x Feb 15, 2021
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.

MacOS release builds don't include support for GeoPackage providers
4 participants