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

make check fails on macOS Monterey (macos-12) #304

Closed
carlocab opened this issue Sep 29, 2022 · 5 comments · Fixed by #305
Closed

make check fails on macOS Monterey (macos-12) #304

carlocab opened this issue Sep 29, 2022 · 5 comments · Fixed by #305

Comments

@carlocab
Copy link

We're trying to update libmaxminddb for Homebrew at Homebrew/homebrew-core#111943. We encountered the following failure in make check:

  not ok 1 - No libs were found to test
  #   Failed test 'No libs were found to test'
  #   at ./external_symbols_t.pl line 53.
  1..1

Complete CI logs available at https://github.com/Homebrew/homebrew-core/actions/runs/3148372901.

The error for ARM Monterey can be found here, and on Intel Monterey here.

Any assistance would be appreciated.

@carlocab carlocab changed the title make check fails on macOS Monterey make check fails on macOS Monterey (macos-12) Sep 29, 2022
@oschwald
Copy link
Member

Can you confirm that make check with 1.6.0 worked on Monterey? I don't know of any relevant differences, but it would narrow the possibilities.

@carlocab
Copy link
Author

It did, yes. We distribute binaries of 1.6.0 built on Monterey, and we run make check during every build of libmaxminddb.

Note, however, that the build environment from when we were able to build 1.6.0 on Monterey and the one where we are seeing failures now is not exactly identical: we've upgraded to Xcode 14 since then, and we've likely upgraded to a newer minor version of Monterey too. Our build of 1.6.0 was built with Xcode 13.1.

I mention this because we've seen other issues with Xcode 14 (e.g. iains/gcc-12-branch#6). The linked issue is a linker bug -- I'm not sure if that could have anything to do with the failing test.

I'm not aware of instances of minor version upgrades ever causing regressions.

@oschwald
Copy link
Member

Would it be possible to force a rebuild of 1.6.0 with the newer environment? I just reviewed the diff and I don't see anything that is obviously related to this issue. It seems much more likely to be an environment issue (e.g., some sort of automake change).

@oschwald
Copy link
Member

I was able to find a macOS user internally who reproduced the failure. It appears that it isn't specific to this release, but also happened with 1.6.0. I suspect previous builds by Homebrew had that test skipped due to missing test dependencies and upgrading Xcode caused those dependencies to be available.

We will likely just skip that test on non-Linux systems as it appears to assume a Linux-like environment and it is primarily beneficial as an author test rather than testing that the build was successful.

@carlocab
Copy link
Author

carlocab commented Oct 1, 2022

Thanks for the lightning-quick fix!

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

Successfully merging a pull request may close this issue.

2 participants