Document process about upgrade buildroot and kernel#1049
Conversation
mattiaswal
force-pushed
the
document-kernel-buildroot-update
branch
from
d93c48f to
e08b702
Compare
jovatn
left a comment
jovatn
left a comment
There was a problem hiding this comment.
LGTM! But someone more skilled should also have a look.
Some minor comments and questions included.
doc/developers-guide.md
Outdated
| branches should contain **only** changes to existing packages (but no | ||
| new patches), modifications to Buildroot itself or upstream backports. |
There was a problem hiding this comment.
Probably obvious to those who understand, but to me it is not clear what "(but no new patches)" means. E.g., is it the "new" that is important, and "old patches" are ok?
And should "(but no new patches)" be put at the end of the sentence, or does it only apply to "existing packages"?
There was a problem hiding this comment.
Always a case by case, new is a (strict) guideline :D
mattiaswal
force-pushed
the
document-kernel-buildroot-update
branch
from
e08b702 to
5f01a50
Compare
troglobit
left a comment
troglobit
left a comment
There was a problem hiding this comment.
Some minor stylistic and formatting related comments only. Other than that, great work!
mattiaswal
force-pushed
the
document-kernel-buildroot-update
branch
from
5f01a50 to
eabaaac
Compare
troglobit
left a comment
troglobit
left a comment
There was a problem hiding this comment.
I like it, nice work on this! (Sorry for dragging my feet.)
4 participants
[skip ci]
Description
Checklist
Tick relevant boxes, this PR is-a or has-a: