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

Additional environment helper scripts #168

Merged
merged 4 commits into from Jun 21, 2017

Conversation

Projects
None yet
2 participants
@decibelcooper
Member

decibelcooper commented Jun 16, 2017

This request resolves #162 in the manner suggested in the thread.

BEGINRELEASENOTES

  • Added environment helper scripts thisdd4hep_only.(c)sh that only set up variables for DD4hep and not for dependencies.

ENDRELEASENOTES

@andresailer

This comment has been minimized.

Member

andresailer commented Jun 16, 2017

You also have to configure the scripts like here:
https://github.com/AIDASoft/DD4hep/blob/master/CMakeLists.txt#L151
And install them like a few lines below that line

@decibelcooper

This comment has been minimized.

Member

decibelcooper commented Jun 16, 2017

@andresailer My apologies for the haste. This has been updated and tested.

@andresailer

This comment has been minimized.

Member

andresailer commented Jun 16, 2017

No worries, thanks for your contribution

dd4hep_parse_this() {
package=${2};
if [ "x${1}" = "x" ]; then
if [ ! -f bin/this${package}.sh ]; then

This comment has been minimized.

@andresailer

andresailer Jun 19, 2017

Member

This should be this${package}_only.sh

package=${2};
if [ "x${1}" = "x" ]; then
if [ ! -f bin/this${package}.sh ]; then
echo ERROR: must "cd where/${package}/is" before calling ". bin/this${package}.sh" for this version of bash!;

This comment has been minimized.

@andresailer

andresailer Jun 19, 2017

Member

This should be this${package}_only.sh

@andresailer andresailer merged commit b442d3f into AIDASoft:master Jun 21, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment