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.

Sign up for GitHub

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

Jump to bottom

citrix_receiver: 13.7.0 -> 13.8.0 #33556

Merged
merged 1 commit into from Jan 8, 2018
Merged

citrix_receiver: 13.7.0 -> 13.8.0 #33556

merged 1 commit into from Jan 8, 2018

Conversation

FRidh
Copy link
Member

@FRidh FRidh commented Jan 7, 2018

Motivation for this change
Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-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 nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@FRidh FRidh requested a review from obadz January 7, 2018 08:26
@obadz obadz merged commit 3b36db7 into NixOS:master Jan 8, 2018
@obadz
Copy link
Contributor

obadz commented Feb 24, 2018

@FRidh, does this commit build for you? I get:

./linuxx64/hinst: line 1286: syntax error: you disabled math support for $((arith)) syntax

obadz added a commit that referenced this pull request Feb 24, 2018
@obadz
citrix_receiver: fix build errors in recent version
5e8fd81 
./linuxx64/hinst: line 1286: syntax error: you disabled math support for $((arith)) syntax

(See #33556)

(cherry picked from commit 2954283)
obadz added a commit that referenced this pull request Feb 24, 2018
@obadz
citrix_receiver: fix build errors in recent version
2954283 
./linuxx64/hinst: line 1286: syntax error: you disabled math support for $((arith)) syntax

(See #33556)
@FRidh FRidh deleted the citrix branch February 24, 2018 13:19
@FRidh
Copy link
Member Author

FRidh commented Feb 24, 2018

I'm pretty sure it did at the time. I gave up on the citrix receiver application though and use a Google Chrome plugin instead.

@obadz
Copy link
Contributor

obadz commented Feb 24, 2018

Ha.. didn't know such a thing existed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@obadz obadz Awaiting requested review from obadz

Assignees
No one assigned
Labels
8.has: package (update) 10.rebuild-darwin: 0 10.rebuild-linux: 1-10
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@FRidh @obadz @GrahamcOfBorg