Skip to content

intiface-central 3.0.1 (new cask)#253949

Open
CockpitCutie wants to merge 2 commits intoHomebrew:mainfrom
CockpitCutie:main
Open

intiface-central 3.0.1 (new cask)#253949
CockpitCutie wants to merge 2 commits intoHomebrew:mainfrom
CockpitCutie:main

Conversation

@CockpitCutie
Copy link

Important: Do not tick a checkbox if you haven’t performed its action. Honesty is indispensable for a smooth review process.

In the following questions <cask> is the token of the cask you're submitting.

After making any changes to a cask, existing or new, verify:

Additionally, if adding a new cask:

  • Named the cask according to the token reference.
  • Checked the cask was not already refused (add your cask's name to the end of the search field).
  • brew audit --cask --new <cask> worked successfully.
  • HOMEBREW_NO_INSTALL_FROM_API=1 brew install --cask <cask> worked successfully.
  • brew uninstall --cask <cask> worked successfully.

If AI was used to generate or assist with generating the PR:

  • I used AI to generate or assist with generating this PR.
  • I have personally reviewed, tested and verified all changes/additions, including zap stanza paths.

I used claude to help with the livecheck strategy, since the versioning tags used by intiface make the default github latest not work properly, but reviewed and manually tested it and all other aspects of the cask.

I tested and verified the zap stanza is correct, one thing I noticed is homebrew cannot delete the folder ~/Library/Containers/com.nonpolynomial.intifacecentral due to SIP, but I wrote the zap consistent to other casks I looked at with similar file structures, like crystalfetch.

Copy link
Member

@samford samford left a comment

Choose a reason for hiding this comment

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

Upstream has used a tag format like v2.6.8 in the not-too-distant past (2025-09-06), so let's treat the numeric suffix (e.g., +37) as optional in the livecheck block regex and cask url.

There's also a newer version (v3.0.3+39) as of today, so it would be good to update to that.

Comment on lines +12 to +17
regex(/^v?(\d+(?:\.\d+)+)\+(\d+)$/i)
strategy :github_latest do |json, regex|
match = json["tag_name"]&.match(regex)
next if match.blank?

"#{match[1]},#{match[2]}"
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
regex(/^v?(\d+(?:\.\d+)+)\+(\d+)$/i)
strategy :github_latest do |json, regex|
match = json["tag_name"]&.match(regex)
next if match.blank?
"#{match[1]},#{match[2]}"
regex(/^v?(\d+(?:\.\d+)+)(?:\+(\d+))?$/i)
strategy :github_latest do |json, regex|
match = json["tag_name"]&.match(regex)
next if match.blank?
match[2].present? ? "#{match[1]},#{match[2]}" : match[1]

version "3.0.1,37"
sha256 "4dca9f17f8d082564e4480b8eb15e68acfeb593be8f000690184d8d585e5c2c9"

url "https://github.com/intiface/intiface-central/releases/download/v#{version.csv.first}%2B#{version.csv.second}/intiface-central-v#{version.csv.first}-macos-universal.dmg"
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
url "https://github.com/intiface/intiface-central/releases/download/v#{version.csv.first}%2B#{version.csv.second}/intiface-central-v#{version.csv.first}-macos-universal.dmg"
url "https://github.com/intiface/intiface-central/releases/download/v#{version.csv.first}#{"%2B#{version.csv.second}" if version.csv.second}/intiface-central-v#{version.csv.first}-macos-universal.dmg"

Comment on lines +2 to +3
version "3.0.1,37"
sha256 "4dca9f17f8d082564e4480b8eb15e68acfeb593be8f000690184d8d585e5c2c9"
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
version "3.0.1,37"
sha256 "4dca9f17f8d082564e4480b8eb15e68acfeb593be8f000690184d8d585e5c2c9"
version "3.0.3,39"
sha256 "c3605043d8a1bbb30a2ce5873e07b5d0d34395f2ad09cc1b82d29a2932ff357e"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants