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
mattermost-desktop: 4.1.2 -> 4.2.0 #56289
Conversation
P.S.: I have also added my GPG key |
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.
Hello @jokogr
Could you please drop
phases = [ "unpackPhase" "installPhase" ];
and replace it with?
dontBuild = true;
dontConfigure = true;
We'd like to deprecate that.
See: #28910
Additionally, after inspecting the output I noticed that everything was
just copied to $out
with no type of hierarchy.
Could you refactor the installPhase
to not do that?
This can be as simple as coping everything to $out/share/mattermost-desktop
and then symlinking things from there.
@worldofpeace thanks for the review! I've refactored the phases to use |
pkgs/applications/networking/instant-messengers/mattermost-desktop/default.nix
Outdated
Show resolved
Hide resolved
6f83cb1
to
2e9c741
Compare
@worldofpeace ready! |
Thanks @jokogr 😄 |
Motivation for this change
Mattermost Desktop update was pending since November 2018.
New version seems to be using GTK 3 and has a Windows security vulnerability fixed (I guess no worries for that).
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)