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

installer: don't uninstall on empty OLD_STATE. #28

Merged
merged 1 commit into from
Jan 24, 2024

Conversation

gmodena
Copy link
Owner

@gmodena gmodena commented Jan 23, 2024

Check that OLD_STATE has been populated before attempting to diff it with NEW_STATE,
when determining what packages to uninstall.

Fixes the following error:

jq: error (at <unknown>): null (null) and array (["im.riot.R...) cannot be subtracted

This is the case on newly provisioned instances, with no flatpaks previously installed (either managed by nix-flatpak or unmanaged).

Check that OLD_STATE has been populated
before attempting to diff it with NEW_STATE
to determine what packages to uninstall.
@gmodena gmodena self-assigned this Jan 23, 2024
@gmodena gmodena merged commit dfa162f into main Jan 24, 2024
1 check passed
@Tomaszal Tomaszal mentioned this pull request Jan 30, 2024
@gmodena gmodena deleted the fix-empty-oldstate-install branch October 20, 2024 19:23
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.

1 participant