Navigation Menu

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

teams: init at 1.2.00.32451 #75505

Merged
merged 1 commit into from Dec 12, 2019
Merged

teams: init at 1.2.00.32451 #75505

merged 1 commit into from Dec 12, 2019

Conversation

liff
Copy link
Contributor

@liff liff commented Dec 11, 2019

Motivation for this change

A Linux version of Microsoft Teams was released.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • 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.
Notes

It’s somewhat similar to Slack so I was using the slack derivation as a reference.

@ghost
Copy link

ghost commented Dec 11, 2019

When trying to execute this package, It crashes with the following stacktrace :

> teams
A JavaScript error occurred in the main process
Uncaught Exception:
Error: libstdc++.so.6: cannot open shared object file: No such file or directory
    at process.module.(anonymous function) [as dlopen] (ELECTRON_ASAR.js:143:31)
    at Object.Module._extensions..node (internal/modules/cjs/loader.js:722:18)
    at Object.module.(anonymous function) [as .node] (ELECTRON_ASAR.js:152:18)
    at Module.load (internal/modules/cjs/loader.js:602:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:541:12)
    at Function.Module._load (internal/modules/cjs/loader.js:533:3)
    at Module.require (internal/modules/cjs/loader.js:640:17)
    at require (/nix/store/w6w2r130ma4aqcn5inxgnvqcjcv5rqzj-teams-1.2.00.32451/opt/teams/resources/app.asar/external/v8-compile-cache/v8-compile-cache.js:173:28)
    at Object.<anonymous> (/nix/store/w6w2r130ma4aqcn5inxgnvqcjcv5rqzj-teams-1.2.00.32451/opt/teams/resources/app.asar/node_modules/native-utils/index.js:1:173)
    at Object.<anonymous> (/nix/store/w6w2r130ma4aqcn5inxgnvqcjcv5rqzj-teams-1.2.00.32451/opt/teams/resources/app.asar/node_modules/native-utils/index.js:3:3)

The slack derivation seems to address this issue at line 50. I will try to do it locally when I get back from work.

@liff
Copy link
Contributor Author

liff commented Dec 12, 2019

Thanks for testing, @elyhaka. Wonder if I should close this in favor of #75525 though?

@worldofpeace
Copy link
Contributor

worldofpeace commented Dec 12, 2019

They're largely similar, but https://github.com/NixOS/nixpkgs/pull/75525/files uses autoPatchelfHook and wrapGAppsHook which i would've suggested here as well. Though the other is a draft.

@liff
Copy link
Contributor Author

liff commented Dec 12, 2019

Changed to use wrapGAppsHook and autoPatchelfHook, and added some dependencies caught by the latter.

@ghost
Copy link

ghost commented Dec 12, 2019

Great ! It works perfectly for me with this update (no more libstdc++ issue) :)

@worldofpeace
Copy link
Contributor

This expression looks pretty ready @liff 👍

@liff
Copy link
Contributor Author

liff commented Dec 12, 2019

Since #75525 was closed, should I add @jonringer as a maintainer?

Copy link
Contributor

@worldofpeace worldofpeace left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. I will defer to people who actually use the application.

Copy link
Member

@aanderse aanderse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have tested this PR and the software works as expected. Thank you very much @liff 🎉

@jonringer I'll leave it to you to convince whomever you need to convince that we need a linux port of outlook next 😉 😆

@aanderse aanderse merged commit 06749e4 into NixOS:master Dec 12, 2019
@jonringer
Copy link
Contributor

not sure why, but i was having problems on my workstation, home computer it works fine though

@jonringer
Copy link
Contributor

I just did a call using this, the audio is very poor, not entire sure why. maybe using libpulseaudio will help remedy that

@jonringer
Copy link
Contributor

also, it was eating my close application, :(

@aanderse
Copy link
Member

@jonringer did not use for phone call but used during the day for regular stuff and so far no issues... What is close application?

@jonringer
Copy link
Contributor

If I try to click the X in the upper right-hand corner

@aanderse
Copy link
Member

Oh sorry I understand... I was able to click and then I received notification that teams wouldn't exit but remain in tray, as expected.

Maybe I'll try to install on another device 🤔

@jonringer
Copy link
Contributor

The other thing is that, I'm only using lightdm+i3, and no desktop environment. Probably the cause of the wonky issues.

@aanderse
Copy link
Member

Yeah definitely might be. I'm on plasma.

I'll test voice call tomorrow.

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

4 participants