-
Notifications
You must be signed in to change notification settings - Fork 463
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
Some improvements #677
Some improvements #677
Conversation
Now, they build the same executables.
Fixes 24765c3 Signed-off-by: Steffen Jaeckel <[email protected]>
Signed-off-by: Steffen Jaeckel <[email protected]>
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 looked through all commits separately. Cool.
That has got to be the understatement of the month! 😆 |
Something is broken with the GH actions... but that doesn't seem to be particularly relevant for this PR |
With this, I think #675 can be closed without merging it. |
Signed-off-by: Steffen Jaeckel <[email protected]>
This reverts 85dc394 Signed-off-by: Steffen Jaeckel <[email protected]>
Signed-off-by: Steffen Jaeckel <[email protected]>
43d86dd
to
842b8b0
Compare
* add `ltc` wrapper script for installed demos * rework demos/CMakeLists.txt and add some more bells and whistles * prefix installed demos with `ltc-` Currently this makes the standard makefiles and CMake results diverge, but we can fix that later if required. Signed-off-by: Steffen Jaeckel <[email protected]>
842b8b0
to
88dcebd
Compare
yeah no, I broke them ... but I've also fixed them again :)
😄 OK, I've updated the comment (...a bit ;) ) |
Check commit messages for details.
This fixes #671
This fixes #675