Skip to content
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

README.md : added ``export'' to the CPPUTEST_HOME env variable #9

Merged

Conversation

Pvag
Copy link
Contributor

@Pvag Pvag commented Apr 7, 2018

The -e flag of Docker requires that the environment variable specified be exported prior to command invocation, as stated at https://docs.docker.com/engine/reference/commandline/run/#set-environment-variables--e---env---env-file

The -e flag of Docker requires that the environment variable specified be exported prior to command invocation, as stated at https://docs.docker.com/engine/reference/commandline/run/#set-environment-variables--e---env---env-file
@Pvag
Copy link
Contributor Author

Pvag commented Apr 7, 2018

I was experiencing troubles before this modification, on Mac OS High Sierra, but now the compilation is fine, since the environment variable CPPUTEST_HOME is properly exported to Docker and then properly substituted inside the Makefiles.

@Pvag Pvag changed the title added export to the CPPUTEST_HOME env variable README.md : added export to the CPPUTEST_HOME env variable Apr 7, 2018
@Pvag Pvag changed the title README.md : added export to the CPPUTEST_HOME env variable README.md : added ``export'' to the CPPUTEST_HOME env variable Apr 7, 2018
@jwgrenning jwgrenning merged commit 46ec03e into jwgrenning:master Sep 9, 2019
@jwgrenning
Copy link
Owner

Hi Brac24, I merged the pull request three days ago. Did that help?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants