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.

Sign up for GitHub

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

Jump to bottom

wezterm: Update to sha 30345b36d8a00fed347e4df5dadd83915a7693fb #26026

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
+1 −1
Open

wezterm: Update to sha 30345b36d8a00fed347e4df5dadd83915a7693fb #26026

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion aqua/wezterm/Portfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ PortSystem 1.0
PortGroup cargo 1.0
PortGroup github 1.0

github.setup wez wezterm 20240203-110809-5046fc22
github.setup wez wezterm 30345b36d8a00fed347e4df5dadd83915a7693fb
Copy link
Member

@herbygillot herbygillot Oct 3, 2024 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The only issue here is that the SHA version doesn't follow the same format as the version there currently.
So maybe we can do this, but explicitly set a datestamped current version with the assumption that the next release will have a higher (later) datestamp.

So perhaps something like this:

github.setup        wez wezterm 30345b36d8a00fed347e4df5dadd83915a7693fb
version             20240813-000000-30345b36d8

20240813 is the datestamp of that commit.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not a bad plan. We should decide if this is the SHA we want, too. I want to stress that this was just HEAD at the time I tested to see if it had been fixed upstream, and I've just run with it so far.

revision 0

homepage https://wezfurlong.org/wezterm
Expand Down