Release structure is inconsistent with other golang based projects in Cloud Foundry

cloudfoundry-attic / bosh-softlayer-cpi-release-DEPRECATED-TO_BE_DELETED

A BOSH release for the bosh-softlayer-cpi written in Golang and using the new BOSH external CPI release fashion.

Apache License 2.0

3 stars 8 forks source link

Release structure is inconsistent with other golang based projects in Cloud Foundry #22

Closed sykesm closed 8 years ago

sykesm commented 8 years ago

The release tree includes an .envrc file in the root of the release that explicitly sets the GOPATH to the base directory of the release. Unfortunately, the only code in the release is in a submodule path that does not include the necessary github.com/cloudfoundry parent directories. This breaks tooling that understands the GOPATH.

If the repository is supposed to be treated as a GOPATH, you need to put the submodule in the correct path.

jianqiu commented 8 years ago

@sykesm Thanks for your insight, we are considering to refine the release structure.

maximilien commented 8 years ago

@sykesm note that this repo will be deleted as we migrate to one repo for the code and release. This will be done in v2 timeframe.

After we merge, please verify we have addressed your concern.

/cc @jianqiu