-
-
Notifications
You must be signed in to change notification settings - Fork 9.9k
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
tools: refresh all patches #15524
tools: refresh all patches #15524
Conversation
We actually do the same here but those packages depends on the target so they were never checked... for tools is should really check why the action doesn't work with checking to-refresh patches... |
I just noticed these lz4 patches are completely pointless. meson is used for lz4, not make. |
3fcda1f
to
83aaa19
Compare
@Ansuel is this ready to go? |
@neheb I would ask just some additional hours or maybe a day... I just fixed a bug with tools/refresh and I introduced support for it in CI... Lets see how it goes! |
@neheb can I drop the packages commit and can we handle that in a different PR? |
OK |
@neheb maybe create a dedicated branch for the packages commit so you don't lose it? |
Yeah I got that. |
@Ansuel slight modification. Smaller refresh. |
meson is used to build it since 291b137. No need to patch Makefiles. Signed-off-by: Rosen Penev <rosenp@gmail.com> Link: openwrt#15524 Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
Refresh all tools patches now that tools/refresh correctly works. CI now checks for them and actively complain if tools have unrefreshed patches. Signed-off-by: Rosen Penev <rosenp@gmail.com> [ reword commit message ] Link: openwrt#15524 Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
meson is used to build it since 291b137. No need to patch Makefiles. Signed-off-by: Rosen Penev <rosenp@gmail.com> Link: openwrt#15524 Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
Refresh all tools patches now that tools/refresh correctly works. CI now checks for them and actively complain if tools have unrefreshed patches. Signed-off-by: Rosen Penev <rosenp@gmail.com> [ reword commit message ] Link: openwrt#15524 Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
CI mandates clean patches in the packages feed but not here. Hrm.