chainkit icon indicating copy to clipboard operation
chainkit copied to clipboard

Following readme leads to a build error & some questions

Open zramsay opened this issue 6 years ago • 3 comments

to reproduce:

go get github.com/blocklayerhq/chainkit
cd $GOPATH/src/github.com/blocklayerhq/chainkit
make
cp chainkit /usr/local/bin
cd ~/go/src/github.com
chainkit create demoapp

is mostly successful in that the directory is created with some files in it, but the last line ... screen shot 2019-01-08 at 14 42 26

zramsay avatar Jan 08 '19 19:01 zramsay

Two questions:

  • why have the step cp chainkit /usr/local/bin which would need to be run again with every new build, rather than have make do a go install here ? That'd be one less line to run
  • a bit awkward to have the demoapp directory created alongside the usernames of repos (a la go get). I don't know a better place to put it, but maybe instructing users to create a repo makes sense. It's what we do with the sdk tutorial. That approach is a bit clunky IMO. Or, have them cd ~ before running chainkit create demoapp

zramsay avatar Jan 08 '19 19:01 zramsay

build error was b/c i didn't have docker running so that can be ignored, see #8.

zramsay avatar Jan 08 '19 20:01 zramsay

I created a makefile rule to install in one command #16.

However I prefer to keep the demoapp directory as a repo (like go get), I think it's more intuitive.

samalba avatar Jan 19 '19 23:01 samalba