search

blablacar / dgr

Container build and runtime tool
Apache License 2.0
249 stars 21 forks source link

bin-dgr/common/acfullname.go:36: not enough arguments in call to discovery.DiscoverACIEndpoints #201

Closed arthurzenika closed 7 years ago

arthurzenika commented 8 years ago

sorry am a go newbie.

had some problems with go getting the dependencies (ended up getting some "manually"). Now stuck on the following : 

$ ./build.sh 
+ date +%s
+ start=1476089404
+ dirname ./build.sh
+ dir=.
+ [ -f /home/arthur/go/bin/godep ]
+ [ -f /home/arthur/go/bin/go-bindata ]
+ go get github.com/jteeuwen/go-bindata
+ [ -f /usr/bin/upx ]
+ rm -Rf ./dist/*-amd64
+ mkdir -p ./dist
+ godep save ././aci-builder/... ././bin-templater/... ././bin-dgr/...
godep: Package (golang.org/x/sys/unix) not found
+ true
+ gofmt -w -s ./bin-dgr ./bin-templater ./aci-builder/bin-run
+ mkdir -p ./dist/bindata/aci/blablacar.github.io/dgr
+ [ -f ./dist/templater ]
+ [ -f ./dist/bindata/aci-tester.aci ]
+ [ -f ./dist/bindata/aci-builder.aci ]
+ ./aci-builder/build.sh
+ set -e
++ dirname ./aci-builder/build.sh
+ dir=./aci-builder
+ dist=./aci-builder/../dist
+ target=./aci-builder/../dist/aci-builder
+ rootfs=./aci-builder/../dist/aci-builder/rootfs
+ echo -e '\033[0;32mBuilding aci-builder\033[0m\n'
Building aci-builder

+ ./aci-builder/clean.sh
+ set -e
++ dirname ./aci-builder/clean.sh
+ dir=./aci-builder
+ target=./aci-builder/../dist/aci-builder
+ rm -Rf ./aci-builder/../dist/aci-builder
+ rm -f ./aci-builder/../dist/bindata/aci-builder.aci
+ mkdir -p ./aci-builder/../dist/aci-builder/rootfs/dgr ./aci-builder/../dist/aci-builder/rootfs/usr/bin
+ '[' -f ./aci-builder/../dist/templater ']'
+ GOOS=linux
+ GOARCH=amd64
+ godep go build --ldflags '-s -w -extldflags "-static"' -o ./aci-builder/../dist/aci-builder/rootfs/dgr/builder/stage1/run ./aci-builder/bin-run
# github.com/blablacar/dgr/bin-dgr/common
bin-dgr/common/acfullname.go:36: not enough arguments in call to discovery.DiscoverACIEndpoints
godep: go exit status 2

ps. any reason in the clean.sh to have sudo rm instead of rm ?

n0rad commented 8 years ago

It's probably because you have old version of github.com/appc/spec in your go path. Build is using your gopath libs and then the vendor directory.

With an empty gopath it's working.

dgr's build is building some aci embedded in the program. it requires root to build it with good permissions and so the cleanup.

The build is not perfect, I'm still thinking about a cleaner solution #140 & #141

n0rad commented 7 years ago

please reopen if you still have issues with new build system