Skip to content

Show git commit hash in gala --version#2815

Open
lenemter wants to merge 2 commits intomainfrom
lenemter/show-git-hash-in-ver
Open

Show git commit hash in gala --version#2815
lenemter wants to merge 2 commits intomainfrom
lenemter/show-git-hash-in-ver

Conversation

@lenemter
Copy link
Copy Markdown
Member

@lenemter lenemter commented Apr 11, 2026

Should make it easier for early access users to get the accurate version information

@lenemter lenemter requested a review from a team April 11, 2026 15:55
@lenemter lenemter force-pushed the lenemter/show-git-hash-in-ver branch from c4904a4 to fce7ac8 Compare April 11, 2026 16:02
Comment thread meson.build Outdated
@lenemter lenemter force-pushed the lenemter/show-git-hash-in-ver branch 2 times, most recently from 64483d2 to 76d1f3e Compare April 12, 2026 08:27
@lenemter lenemter requested a review from ryonakano April 12, 2026 08:27
@lenemter lenemter force-pushed the lenemter/show-git-hash-in-ver branch 2 times, most recently from db02bad to 37342d6 Compare April 12, 2026 08:36
@lenemter lenemter force-pushed the lenemter/show-git-hash-in-ver branch from 37342d6 to bc7246a Compare April 12, 2026 08:37
env:
DESTDIR: out
run: |
git config --global --add safe.directory "$(pwd)"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
git config --global --add safe.directory "$(pwd)"
git config --global --add safe.directory "$GITHUB_WORKSPACE"

It seems like we use this environment variable in other places. Also this is (slightly) more efficient than a command substitution.

env:
DESTDIR: out
run: |
git config --global --add safe.directory "$(pwd)"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
git config --global --add safe.directory "$(pwd)"
git config --global --add safe.directory "$GITHUB_WORKSPACE"

Same here.

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.

3 participants