Skip to content

base-files: add migration information for APK #17098

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Nov 27, 2024
Merged

base-files: add migration information for APK #17098

merged 1 commit into from
Nov 27, 2024

Conversation

aparcar
Copy link
Member

@aparcar aparcar commented Nov 27, 2024

If the apk package manager is installed, show a table of common commands to deal with the system.

@aparcar aparcar requested review from ynezz and Ansuel November 27, 2024 16:11
@github-actions github-actions bot added the core packages pull request/issue for core (in-tree) packages label Nov 27, 2024
@efahl
Copy link
Contributor

efahl commented Nov 27, 2024

I might add a caveat at the bottom regarding use of "upgrade" (or maybe leave it out???), as we've long discouraged people from doing a blind opkg upgrade due to the risk of breaking a shady ABI-versioned library.

@hnyman
Copy link
Contributor

hnyman commented Nov 27, 2024

Please start with apk update, as that is usually needed first (before apk add etc.)

@ynezz
Copy link
Member

ynezz commented Nov 27, 2024

Please start with apk update, as that is usually needed first (before apk add etc.)

apk add will do the update of the feeds when needed, no need to do it explicitly.

ynezz
ynezz approved these changes Nov 27, 2024
View reviewed changes
Copy link
Member

@ynezz ynezz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@aparcar
Copy link
Member Author

aparcar commented Nov 27, 2024

Please feel free to merge I'm not on my computer anytime soon

@Ansuel
Copy link
Member

Ansuel commented Nov 27, 2024

@aparcar ok to include changes from @ynezz ?

@aparcar @Ansuel
base-files: add migration information for APK
c19c31f 
If the `apk` package manager is installed, show a table of common
commands to deal with the system.

Signed-off-by: Paul Spooren <mail@aparcar.org>
@openwrt-bot openwrt-bot merged commit c19c31f into openwrt:main Nov 27, 2024
3 checks passed
@hnyman
Copy link
Contributor

hnyman commented Nov 27, 2024

Maybe the spelling mistake should be corrected...

OpenWrt recently switched to the "apk" package manger!
-->
OpenWrt recently switched to the "apk" package manager!

@Ansuel
Copy link
Member

Ansuel commented Nov 27, 2024 •
edited
Loading

>:(

@efahl
Copy link
Contributor

efahl commented Nov 27, 2024

OpenWrt recently switched to the "apk" package manger!

I thought it should be mangler.

@raenye
Copy link
Contributor

raenye commented Nov 29, 2024

Maybe also add a warning about apk upgrade -- this upgrades all packages without confirmation. Caught me by surprise.

@ynezz
Copy link
Member

ynezz commented Nov 30, 2024

Maybe also add a warning about apk upgrade -- this upgrades all packages without confirmation. Caught me by surprise.

@raenye You can propose improvements via pull request :)

BTW since this is still ongoing effort, we've added link to https://openwrt.org/docs/guide-user/additional-software/opkg-to-apk-cheatsheet where you can help make this clear. There is https://openwrt.org/docs/guide-user/additional-software/apk which needs some work as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ynezz ynezz ynezz approved these changes

@Ansuel Ansuel Awaiting requested review from Ansuel

Assignees
No one assigned
Labels
core packages pull request/issue for core (in-tree) packages
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@aparcar @efahl @hnyman @ynezz @Ansuel @raenye @dave14305 @openwrt-bot