-
Notifications
You must be signed in to change notification settings - Fork 55
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
Adding a section on MakeFiles and compiling ROOT macros using g++ #194
Comments
We just talked about this on Slack! 😄 I agree it could be useful. It would probably be more at home in analysis essentials. The problem is that we don't teach any C++/ROOT, so the compiling part wouldn't be relevant for the rest of the lessons. But I think Makefiles in general are super helpful, like for defining analysis workflows. That could make a cool lesson. And I'm not adverse to having a more general resource, even if it's not taught in the Starterkit. |
Feel free to import from my blog ;) or on top of that, i would love to learn myself about auto dependency generation. |
I personally use CMake most of the time, but it adds another layer of indirection (But for any sort of real application, or for using IDEs, etc, it’s essential). CMake 3+ and my additions to FindROOT make it pretty easy to work with. I’ll try to write up an example in a few weeks and put it on my blog and put the FindROOT in a reasonable location (It’s part of GooFit currently). |
@pseyfert, from the last section of the paper
This is already done with |
@rmatev good to know. though i was mostly interested in just getting the |
thinking about it again, aren't auto dependency generation and cmake are out of scope … |
Having recently gone through the process of converting a jumbled mess of macros into a working application, a section on writing a Makefile could be useful.
A quick braindump of this could include:
root-config --cflags
etc.I'l happily start work on this when I have some time, but having only recently gone through this process a more experienced eye watching over would be useful.
The text was updated successfully, but these errors were encountered: