-
Notifications
You must be signed in to change notification settings - Fork 92
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
Added instructions on how to use argh with tipi.build #70
base: master
Are you sure you want to change the base?
Conversation
Hi @pysco68 - any updated on this? |
Hi @adishavit I didn't get any notification about comments on this PR nor could I find any. Would you mind checking if they have been posted correctly or point me a whereever they are? Best, |
See this thread on GitHub. |
} | ||
``` | ||
|
||
> `argh` itself is a header-only library hence the exclusion of all `.cpp` files above. This isn't stricly required to inlcude `argh` but speeds up the build. |
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.
Why do you say it "speeds up the build."?
There are no .cpp
files - it is a header only library.
The .cpp
files are only for testing and the example files. You do not need them for using argh.
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.
In tipi we use a source scan based approach to dependency management, instead of having you provide some type of script to make argh
usable in someone else's project. Basically when someone adds that line to his .tipi/deps
files, tipi will pull argh
straight from Github and install it in a project specific sysroot.
By excluding the *.cpp files that are not actually part of argh
(but rather only test cases and examples) these files are not being scanned or compiled which can save a bit of time.
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.
I don't completely understand.
So is adding that json snippet with the *.cpp
needed or not?
Is there some way or a need to exclude stuff?
The GH UI confused me - check now.. |
No worries ;) The code review UI is not stellar IMHO. Confuses the heck out of anyone I guess, plus the code hiding in the diffs makes some changes very hard to review. |
Please add your README example usage under the new, collapsable, Additional Build Systems section. |
Hey! I just stumbled upon
argh
during a demo with a customer and used it to demo our tool (they are using it). As depending on it with tipi just worked out of the box I though you may be happy to add the instructions about how to useargh
in tipi.build projects to the readme.I hope this is okay with you. If there's something broken we'd love to help and fix.
P.S.: Full disclaimer: I'm one of the co-founders of tipi.build