Skip to content
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

Makefile: Keep going when building with make -k #582

Closed
wants to merge 1 commit into from
Closed

Makefile: Keep going when building with make -k #582

wants to merge 1 commit into from

Conversation

algernon
Copy link
Contributor

Instead of failing early on the first error, when building with make -k, keep going until the end, and only fail the build then.

@algernon
Copy link
Contributor Author

Still working on this one, there are some issues to iron out. The next push should make it behave properly...

Instead of failing early on the first error, when building with `make -k`, keep
going until the end, and only fail the build then.

Signed-off-by: Gergely Nagy <[email protected]>
@algernon
Copy link
Contributor Author

This last one should do. Worked locally. There may be build failures nevertheless.. at least I've seen some when building on my laptop, but I doubt those are the results of this change.

@ezuk
Copy link
Contributor

ezuk commented Jul 29, 2016

Yeah, CI bombs on this one for some reason, so not merging for now. You think the errors are not related to this branch?

@algernon
Copy link
Contributor Author

Not sure. I see [ERROR] messages in my local build, but none here, will have to investigate further, to see where the problem may be. Will try to do that during the weekend.

@algernon
Copy link
Contributor Author

I gave this another go, and came to the conclusion that doing this would be a huge pain, unless the makefile system gets a big overhaul, to make things like this easier. The problem is, that the keymaps and keyboards are built in a for loop, and make gets called recursively many times, and the exit status would need to be bubbled up. That's ugh...

A better way would be to use dependencies, such that make keyboards/ergodox/keymaps/algernon would compile just that, and make keyboards/ergodox would be a rule that depends on all the keymaps, and does nothing else.

This would require a major overhaul of the build system, and I'm not prepared for that at the moment. If anyone finds a way to only fail at the very end, and still keep the current fail-early behaviour as an option, without sacrificing many goats to the gods of Make, please let me know.

Closing the PR, because I'm not able to make it work with the resources I have.

@algernon algernon closed this Jul 30, 2016
@algernon algernon deleted the f/make-all-keep-going branch July 30, 2016 07:04
@fredizzimo fredizzimo mentioned this pull request Aug 20, 2016
BlueTufa pushed a commit to BlueTufa/qmk_firmware that referenced this pull request Aug 6, 2021
* Add default keymaps for Zinc

revs. 1 and A

* Add default keymap for ZJ68
almaqyn pushed a commit to almaqyn/qmk_firmware that referenced this pull request Nov 23, 2023
* Add Vial support to Keebio Sinc keyboard

* Move Vial keymap to rev4 specifically

* Remove unnecessary layer
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants