Skip to content

Fixed small bug with license file.#251

Merged
boegel merged 3 commits intoeasybuilders:developfrom
wpoely86:intelbase-lic
Oct 1, 2013
Merged

Fixed small bug with license file.#251
boegel merged 3 commits intoeasybuilders:developfrom
wpoely86:intelbase-lic

Conversation

@wpoely86
Copy link
Copy Markdown
Member

$INTEL_LICENSE_FILE can be a list of directory which contain the license
file. For the install cfg, PSET_LICENSE_FILE must point to the license
file itself, not a directory. This patch iterates throught the
directories in $INTEL_LICENSE_FILE until a .lic file is found.

$INTEL_LICENSE_FILE can be a list of directory which contain the license
file. For the install cfg, PSET_LICENSE_FILE must point to the license
file itself, not a directory. This patch iterates throught the
directories in $INTEL_LICENSE_FILE until a .lic file is found.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use os.path.sep instead of hardcoding :

@boegel
Copy link
Copy Markdown
Member

boegel commented Sep 28, 2013

@wpoely86: Very nice, just a couple of small (style) remarks. Once those are excepted, this should be good to go, thanks!

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 1, 2013

Thanks for resolving the remarks @wpoely86!

(psst: next time, add a comment after you're done, we don't get mails for added commits)

boegel added a commit that referenced this pull request Oct 1, 2013
Fixed small bug with license file.
@boegel boegel merged commit dc1b9c7 into easybuilders:develop Oct 1, 2013
@wpoely86 wpoely86 deleted the intelbase-lic branch October 1, 2013 12:07
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