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

alacritty: Fix nerdfont variant #27758

Merged
merged 1 commit into from
Feb 26, 2025

Conversation

Koston-0xDEADBEEF
Copy link
Contributor

Description

Alacritty variant +nerdfont patches alacritty dependency crossfont, which had its version number bumped. Reflect the change in patch to fix the variant.

Type(s)
  • bugfix

@macportsbot
Copy link

Notifying maintainers:
@akierig for port alacritty.
@herbygillot for port alacritty.

@macportsbot macportsbot added type: bugfix maintainer: open Affects an openmaintainer port labels Feb 24, 2025
@akierig
Copy link
Contributor

akierig commented Feb 25, 2025

LGTM, thanks for fixing this @Koston-0xDEADBEEF.

Would you mind adding a comment in the Portfile to check the crossfont patch vs. crossfont version so I don't forget next time we update it?

@Koston-0xDEADBEEF
Copy link
Contributor Author

@akierig Sure thing, comment added. Crossfont codebase hasn't really changed in years so the patch is unlikely to break - except when a minor fix bumps the version number.

@akierig
Copy link
Contributor

akierig commented Feb 25, 2025

Appreciate it a lot! thanks! LGTM.

@akierig
Copy link
Contributor

akierig commented Feb 26, 2025

oh yeah you will need to bump the revision in the port file from 0 to 1

@Koston-0xDEADBEEF
Copy link
Contributor Author

Good catch, thanks!

@herbygillot herbygillot merged commit 432e9e8 into macports:master Feb 26, 2025
3 checks passed
@herbygillot
Copy link
Member

Thanks @Koston-0xDEADBEEF

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintainer: open Affects an openmaintainer port type: bugfix
Development

Successfully merging this pull request may close these issues.

4 participants