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
"seq" tool check fixed #1926
"seq" tool check fixed #1926
Conversation
The PR has formal issues as it has no SoB; please read https://openwrt.org/submitting-patches. |
@dedeckeh hmm, I add signoff to comment, but can't make force push because pull request was made from github web ui I think. Is it enough? |
@imShara No it's not as the SoB needs to be added to the git commit description. You can do git commit --amend -s followed by git push -f |
Are you going to address the comments so we can move forward with this PR? |
Problem found on alpine linux when tried to `./scripts/feeds update -a` ```sh Build dependency: Missing seq command ``` Signed-off-by: imShara <shara@protonmail.com>
@dedeckeh ready to merge? |
Nope, you need to provide your real name in |
Pulled into my staging tree at https://git.openwrt.org/openwrt/staging/jow.git |
Problem found on alpine linux when trying to `./scripts/feeds update -a`, which results in `Build dependency: Missing seq command`. Ref: #1926 Suggested-by: imShara <shara@protonmail.com> [reworded commit and turned faulty Sob into Suggested-by] Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Problem found on alpine linux when trying to `./scripts/feeds update -a`, which results in `Build dependency: Missing seq command`. Ref: openwrt/openwrt#1926 Suggested-by: imShara <shara@protonmail.com> [reworded commit and turned faulty Sob into Suggested-by] Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Problem found on alpine linux when tried to
./scripts/feeds update -a
Build dependency: Missing seq command
Signed-off-by: Shara shara@protonmail.com