Commit c3a86691 authored by Yarny0's avatar Yarny0
Browse files

zoom-us: drop `extraPreBwrapCmds` (`QT_PLUGIN_PATH` & `LANG`) 

References/History:

* `unset LANG` was introduced in
  69ec89cd
  https://github.com/NixOS/nixpkgs/pull/410244
  for version 6.4.10.*,
  to restore the visually broken settings dialog.
  It seems no longer necessary as of version 6.5.1, cf.
  https://github.com/NixOS/nixpkgs/pull/419757#pullrequestreview-2959221263

* `unset QT_PLUGIN_PATH` was introduced in
  b747aa0f
  https://github.com/NixOS/nixpkgs/pull/116355
  for version 5.5.7938.0228,
  to fix the display of the "Participants" list.
  In recent versions, it ensured that the avatar icons
  of persons (conference and team chat "Participants")
  are displayed correctly.
  It seems no longer necessary as of version 6.5.3.*, cf.
  https://github.com/NixOS/nixpkgs/pull/422532#pullrequestreview-2989636069
parent 62e0f05e

pkgs/by-name/zo/zoom-us/package.nix

+0 −4
Original line number Diff line number Diff line
@@ -245,10 +245,6 @@ let 
    version = versions.${system} or throwSystem;



    targetPkgs = pkgs: (linuxGetDependencies pkgs) ++ [ unpacked ];

    extraPreBwrapCmds = ''

      unset QT_PLUGIN_PATH

      unset LANG  # would break settings dialog on non-"en_XX" locales

    '';

    extraBwrapArgs = [ "--ro-bind ${unpacked}/opt /opt" ];

    runScript = "/opt/zoom/ZoomLauncher";