mth/ebiten
1
0
Fork 0
mirror of https://github.com/hajimehoshi/ebiten.git synced 2024-12-25 19:28:57 +01:00
Code Issues Packages Projects Releases Wiki Activity

cmd/ebitenmobile: Bug fix: .exe was requried on Windows

Browse Source 
On Windows, executable files must have .exe extension, or
exec.LookPath since exec.LookPath checks %PATHEXT%.

This fixes the issue that ebitenmobile didn't invoke the original
gobind wrapper and then didn't include and compile some Java files.

Fixes #1096
This commit is contained in:
Hajime Hoshi 2020-03-05 01:05:53 +09:00
parent ff9e0d699e
commit 0ae00aa138
1 changed files with 12 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
cmd/ebitenmobile/gomobile.go
View File

@ -59,6 +59,15 @@ func runGo(args ...string) error {
return runCommand(gocmd, args, env) return runCommand(gocmd, args, env)
} }
// exe adds the .exe extension to the given filename.
// Without .exe, the executable won't be found by exec.LookPath on Windows (#1096).
func exe(filename string) string {
if runtime.GOOS == "windows" {
return filename + ".exe"
}
return filename
}
func prepareGomobileCommands() error { func prepareGomobileCommands() error {
tmp, err := ioutil.TempDir("", "ebitenmobile-") tmp, err := ioutil.TempDir("", "ebitenmobile-")
if err != nil { if err != nil {
@ -98,10 +107,10 @@ func prepareGomobileCommands() error {
if err := runGo("get", "golang.org/x/mobile@"+gomobileHash); err != nil { if err := runGo("get", "golang.org/x/mobile@"+gomobileHash); err != nil {
return err return err
} }
if err := runGo("build", "-o", filepath.Join("bin", "gomobile"), "golang.org/x/mobile/cmd/gomobile"); err != nil { if err := runGo("build", "-o", exe(filepath.Join("bin", "gomobile")), "golang.org/x/mobile/cmd/gomobile"); err != nil {
return err return err
} }
if err := runGo("build", "-o", filepath.Join("bin", "gobind-original"), "golang.org/x/mobile/cmd/gobind"); err != nil { if err := runGo("build", "-o", exe(filepath.Join("bin", "gobind-original")), "golang.org/x/mobile/cmd/gobind"); err != nil {
return err return err
} }
@ -112,7 +121,7 @@ func prepareGomobileCommands() error {
return err return err
} }
if err := runGo("build", "-o", filepath.Join("bin", "gobind"), "-tags", "ebitenmobilegobind", filepath.Join("src", "gobind.go")); err != nil { if err := runGo("build", "-o", exe(filepath.Join("bin", "gobind")), "-tags", "ebitenmobilegobind", filepath.Join("src", "gobind.go")); err != nil {
return err return err
} }