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

citrix_workspace: add 1906 (citrix_workspace_19_6_0) #64645

Merged
merged 1 commit into from Jul 24, 2019

Conversation

Ma27
Copy link
Member

@Ma27 Ma27 commented Jul 11, 2019

Motivation for this change

New release available:
https://www.citrix.com/downloads/workspace-app/linux/workspace-app-for-linux-latest.html

Apart from the new version the following things changed:

  • Updated the docs as all notes about citrix_receiver also apply for
    citrix_workspace. Also added a deprecation warning about the
    upcoming removal.

  • Removed the libidn_134 override as neither citrix_workspace_19_3_0
    nor citrix_workspace_19_6_0 require this library anymore according
    to readelf -d ./result/opt/citrix-icaclient/wfica (in contrast to
    citrix_receiver_13_10_0).

  • Added myself as maintainer as well.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.

New release available:
https://www.citrix.com/downloads/workspace-app/linux/workspace-app-for-linux-latest.html

Apart from the new version the following things changed:

* Updated the docs as all notes about `citrix_receiver` also apply for
  `citrix_workspace`. Also added a deprecation warning about the
  upcoming removal.

* Removed the `libidn_134` override as neither `citrix_workspace_19_3_0`
  nor `citrix_workspace_19_6_0` require this library anymore according
  to `readelf -d ./result/opt/citrix-icaclient/wfica` (in contrast to
  `citrix_receiver_13_10_0`).

* Added myself as maintainer as well.
@Ma27
Copy link
Member Author

Ma27 commented Jul 11, 2019

cc @garry-cairns @FRidh @obadz

As Citrix Receiver will be EOLed at the end of 2019 (https://www.citrix.com/support/product-lifecycle/milestones/receiver.html) we should probably drop this before 19.09.

@FRidh
Copy link
Member

FRidh commented Jul 12, 2019

@Ma27 I am fine with that.

@FRidh FRidh merged commit 0a571ae into NixOS:master Jul 24, 2019
@FRidh
Copy link
Member

FRidh commented Jul 24, 2019

Still get a segmentation fault unfortunately :(

@Ma27
Copy link
Member Author

Ma27 commented Jul 24, 2019

What do you mean with "Still"?

@Ma27 Ma27 deleted the citrix-workspace-1906 branch July 24, 2019 11:22
@FRidh
Copy link
Member

FRidh commented Jul 24, 2019

Well, with version 19.03 I got segfaults at some point as well. I never bothered with bisecting it, primarily because with Citrix you'll never know whether it works anyway...

@Ma27
Copy link
Member Author

Ma27 commented Jul 24, 2019

Hmm that's pretty unfortunate as I tested the package on master and cherry-picked the commit onto my personal release-19.03 tracking branch. Is there any chance to get a minimal VM where I could reproduce this? :)

obadz added a commit to obadz/nixpkgs that referenced this pull request Sep 14, 2019
Ma27 pushed a commit that referenced this pull request Sep 14, 2019
Already documented in #64645

(cherry picked from commit e5e6b51)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants