-
Notifications
You must be signed in to change notification settings - Fork 36
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CLI $GOPATH not resolved properly #107
Comments
Hey,
And it worked just fine. |
Hi there, Here is a log of my results:
Does Flogo not use the system default $GOPATH? seems to indicate that this is fine and should work. |
I'm no golang pro, but it seems something like this might be in the way:
and finds my |
I looked into it. As a matter of fact we did took that thing into consideration. The only issue |
I guess the workaround here is to explicitly set |
Yes. |
When is the release? |
This problem still exists - setting MacOS / zsh / golang installed with
export GOROOT="$(brew --prefix golang)/libexec"
export GOPATH="${HOME}/go"
export PATH="$PATH:$GOPATH/bin" |
Current behavior (how does the issue manifest):
Flogo fails to create any and all projects with Centos8 (other OS not yet checked)
Expected behavior:
Not that. Anything but that.
Minimal steps to reproduce the problem (not required if feature enhancement):
(taken from my initial gitter report)
...profit?
What is the motivation / use case for changing the behavior?
To make the core product function.
Please tell us about your environment (Operating system, docker version, browser & verison if webui, etc):
As described in the centos8 default docker image.
Flogo version (CLI & contrib/lib. If unknown, leave empty or state unknown): 0.X.X
All?
Additional information you deem important (e.g. issue happens only occasionally):
This issue is a path resolving problem. Flogo clearly has no problem pulling the modules and installing them in
$GOPATH/pkg
, but when it tries to load them, it appears to load from$GOPATHpkg
without a '/'. Flogo should not make whatever assumption it seems to be making at this step.The text was updated successfully, but these errors were encountered: