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

Restructure macOS app structure #65

Merged
merged 9 commits into from
Dec 20, 2023
Merged

Conversation

rsmmr
Copy link
Member

@rsmmr rsmmr commented Dec 12, 2023

This passes tests now, including full notarization on macOS. I'm planing to review the changes once more (some of them have been sitting in a branch for a while now) and then merge this into main.

  • Silence new clang-tidy warnings.
  • Build without Broker by default.
  • Add helper to table class for getting current system time.
  • Make tables thread-safe.
  • Rework handling of macOS deployment target.
  • Use gawk for auxin scripts.
  • Small macOS tweaks.
  • Rework the EndpointSecurity manager.
  • Support tables that are disabled at first, but may later become active.
  • [macOS] Notarization updates and fixes.

@rsmmr rsmmr force-pushed the topic/robin/topic/robin/macos-es-new branch 3 times, most recently from 4dd7ece to 3b844ba Compare December 12, 2023 17:02
@rsmmr rsmmr force-pushed the topic/robin/topic/robin/macos-es-new branch 2 times, most recently from fae1479 to 32daa45 Compare December 14, 2023 08:48
@rsmmr rsmmr force-pushed the topic/robin/topic/robin/macos-es-new branch from 32daa45 to c3bd582 Compare December 15, 2023 12:36
@rsmmr rsmmr force-pushed the topic/robin/topic/robin/macos-es-new branch from c3bd582 to 5be1193 Compare December 15, 2023 12:51
@rsmmr rsmmr marked this pull request as ready for review December 15, 2023 13:03
@rsmmr rsmmr force-pushed the topic/robin/topic/robin/macos-es-new branch from 5be1193 to 48930db Compare December 20, 2023 10:52
Broker support isn't needed anymore with Zeek >= 5.0. We now build
without it by default. To re-enable Broker support for Zeek 4.x,
you can still configure with ``--enable-broker``.

The pre-build binaries also no longer include Broker support.
This will be used for tables relying on system services that need some
time to initialize.
By default, our deployment target is now the current platform's major
version. That avoids problems with system libraries not matching in
for what they were built.

To set a different deployment target, there's now a corresponding
`configure` option. We set that for CI builds.

As part of this change, we also raise the CI's deployment target to
Ventura.
We now use Apple's new `notarytool` along with App Store Connect
credentials.

In additional to `main`, CI now runs notarization (but not publishing) on a new (protected)
branch `topic/ci-release-test`. That can be used to test notarization
from CI by pushing changes directly to that branch.
@rsmmr rsmmr force-pushed the topic/robin/topic/robin/macos-es-new branch from 48930db to 96019f8 Compare December 20, 2023 11:10
@rsmmr rsmmr merged commit 7a03127 into main Dec 20, 2023
12 checks passed
@rsmmr rsmmr deleted the topic/robin/topic/robin/macos-es-new branch December 20, 2023 11:23
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.

1 participant