-
Notifications
You must be signed in to change notification settings - Fork 61
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
fix(webinstall): Correct use of makepkg
for Arch Linux
#838
Conversation
makepkg
for Arch Linux
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @SorenEricMent,
Thanks a lot for this fix. I left a comment for a change.
Also, can you change you commit message to have this format? Thanks
fix(webinstall): Correct use of `makepkg` for Arch Linux
Please also squash the 2 commits together and add the name above |
f3bb7bd
to
09c0cc1
Compare
Done! Sorry I was bit last, kinda busy with school stuff xx |
@SorenEricMent It's fine, no worries, but it seems you rebased one more commit and this PR now has two commits 😅 Can you drop the second to last one and leave only yours? Thanks 🙏 |
Signed-off-by: Yunchuan "Winslow" Hu i@winsloweric.com Fix makepkg for Archlinux Signed-off-by: Yunchuan "Winslow" Hu <i@winsloweric.com> Update tools/webinstall/install.sh Tested, totally work, thank you. Co-authored-by: Cezar Craciunoiu <craciunoiu.cezar@yahoo.com> Signed-off-by: WinslowEric <i@winsloweric.com>
Should be good to go! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All good here. Thanks!
Reviewed-by: Cezar Craciunoiu cezar.craciunoiu@unikraft.io
Approved-by: Cezar Craciunoiu cezar.craciunoiu@unikraft.io
Prerequisite checklist
make fmt
on your commit series before opening this PR;This change does not impact building, thus there is no need for running make fmt nor updating relevant documentation. This patch is Arch Linux platform specific.
Description of changes
Fixed makepkg command for ArchLinux, addressing issue #837
Signed-off-by: Yunchuan "Winslow" Hu i@winsloweric.com