-
Notifications
You must be signed in to change notification settings - Fork 16
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
Upgrade TBB and ANTLR #1714
Upgrade TBB and ANTLR #1714
Conversation
Co-authored-by: Spitz <noella.spitz@ndw2544mac.isis.cclrc.ac.uk>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking very good indeed. I quite like it now that the Qt stuff (e.g. Q_SIGNALS, Q_EMIT) stand out from "standard" C++, rather than insidiously sneak in!
Couple of queries and suggestions for harmonising up the workflows.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Just one small question.
cp ./deploy/antlr4-cppruntime/lib/libantlr4-runtime.*.dylib /usr/local/lib | ||
cp ./deploy/antlr4-cppruntime/lib/libantlr4-runtime.dylib /usr/local/lib/libantlr4-runtime.dylib |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need both of these?
As a side effect, we lose weggli, but even I wasn't using it
Co-authored-by: Spitz <noella.spitz@ndw2544mac.isis.cclrc.ac.uk> Co-authored-by: Adam Washington <adam.washington@stfc.ac.uk>
Co-authored-by: Spitz <noella.spitz@ndw2544mac.isis.cclrc.ac.uk> Co-authored-by: Adam Washington <adam.washington@stfc.ac.uk>
Co-authored-by: Spitz <noella.spitz@ndw2544mac.isis.cclrc.ac.uk> Co-authored-by: Adam Washington <adam.washington@stfc.ac.uk>
Change from the depreciated TBB (2020.3) to the new oneAPI version (2021.10.0). as well as ANTLR from version 4.9.3 to 4.13.1. Would allow for Dissolve to work on Apple Sillicon.
Note:
Due to naming conflicts
QT_NO_KEYWORDS
was set meaning we can't use the QT generated MOC keywords anymore such asslots
oremit
. These now have to be replaced withQ_SLOTS
andQ_EMIT
, respectively.Closes #1712