feat(update): add updateable build tag for update notifications
This commit is contained in:
parent
3d27e61aa9
commit
22504bfa96
4 changed files with 25 additions and 9 deletions
|
|
@ -29,14 +29,6 @@ import (
|
|||
"github.com/spf13/cobra"
|
||||
)
|
||||
|
||||
// updaterEnabled is a statically linked build property set in gh formula within homebrew/homebrew-core
|
||||
// used to control whether users are notified of newer GitHub CLI releases.
|
||||
// This needs to be set to 'cli/cli' as it affects where update.CheckForUpdate() checks for releases.
|
||||
// It is unclear whether this means that only homebrew builds will check for updates or not.
|
||||
// Development builds leave this empty as impossible to determine if newer or not.
|
||||
// For more information, <https://github.com/Homebrew/homebrew-core/blob/master/Formula/g/gh.rb>.
|
||||
var updaterEnabled = ""
|
||||
|
||||
type exitCode int
|
||||
|
||||
const (
|
||||
|
|
|
|||
11
internal/ghcmd/update_disabled.go
Normal file
11
internal/ghcmd/update_disabled.go
Normal file
|
|
@ -0,0 +1,11 @@
|
|||
//go:build !updateable
|
||||
|
||||
package ghcmd
|
||||
|
||||
// updaterEnabled is a statically linked build property set in gh formula within homebrew/homebrew-core
|
||||
// used to control whether users are notified of newer GitHub CLI releases.
|
||||
// This needs to be set to 'cli/cli' as it affects where update.CheckForUpdate() checks for releases.
|
||||
// It is unclear whether this means that only homebrew builds will check for updates or not.
|
||||
// Development builds leave this empty as impossible to determine if newer or not.
|
||||
// For more information, <https://github.com/Homebrew/homebrew-core/blob/master/Formula/g/gh.rb>.
|
||||
var updaterEnabled = ""
|
||||
11
internal/ghcmd/update_enabled.go
Normal file
11
internal/ghcmd/update_enabled.go
Normal file
|
|
@ -0,0 +1,11 @@
|
|||
//go:build updateable
|
||||
|
||||
package ghcmd
|
||||
|
||||
// updaterEnabled is a statically linked build property set in gh formula within homebrew/homebrew-core
|
||||
// used to control whether users are notified of newer GitHub CLI releases.
|
||||
// This needs to be set to 'cli/cli' as it affects where update.CheckForUpdate() checks for releases.
|
||||
// It is unclear whether this means that only homebrew builds will check for updates or not.
|
||||
// Development builds leave this empty as impossible to determine if newer or not.
|
||||
// For more information, <https://github.com/Homebrew/homebrew-core/blob/master/Formula/g/gh.rb>.
|
||||
var updaterEnabled = "cli/cli"
|
||||
|
|
@ -53,7 +53,9 @@ var tasks = map[string]func(string) error{
|
|||
ldflags = fmt.Sprintf("-X github.com/cli/cli/v2/internal/authflow.oauthClientID=%s %s", os.Getenv("GH_OAUTH_CLIENT_ID"), ldflags)
|
||||
}
|
||||
|
||||
return run("go", "build", "-trimpath", "-ldflags", ldflags, "-o", exe, "./cmd/gh")
|
||||
buildTags, _ := os.LookupEnv("GO_BUILDTAGS")
|
||||
|
||||
return run("go", "build", "-trimpath", "-tags", buildTags, "-ldflags", ldflags, "-o", exe, "./cmd/gh")
|
||||
},
|
||||
"manpages": func(_ string) error {
|
||||
return run("go", "run", "./cmd/gen-docs", "--man-page", "--doc-path", "./share/man/man1/")
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue