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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
matomo: 3.11.0 -> 3.13.0 #74319
matomo: 3.11.0 -> 3.13.0 #74319
Conversation
@Kiwi please rebase as I merged the PR for your first commit. |
816d00b
to
2da6f99
Compare
You just love to make me work don't you :P |
2da6f99
to
5594fdf
Compare
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.
tested with nix-review
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 to all for testing that Matomo 3.13 indeed fixes our problems with 3.12. Let's rework and rethink PermissionsStartOnly / the PreStart script later. I did not know of stateVersion at the time which would have been the right thing to do, but we can probably start to just drop the migration code and maybe even the piwik aliases by now as 19.09 is the second stable release with the renaming.
backport of NixOS#74319 without the file consistency checks. 3.12 was rated a major security update but was broken for NixOS, therefore jump to 3.13 which incorporates the necessary fix.
backport of #74319 without the file consistency checks. 3.12 was rated a major security update but was broken for NixOS, therefore jump to 3.13 which incorporates the necessary fix.
Motivation for this change
The long awaited update to Matomo 3.12.0 has finally arrived! Here it comes... wait what do you mean you missed it?! Oh no, too slow! Well here's 3.13.0 instead!
Things done
I checked out this other PR before I made these additional changes. Now that one is clearly inferior at this point and should be closed in favor of this one so that I don't have to redo everything. 馃槃
I had to add an additional file to the list to fix; and then add in a check to keep the builds succeeding. The matomo-beta would fail when
wc
tried to count a non-existent file that is only in the latest and greatest. No file? No problem! nop!I also removed the recently (within the last few months, I'm sure) deprecated PermissionsStartOnly; and while I'm not certain that there aren't additional necessary changes that I am unaware of I do know I've been running it in production...sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @florianjacob @aanderse @dasJ @Kiwi