Skip to content

Main - #24

Merged
tgiap04 merged 2 commits into
developfrom
main
Aug 28, 2026
Merged

Main#24
tgiap04 merged 2 commits into
developfrom
main

Conversation

@tgiap04

@tgiap04 tgiap04 commented Aug 28, 2026

Copy link
Copy Markdown
Owner

Self-Review Checklist:

  • I've reviewed my own diff for quality, security, and reliability
  • Unsafe blocks (if any) have justifying comments
  • The content is consistent with the UI/UX checklist
  • Tests cover the new/changed behavior
  • Performance impact has been considered and is acceptable

Closes #ISSUE

Release Notes:

  • N/A or Added/Fixed/Improved ...

tgiap04 and others added 2 commits August 28, 2026 22:14
`script/bundle-linux` dies at packaging time with "detected dubious
ownership in repository at '/__w/zode/zode'", after the whole release
build has been paid for. Both Linux bundles for v0.1.2 failed there.

`actions/checkout` does add the workspace as a safe directory, but it
writes that under a `HOME` it overrides for the duration of the action
and then discards -- the log says so plainly:

    Temporarily overriding HOME='/__w/_temp/7a763e20-...' before making
    global git config changes
    git config --global --add safe.directory /__w/zode/zode

So nothing survives into the later steps. The workspace is owned by the
runner's uid on the host while the container runs as root, and the first
git call outside checkout is refused.

Only container jobs are affected, which is why this appeared with the
move to ubuntu:22.04 for the glibc 2.35 floor and not before: on a hosted
runner the job runs as the uid that owns the checkout.

`--system` rather than `--global` because /etc/gitconfig is read whatever
HOME happens to be, and HOME is not stable here -- rustup warns in this
same job that it differs from the euid-obtained one.

Verified in ubuntu:22.04 with the checkout chowned to uid 1001 and the
shell as root: `git rev-parse HEAD` reproduces the CI error verbatim
before the config line and succeeds after it.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@tgiap04
tgiap04 merged commit f0b562c into develop Aug 28, 2026
36 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