Skip to content
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

make lint fails on macos #4814

Closed
ssbarnea opened this issue Jan 8, 2020 · 5 comments · Fixed by #5011
Closed

make lint fails on macos #4814

ssbarnea opened this issue Jan 8, 2020 · 5 comments · Fixed by #5011
Assignees
Labels
locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments.

Comments

@ssbarnea
Copy link
Collaborator

ssbarnea commented Jan 8, 2020

$ make lint                                                                                                                                                                                                    Makefile:47: Podman is being compiled without the systemd build tag. Install libsystemd on Ubuntu or systemd-devel on rpm based distro for journald support
X11 forwarding request failed
X11 forwarding request failed
GO111MODULE=off go generate ./cmd/podman/varlink/...
../../../vendor/github.com/varlink/go/cmd/varlink-go-interface-generator/main.go:12:2: cannot find package "github.com/varlink/go/varlink/idl" in any of:
/usr/local/Cellar/go/1.13.4/libexec/src/github.com/varlink/go/varlink/idl (from $GOROOT)
/Users/ssbarnea/go/src/github.com/varlink/go/varlink/idl (from $GOPATH)
cmd/podman/varlink/generate.go:3: running "go": exit status 1
make: *** [cmd/podman/varlink/iopodman.go] Error 1
FAIL: 2
@ssbarnea ssbarnea self-assigned this Jan 8, 2020
@rhatdan
Copy link
Member

rhatdan commented Jan 8, 2020

@baude PTAL

@baude
Copy link
Member

baude commented Jan 26, 2020

We dont support native compilation on osx ... only cross-compile from linux. Varlink is one reason. @ssbarnea how do you want to deal with this/go from here.

@ssbarnea
Copy link
Collaborator Author

Linting must work from any platform likely to support contributions from, even if compilation would be cross-compiled.

I was planning to do something about it myself but got busy with other stuff. Keep it open until someone fixes it.

I would consider acceptable even if only pre-commit runs on macos and the rest is skipped. The more interesting aspect is that pre-commit works but make never reaches it because chokes at the dependency target it cannot build.

If any idea about how to make that target optional for macos it would be great.

PS. In fact make could start some containers and do the build with them for unsupported platforms, but this would require lots of changes.

If you tell me which patch to take, I can look into making a PR to address it.

@baude baude assigned vrothberg and unassigned ssbarnea and baude Jan 28, 2020
vrothberg added a commit to vrothberg/libpod that referenced this issue Jan 29, 2020
Running the `varlink_generate` make target on non-Linux machines is not
supported, so restrict it to Linux only.

Fixes: containers#4814
Signed-off-by: Valentin Rothberg <[email protected]>
@medyagh
Copy link

medyagh commented Feb 10, 2020

@vrothberg this issue seem to still exist !
on mac :

==> make podman-remote
Last 15 lines from /Users/medya/Library/Logs/Homebrew/podman/01.make:
make
podman-remote
uname: illegal option -- o
usage: uname [-amnprsv]
mkdir -p "/private/tmp/podman-20200210-97980-183905n/_output/src/github.com/containers"
ln -sfn "/private/tmp/podman-20200210-97980-183905n" "/private/tmp/podman-20200210-97980-183905n/_output/src/github.com/containers/libpod"
ln -sfn "/private/tmp/podman-20200210-97980-183905n/vendor/github.com/varlink" "/private/tmp/podman-20200210-97980-183905n/_output/src/github.com/varlink"
touch .gopathok
GO111MODULE=on go build -mod=vendor  -gcflags 'all=-trimpath=/private/tmp/podman-20200210-97980-183905n' -asmflags 'all=-trimpath=/private/tmp/podman-20200210-97980-183905n' -ldflags '-X github.com/containers/libpod/libpod/define.gitCommit=e57253d06841d7a128ef760f8c47acf4b59157df -X github.com/containers/libpod/libpod/define.buildInfo=1581368899 -X github.com/containers/libpod/libpod/config._installPrefix=/usr/local -X github.com/containers/libpod/libpod/config._etcDir=/etc -extldflags ""' -tags "containers_image_openpgp exclude_graphdriver_btrfs exclude_graphdriver_devicemapper exclude_graphdriver_overlay varlink remoteclient" -o bin/podman-remote github.com/containers/libpod/cmd/podman
# github.com/containers/libpod/cmd/podman/shared
cmd/podman/shared/intermediate_varlink.go:70:42: undefined: iopodman.Create
cmd/podman/shared/intermediate_varlink.go:71:7: undefined: iopodman.Create
cmd/podman/shared/intermediate_varlink.go:317:34: undefined: iopodman.Create
make: *** [bin/podman-remote] Error 2
READ THIS: https://docs.brew.sh/Troubleshooting

@vrothberg
Copy link
Member

-> #5162

Thanks for the ping!

mheon pushed a commit to mheon/libpod that referenced this issue Oct 14, 2020
From a fresh install of Fedora 33 Beta and a fresh clone of the repo, `make` fails with the following error when Go modules are disabled:

    # Only generate the varlink code on Linux (see issue containers#4814).
    GO111MODULE=off go generate ./pkg/varlink/...
    ../../vendor/github.com/varlink/go/cmd/varlink-go-interface-generator/main.go:12:2: cannot find package "github.com/varlink/go/varlink/idl" in any of:
    	/usr/lib/golang/src/github.com/varlink/go/varlink/idl (from $GOROOT)
    	/home/test/src/podman/_output/src/github.com/varlink/go/varlink/idl (from $GOPATH)
    pkg/varlink/generate.go:3: running "go": exit status 1
    make: *** [Makefile:646: pkg/varlink/iopodman.go] Error 1

Signed-off-by: Jordan Christiansen <[email protected]>
@github-actions github-actions bot added the locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. label Sep 23, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants