undefined: revel.Walk on go get about cmd

[xoen]

This could be caused by this: #20 - Looks like Walk is on last version of revel but not on the old one we're using (v0.12.0-1-g83c6204 / 83c62046c726f36daec7a28a335521f94b2561a6 according to Godep).

[xoen]

I'm trying to bump the github.com/revel/revel package (which is a bit scary). Tests seem to pass locally, hopefully it will not break anything.

Is this config change the only required change on the code? Rename the watcher = true to watch = true in the app.config file?

[jeevatkm]

@xoen - You may know this, however I will describe it.

revel.Walk is introduced in v0.13.0 release, if you have correct version of revel/revel & cmd/revel you will not have any issues.

Let's say you already have github.com/revel/revel and github.com/revel/cmd/revel on GOPATH then to get an updated version, please use go get with -u to get updated version.

go get -u github.com/revel/revel
go get -u github.com/revel/cmd/revel

Next, yes only breaking change is config name as described in the release notes.

Next, every revel/revel release will have corresponding release on cmd/revel repo.

Can you please describe about scary part you mentioned above?

[xoen]

Hi @jeevatkm yep, revel/cmd is not part of our Godeps and we go get before running the tests. Unfortunately there is no way to go get a specific version therefore the issue (as I can't lock both packages).

I updated revel and so far we didn't see any problems (on staging).

[xoen]

BTW, I think this can be closed as upgrade revel/revel solves the problem.

[jeevatkm]

Thanks for your update, good to hear that. Just a thought may be you can try cloning specific version of revel/cmd from github tags corresponding to revel/revel as part of CI.