Skip to content
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

luci-app-attendedsysupgrade: sort latest version array #7579

Merged
merged 1 commit into from
Jan 24, 2025

Conversation

efahl
Copy link
Contributor

@efahl efahl commented Jan 23, 2025

The Attended Sysupgrade app requires that the array of latest versions be in order from newest release to oldest. Ensure that this is true even when upstream presents them in arbitrary order.

The Attended Sysupgrade app requires that the array of latest
versions be in order from newest release to oldest.  Ensure that
this is true even when upstream presents them in arbitrary order.

Signed-off-by: Eric Fahlgren <ericfahlgren@gmail.com>
@efahl
Copy link
Contributor Author

efahl commented Jan 24, 2025

Should be backported to 24.10.

@hnyman hnyman merged commit 2c7b18b into openwrt:master Jan 24, 2025
5 checks passed
@aparcar
Copy link
Member

aparcar commented Jan 24, 2025

I think this isn't the best solution, it requires clients to update packages before a firmware upgrade. We should always avoid that

@systemcrash
Copy link
Contributor

I think this isn't the best solution, it requires clients to update packages before a firmware upgrade. We should always avoid that

And that's because of a list sort?

@hnyman
Copy link
Contributor

hnyman commented Jan 24, 2025

isn't the best solution, it requires clients to update packages before a firmware upgrade. We should always avoid that

And that's because of a list sort?

Likely aparcar means that is does not yet benefit those who are trying to upgrade from versions before this patch. But this will help in future when later upgrading from versions with this patch.

@efahl
Copy link
Contributor Author

efahl commented Jan 24, 2025

Plus Paul has merged a change to the ASU server, too, so we're fixing the issue in two ways: ASU server sorts according to the LuCI app's preference (handles 23.05 and earlier app users); and with this the PR LuCI app is robust in the presence of unsorted data (handles any changes we might make to ASU server in the future).

@efahl efahl deleted the latest-order branch January 24, 2025 16:21
@systemcrash
Copy link
Contributor

Sorry, I read "update packages" as "update all packages" instead of "update this package".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants