Skip to content

{vis}[intel/2016b] QScintilla2 v2.10#4306

Merged
boegel merged 2 commits intoeasybuilders:developfrom
boegel:20170309143934_new_pr_QScintilla2210
Mar 9, 2017
Merged

{vis}[intel/2016b] QScintilla2 v2.10#4306
boegel merged 2 commits intoeasybuilders:developfrom
boegel:20170309143934_new_pr_QScintilla2210

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Mar 9, 2017 •
edited
Loading

(created using eb --new-pr)

edit: now depends on easybuilders/easybuild-easyblocks#1127

@boegel boegel mentioned this pull request Mar 9, 2017
Merged
@boegel
Copy link
Copy Markdown
Member Author

boegel commented Mar 9, 2017

Test report by @boegel
FAILED
Build succeeded for 0 out of 1 (1 easyconfigs in this PR)
node2028.delcatty.os - Linux centos linux 7.3.1611, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/cb3f83c4e828eb47e13250f8ad4c98b0 for a full test report.

boegel
boegel commented Mar 9, 2017
View reviewed changes
Comment thread easybuild/easyconfigs/q/QScintilla2/QScintilla2-2.10-intel-2016b-Python-2.7.12.eb Outdated
python_bindings_install_cmd += "make && make install && cd %(installdir)s/lib/python%(pyshortver)s/site-packages/PyQt4 && "
python_bindings_install_cmd += "for x in $(ls $EBROOTPYQT/lib/python2.7/site-packages/PyQt4); do ln -s $EBROOTPYQT/lib/python2.7/site-packages/PyQt4/$x; done && "
python_bindings_install_cmd += "rm __init__.py* && touch __init__.py"
postinstallcmds = [python_bindings_install_cmd]
Copy link
Copy Markdown
Member Author

@boegel boegel Mar 9, 2017

Choose a reason for hiding this comment

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

this is way too messy for an easyconfig, but at least I got it to work...

I'll implement a proper QScintilla easyblock...

Copy link
Copy Markdown
Member Author

@boegel boegel Mar 9, 2017

Choose a reason for hiding this comment

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

see easybuilders/easybuild-easyblocks#1127

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Mar 9, 2017

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
node2635.swalot.os - Linux centos linux 7.3.1611, Intel(R) Xeon(R) CPU E5-2660 v3 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/e6c775d892ae751f2a1acaab163a855f for a full test report.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Mar 9, 2017

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
node2028.delcatty.os - Linux centos linux 7.3.1611, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/7d3ba0440eea6ab549e6d103b67c0588 for a full test report.

@boegel boegel modified the milestone: 3.1.2 Mar 9, 2017
@boegel
Copy link
Copy Markdown
Member Author

boegel commented Mar 9, 2017

@wpoely86 please review?

@boegel boegel mentioned this pull request Mar 9, 2017
Merged
@boegel
Copy link
Copy Markdown
Member Author

boegel commented Mar 9, 2017

Thanks for the review @wpoely86!

@boegel boegel merged commit 688d14e into easybuilders:develop Mar 9, 2017
@boegel boegel deleted the 20170309143934_new_pr_QScintilla2210 branch March 9, 2017 21:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wpoely86 wpoely86 wpoely86 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

3.1.2

Development

Successfully merging this pull request may close these issues.

2 participants

@boegel @wpoely86