Skip to content

use --with-tcltk* configure options to point to ensure Tcl/Tk deps are picked up#729

Merged
boegel merged 1 commit intoeasybuilders:developfrom
boegel:python_tcltk
Oct 29, 2015
Merged

use --with-tcltk* configure options to point to ensure Tcl/Tk deps are picked up#729
boegel merged 1 commit intoeasybuilders:developfrom
boegel:python_tcltk

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Oct 29, 2015

fix for #728

@boegel boegel added this to the v2.4.0 milestone Oct 29, 2015
@hpcugentbot
Copy link
Copy Markdown

Easyblocks unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyblocks-pr-builder/1330/console for more details).

This pull request is now ready for review/testing.

Please try and find someone who can tackle this; contact @boegel if you're not sure what to do.

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.

maybe warning in the else?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

People are free to use the system Tcl/Tk (e.g. by suing --filter-deps), which is fine.

@wpoely86
Copy link
Copy Markdown
Member

lgtm

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Oct 29, 2015

Testing confirms that this fixes the (original) issue reported in #728.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Oct 29, 2015

Thanks for the review @wpoely86!

boegel added a commit that referenced this pull request Oct 29, 2015
use --with-tcltk* configure options to point to ensure Tcl/Tk deps are picked up
@boegel boegel merged commit 2662180 into easybuilders:develop Oct 29, 2015
@boegel boegel deleted the python_tcltk branch October 29, 2015 19:38
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.

3 participants