Skip to content

Previous icns file didn't seem to be working when building on MacOS. … - #1

Open
gm5dna wants to merge 2 commits into
g8bpq:masterfrom
gm5dna:master
Open

Previous icns file didn't seem to be working when building on MacOS. …#1
gm5dna wants to merge 2 commits into
g8bpq:masterfrom
gm5dna:master

Conversation

@gm5dna

@gm5dna gm5dna commented Feb 19, 2025

Copy link
Copy Markdown

…Here's a replacement.

Hi John,

I build this on MacOS and it's working perfectly, however the icon wasn't appearing. I've rebuild the icns file and it seems to be working fine now. Thought I'd share it in case it's helpful. No worries if not.

All the best,

Stuart
GM5DNA

gm5dna added a commit to gm5dna/homebrew-amateur-radio that referenced this pull request Apr 17, 2026
Repoints the formula from the gm5dna fork to the canonical
g8bpq/QtTermTCP repo and bumps the version to 0.81 to match the
VersionString in source. Carries g8bpq/QtTermTCP#1 inline (regenerated
icns + ICONS->ICON qmake fix) so the app icon actually renders; the PR
remains unmerged upstream. Install step also wipes the stale build
artefacts and prebuilt .app skeleton that upstream ships in the master
tarball, since the bundled .qmake.stash is pinned to an older SDK and
the prebuilt Info.plist has an empty CFBundleIconFile.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant