swiftly 1.0.0 (new formula)

[SimplyDanny]

• Have you followed the guidelines for contributing?
• Have you ensured that your commits follow the commit style guide?
• Have you checked that there aren't other open pull requests for the same formula update/change?
• Have you built your formula locally with HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
• Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
• Does your build pass brew audit --strict <formula> (after doing HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>)? If this is a new formula, does it pass brew audit --new <formula>?

---

https://github.com/swiftlang/swiftly

[SimplyDanny]

Any idea why the Linux build couldn't locate zlib.h even though I've added the dependency?

[nikitabobko]

Any idea why the Linux build couldn't locate zlib.h even though I've added the dependency?

It looks like xclogparser had the same problem 55d4188 + 1024jp/GzipSwift#71 so they had to patch it

Apparently, the downstream dependency swift-nio-extras doesn't search C headers in ${brew --prefix}.

I don't know what the proper fix should be yet. I've stopped my investigation on checking how they build swift-nio-extras on Linux on GH Actions + how to add system libraries to SPM https://github.com/swiftlang/swift-package-manager/blob/main/Documentation/Usage.md#requiring-system-libraries

[nikitabobko]

Suggestion: what if we restrict the formula to macOS for now and merge it if there are no other complaints?

  # On Linux, SPM can't find zlib installed by brew
  depends_on :macos

At least it will already be available on macOS

Later, whoever needs the support for Linux might work on figuring out the zlib problem

[SimplyDanny]

Suggestion: what if we restrict the formula to macOS for now and merge it if there are no other complaints?

I think that's fine for now. I've restricted the formula to macOS as you suggested. Would appreciate your review, @nikitabobko.

[nikitabobko]

I am not a maintainer of this repo, and I don't have write access. I am just a random guy on the internet who wants this PR merged 🙂

[MikeMcQuaid]

Thanks for the PR @SimplyDanny! I'm fine with shipping this without Linux support if that's the blocker. Would like to see suggestions applied and a better test before that though.

[github-actions]

🤖 An automated task has requested bottles to be published to this PR.

Please do not push to this PR branch before the bottle commits have been pushed, as this results in a state that is difficult to recover from. If you need to resolve a merge conflict, please use a merge commit. Do not force-push to this PR branch.