Skip to content

Commit 5cea5b4

Browse files
committed
fix(pytools): run diff-index in the asf-site-next worktree
The diff-index call was missing cwd=site, so it checked the main repo (always clean after checkout) instead of the asf-site-next worktree, causing commit+push to be skipped. The bug was masked until now by scripts/replace.js mutating the main tree in place; removing replace.js exposed it and stopped the site from updating. BUILD_ALL_VERSION=1
1 parent 7291fca commit 5cea5b4

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

tools/pytools/lib/execute/site_uploader.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ def _do_push(msg: str, site: Path, branch: str):
4848
git = find_command('git', msg="git is required")
4949

5050
run(git, 'add', '-A', '.', cwd=site)
51-
changed = run(git, 'diff-index', '--quiet', 'HEAD', codes={0, 1}).returncode
51+
changed = run(git, 'diff-index', '--quiet', 'HEAD', codes={0, 1}, cwd=site).returncode
5252
print(f'changed: {changed}')
5353

5454
run(git, 'status', cwd=site)

0 commit comments

Comments
 (0)