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
nextcloud: 19.0.1 -> 19.0.3, deprecate/remove older versions #98333
Conversation
cc @worldofpeace @jonringer I'd consider this and #98332 relevant for 20.09 :) |
I guess this should be a blocker then |
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.
diff LGTM
tests pass
https://github.com/NixOS/nixpkgs/pull/98333
1 package marked as broken and skipped:
nextcloud18
1 package blacklisted:
tests.nixos-functions.nixos-test
1 package built:
nextcloud19
@worldofpeace anything else to add? I'd consider this good to go and would backport after merge :) |
I had @cole-h look at the release note and he pre-approved on IRC with my suggested changes. |
0b97603
to
dc16ba5
Compare
@worldofpeace @cole-h thanks, fixed :) |
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.
2 more nits, and then diff LGTM.
Both packages will get EOLed within the lifetime of 20.09. `nextcloud17` can be removed entirely (the attribute-path is kept however to provide meaningful errors), however `nextcloud18` must be kept as `insecure` to make sure that users from `nextcloud17` can properly upgrade to `nextcloud19` on NixOS 20.09.
dc16ba5
to
6b48dee
Compare
@GrahamcOfBorg eval |
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.
Thanks!
Motivation for this change
Closes #97751.
ChangeLogs:
Also removes
nextcloud17
and deprecatesnextcloud18
since both will be EOLed within the lifetime of 20.09[1]. Whilenextcloud17
can be removed entirely now,nextcloud18
must stay (and marked as insecure) to make sure that users from Nextcloud17 can seamlessly upgrade tonextcloud19
on 20.09.[1] https://docs.nextcloud.com/server/19/admin_manual/release_schedule.html
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)