testcontainers-go icon indicating copy to clipboard operation
testcontainers-go copied to clipboard

golang.org/x/sys/windows".SECURITY_DESCRIPTOR

Open Ya3Sh opened this issue 5 years ago • 6 comments

Describe the bug Project fails to build on windows when I depend on testcontainers-go v0.2.0 with below error: ........\pkg\mod\github.com\docker\[email protected]\pkg\system\filesys_windows.go:111:24: cannot use uintptr(unsafe.Pointer(&sd[0])) (type uintptr) as type *"golang.org/x/sys/windows".SECURITY_DESCRIPTOR in assignment

To Reproduce Can you write a unit test to reproduce your issue?

Expected behavior A clear and concise description of what you expected to happen.

Additional context Tried replacing the docker/docker dependency with different versions. No luck.

Ya3Sh avatar Feb 13 '20 16:02 Ya3Sh

Possibly dupe of https://github.com/testcontainers/testcontainers-go/issues/143

Ya3Sh avatar Feb 13 '20 16:02 Ya3Sh

#143 resolution didn't work for me.

Ya3Sh avatar Feb 17 '20 13:02 Ya3Sh

Fixed for me by pulling the latest version of docker go get github.com/docker/docker@2200d938a2d5e7cd7437489c22a32d37d9bb380d This seems to stem from the testcontainers using an old version of docker.

ImVexed avatar Apr 16 '20 20:04 ImVexed

Also seeing this. Will try and work out how to use the PR

tomqwpl avatar Apr 17 '20 09:04 tomqwpl

Seeing this behaviour here: https://github.com/testcontainers/testcontainers-go/runs/1384925491?check_suite_focus=true

Currently reading https://github.com/golang/go/issues/34610

mdelapenya avatar Nov 11 '20 11:11 mdelapenya

I've sent a PR here #235 running the tests in multiple OS: windows and linux (unfortunately, GH does not support Docker on mac)

At this moment, the error described in this issue is fixed

mdelapenya avatar Nov 11 '20 12:11 mdelapenya

Since we do run go build in Linux, Windows and Mac, I'd say this issue is no longer reproducible at least since v0.10.0.

Could you please double check with the latest version of tc-go? If still seeing the issue, please reopen so we can dig into the root cause. Thanks in advance!!

mdelapenya avatar Sep 15 '22 11:09 mdelapenya