Add some distribution configurations #172
Conversation
nathanchance
force-pushed
the
distro-configs
branch
2 times, most recently
from
3c0fda4 to
f4ac98e
Compare
|
https://gitlab.com/Linaro/tuxsuite/-/issues/124 has been resolved so we can do this now. Certain builds are broken in -next but that is just -next for ya :^) |
…ith clang 11+ Closes: ClangBuiltLinux#25 Signed-off-by: Nathan Chancellor <[email protected]>
Signed-off-by: Nathan Chancellor <[email protected]>
nathanchance
force-pushed
the
distro-configs
branch
from
f4ac98e to
0fc784b
Compare
They should boot fine as I boot test these locally but I will have to modify the logic in |
…ance Signed-off-by: Nathan Chancellor <[email protected]>
Signed-off-by: Nathan Chancellor <[email protected]>
Signed-off-by: Nathan Chancellor <[email protected]>
Signed-off-by: Nathan Chancellor <[email protected]>
|
Alright, I believe this should work fine, as |
Signed-off-by: Nathan Chancellor <[email protected]>
Rather than checking if the entry exists in the map and using the value of the architecture in the URL if not, just add the entries to the map and always return from the map. This is a little bit more verbose but there will be a more obvious error if we add an architecture without supporting it here. Suggested-by: Nick Desaulniers <[email protected]> Signed-off-by: Nathan Chancellor <[email protected]>
|
I am going to merge this without explicit approval after our conversation offline. Please scream if you see anything wrong after the fact and I am more than happy to push subsequent fixups. |
|
I pushed f9f368d to make the build green. I am not sure it warrants an investigation on our part given it happens with GCC but I can open a follow up issue if you feel it is worth looking into. |
Arch, Fedora, and OpenSUSE to start because their configurations are easily accessible from the web.
This is currently blocked on https://gitlab.com/Linaro/tuxsuite/-/issues/124 in its current form, I am just pushing for early review.