Skip to content

otf2bdf 3.1_p1 #221922

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

Merged
merged 2 commits into from
May 1, 2025
Merged

otf2bdf 3.1_p1 #221922

merged 2 commits into from
May 1, 2025

Conversation

samford
Copy link
Member

@samford samford commented Apr 29, 2025

  • 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>?

I haven't been able to identify a way of finding version information for otf2bdf from the current stable source, so I left it alone for some time. We recently discussed checking the GitHub repository's tags but this doesn't align with the stable source.

The software hasn't been meaningfully updated since 2015 but the GitHub repository contains a few commits from 2021. The 3.1 tag is for the code as it was in 2015 but there's also a 3.1_p1 tag that includes the 2021 changes (the tags were created out of order, so it looks like 3.1 is newer than 3.1_p1).

This updates the formula to 3.1_p1, using the tag tarball from GitHub, and adds a livecheck block that checks the Git tags. The build was failing on Linux CI with an ft2build.h error, due to otf2bdf.c using #include <ft2build.h> instead of <freetype2/ft2build.h>, but the CFLAGS snippet from other formulae appears to resolve the error here as well.

@samford samford added the livecheck Issues or PRs related to livecheck label Apr 29, 2025
@samford samford mentioned this pull request Apr 29, 2025
6 tasks
@github-actions github-actions bot added the autosquash Automatically squash pull request commits according to Homebrew style. label May 1, 2025
@samford samford added the CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. label May 1, 2025
@samford samford force-pushed the otf2bdf-update-stable branch from 0c0dc58 to 909c47f Compare May 1, 2025 12:39
I haven't been able to identify a way of finding version information
for `otf2bdf` from the current `stable` source, so I left it alone
for some time. We recently discussed checking the GitHub repository's
tags but this doesn't align with the `stable` source.

The software hasn't been meaningfully updated since 2015 but the
GitHub repository contains a few commits from 2021. The `3.1` tag is
for the code as it was in 2015 but there's also a `3.1_p1` tag that
includes the 2021 changes (the tags were created out of order, so
it looks like `3.1` is newer than `3.1_p1`).

This updates the formula to `3.1_p1`, using the tag tarball from
GitHub, and adds a `livecheck` block that checks the Git tags. The
build was failing on Linux CI with an `ft2build.h` error, due to
`otf2bdf.c` using `#include <ft2build.h>` instead of
`<freetype2/ft2build.h>`, but the `CFLAGS` snippet from other formulae
appears to resolve the error here as well.

Co-authored-by: Sean Molenaar <SMillerDev@users.noreply.github.com>
@samford samford force-pushed the otf2bdf-update-stable branch from 909c47f to 30800b1 Compare May 1, 2025 12:50
@github-actions github-actions bot removed the autosquash Automatically squash pull request commits according to Homebrew style. label May 1, 2025
@samford samford removed the CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. label May 1, 2025
Copy link
Contributor

github-actions bot commented May 1, 2025

🤖 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.

@github-actions github-actions bot added the CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. label May 1, 2025
@BrewTestBot BrewTestBot enabled auto-merge May 1, 2025 13:57
@BrewTestBot BrewTestBot added this pull request to the merge queue May 1, 2025
Merged via the queue into master with commit fd303f4 May 1, 2025
17 checks passed
@BrewTestBot BrewTestBot deleted the otf2bdf-update-stable branch May 1, 2025 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. livecheck Issues or PRs related to livecheck
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants