-
Notifications
You must be signed in to change notification settings - Fork 801
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
No version info in homebrew package #3102
Comments
How about using I will file a PR if it looks good to you. 😃 |
I think this would result in us displaying the version of the |
I noticed that the lighthouse/scripts/change_version.sh Line 34 in 46a0606
the lighthouse/scripts/change_version.sh Line 17 in 46a0606
Therefore we don't need to use pub const VERSION: &str = git_version!(
args = [
"--always",
"--dirty=+",
"--abbrev=7",
// NOTE: using --match instead of --exclude for compatibility with old Git
"--match=thiswillnevermatchlol"
],
prefix = "Lighthouse/v2.2.1-",
- fallback = "unknown"
+ fallback = "Lighthouse/v2.2.1" // Note: unlike `prefix`, the trailing hyphen was striped.
); |
The homebrew package builds without the full git sources, which leads to its version showing as
Lighthouse unknown
. We should work out a way to inject the version from homebrew, or switch homebrew to use git (I suspect they won't want to do this due to the increased size)The text was updated successfully, but these errors were encountered: