Skip to content

Commit

Permalink
Merge pull request #248 from nobe4/fix-missing-requirement
Browse files Browse the repository at this point in the history
fix: update install suggestion for missing `fyne` command
  • Loading branch information
andydotxyz committed May 7, 2024
2 parents 5448401 + 26515d9 commit 187fb53
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 7 deletions.
8 changes: 2 additions & 6 deletions .github/workflows/ci.yml
Expand Up @@ -131,11 +131,9 @@ jobs:
working-directory: fyne-cross
run: go install

# attempt to use "go install" but fallback to "go get"
- name: Install Fyne
run: |
go install fyne.io/fyne/v2/cmd/fyne@latest ||
go get fyne.io/fyne/v2/cmd/fyne@latest
go install fyne.io/fyne/v2/cmd/fyne@latest
- name: Install Podman
if: ${{ runner.os == 'macos' }}
Expand Down Expand Up @@ -213,11 +211,9 @@ jobs:
working-directory: fyne-cross
run: go install

# attempt to use "go install" but fallback to "go get"
- name: Install Fyne
run: |
go install fyne.io/fyne/v2/cmd/fyne@latest ||
go get fyne.io/fyne/v2/cmd/fyne@latest
go install fyne.io/fyne/v2/cmd/fyne@latest
- name: Install Podman
if: ${{ runner.os == 'macos' }}
Expand Down
2 changes: 1 addition & 1 deletion internal/command/command.go
Expand Up @@ -157,7 +157,7 @@ func printUsage(template string, data interface{}) {
func checkFyneBinHost(ctx Context) (string, error) {
fyne, err := execabs.LookPath("fyne")
if err != nil {
return "", fmt.Errorf("missed requirement: fyne. To install: `go get fyne.io/fyne/v2/cmd/fyne` and add $GOPATH/bin to $PATH")
return "", fmt.Errorf("missed requirement: fyne. To install: `go install fyne.io/fyne/v2/cmd/fyne@latest` and add $GOPATH/bin to $PATH")
}

if debugging() {
Expand Down

0 comments on commit 187fb53

Please sign in to comment.