ClangBuiltLinux / continuous-integration

Continuous integration of latest Linux kernel with daily build of Clang & LLVM tools
https://travis-ci.com/ClangBuiltLinux/continuous-integration
Apache License 2.0
44 stars 18 forks source link

add facility to apply patches to all branches. #136

Closed nickdesaulniers closed 5 years ago

nickdesaulniers commented 5 years ago

This probably wont get used too often, as patches probably wont apply cleanly across all these different branches, but makes it simple to apply patches to many trees.

Also, add patch for https://github.com/ClangBuiltLinux/linux/issues/416

nathanchance commented 5 years ago

I would also presubmit one architecture on each different repo/branch combo that we support, just to make sure the patch applied (I've tested arm64 on common 4.9 and stable 4.4).

nickdesaulniers commented 5 years ago

odd, @nathanchance any idea why the build is stopping after applying the patches? Seems like the last output is + [[ -d ../patches/linux/arm ]].

nickdesaulniers commented 5 years ago

https://stackoverflow.com/a/17336953/1027966:

The default return value is the exit value of the last statement executed within the function.

so I probably need an explicit return 0 in there.

nathanchance commented 5 years ago

I would do an if else with an explicit return zero in the else case, otherwise we won't capture the git apply error code.

nickdesaulniers commented 5 years ago

green presubmit showing patch applies to each branch: https://travis-ci.com/ClangBuiltLinux/continuous-integration/builds/104603172