Skip to content

Fix: mark build-macos.sh executable (unblocks release DMG job) - #3

Merged
YodasMyDad merged 2 commits into
masterfrom
fix-macos-exec-bit
Apr 18, 2026
Merged

Fix: mark build-macos.sh executable (unblocks release DMG job)#3
YodasMyDad merged 2 commits into
masterfrom
fix-macos-exec-bit

Conversation

@YodasMyDad

Copy link
Copy Markdown
Owner

Summary

  • The release workflow's new Build ContextMenu DMG (macos) job failed with Permission denied on ./build-macos.sh (run 24605081151).
  • Root cause: the script was committed from Windows bash where chmod +x does not propagate into git's mode bits, so it stayed at 100644.
  • Fix: git update-index --chmod=+x build-macos.sh — now 100755, executable on macOS / Linux runners.

Test plan

  • git ls-files --stage build-macos.sh100755 locally.
  • Merging this PR triggers the release workflow → macos job executes the script → v3.5.0 release gets the DMG attached alongside the Windows installers.

Impact on v3.4.0 release

v3.4.0 did publish the .nupkg and the Windows .exe installers — the publish job only depends on all three builds succeeding, and since build-dmg failed, the publish job was skipped. So:

  • v3.4.0 on nuget.org / GitHub Release: missing (was blocked by the same failure — if it's there, something else is going on).
  • The next auto-release will start from 3.4.0 and bump to 3.5.0.

🤖 Generated with Claude Code

YodasMyDad and others added 2 commits April 18, 2026 13:58
The macOS release job failed with "Permission denied" on ./build-macos.sh
because the file was committed from Windows bash, where chmod +x does
not propagate into git's mode bits (stays at 100644). Setting 100755 so
macos-latest runners can execute it directly.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
ImageResize.ContextMenu/nuget.config had a Local source pointing at
../nupkgs. ContextMenu references Core via ProjectReference (never
PackageReference), so the Local feed isn't used by the restore — but
NuGet still errors with NU1301 at restore time when the directory is
missing, which is always the case on a fresh CI checkout. Was breaking
both the build.yml ContextMenu (windows + macos) jobs.

Also drops the now-redundant "Restore nupkgs" step in release.yml's
Windows installer job — it was only there to paper over the same
NU1301. The nupkgs artifact is still uploaded by build-library and
downloaded by publish for nuget.org push + GitHub Release asset.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@YodasMyDad
YodasMyDad merged commit 4bdf3ab into master Apr 18, 2026
4 checks passed
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