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-for-linux: 1.0.45 -> 1.0.53 #219393

Merged

Conversation

lilyinstarlight
Copy link
Member

@lilyinstarlight lilyinstarlight commented Mar 3, 2023

Description of changes

The biggest changes are having macOS support and having a multi-account command-line flag. Other release notes at https://github.com/ismaelmartinez/teams-for-linux/releases

I added myself as maintainer and am letting OfBorg test to see if it builds on macOS or not (and I'll mark it broken if not)

Versions in-between these were waiting on #214062 but this version works without it again (so some of the drive-by changes were originally related to getting it to work with that)

I also tossed in a fix for notification sounds

Fixes #215593

Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 23.05 Release Notes (or backporting 22.11 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

@ofborg ofborg bot added the 8.has: package (new) This PR adds a new package label Mar 3, 2023
@ofborg ofborg bot requested a review from muscaln March 3, 2023 19:23
@lilyinstarlight lilyinstarlight force-pushed the upd/teams-for-linux-1.0.49 branch 3 times, most recently from c3ed927 to 95ffe74 Compare March 3, 2023 21:08
@lilyinstarlight lilyinstarlight marked this pull request as draft March 4, 2023 19:25
@lilyinstarlight lilyinstarlight force-pushed the upd/teams-for-linux-1.0.49 branch from 95ffe74 to 356785d Compare March 7, 2023 14:51
@lilyinstarlight
Copy link
Member Author

I'm just marking darwin broken for now unless someone else figures something out or I get access to the Darwin community builder (nix-community/darwin-build-box#12) and can debug

adamcstephens

This comment was marked as outdated.

@figsoda figsoda added the 12.approvals: 1 This PR was reviewed and approved by one reputable person label Mar 10, 2023
The biggest changes are having macOS support and having a multi-account
command-line flag. See all other changes at
<IsmaelMartinez/teams-for-linux@v1.0.45...v1.0.52>

macOS support is not currently working so it has been marked broken
after expanding `meta.platforms`.
@lilyinstarlight lilyinstarlight force-pushed the upd/teams-for-linux-1.0.49 branch from 92550ec to fadb7ec Compare March 15, 2023 14:10
@lilyinstarlight lilyinstarlight changed the title teams-for-linux: 1.0.45 -> 1.0.52 teams-for-linux: 1.0.45 -> 1.0.53 Mar 15, 2023
Copy link
Contributor

@adamcstephens adamcstephens left a comment

Choose a reason for hiding this comment

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

Tested on x86_64-linux and was able to log in and use Teams.


Result of nixpkgs-review pr 219393 run on x86_64-linux 1

1 package built:
  • teams-for-linux

Result of nixpkgs-review pr 219393 run on aarch64-linux 1

1 package built:
  • teams-for-linux

@lilyinstarlight
Copy link
Member Author

Fixes a known bug (two people verified on the issue thread) and has an approval already, merging

@lilyinstarlight lilyinstarlight merged commit 0b8da88 into NixOS:master Mar 17, 2023
@lilyinstarlight lilyinstarlight deleted the upd/teams-for-linux-1.0.49 branch March 17, 2023 21:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
8.has: package (new) This PR adds a new package 10.rebuild-darwin: 1-10 10.rebuild-darwin: 1 10.rebuild-linux: 1-10 10.rebuild-linux: 1 11.by: package-maintainer This PR was created by the maintainer of the package it changes 12.approvals: 1 This PR was reviewed and approved by one reputable person
Projects
None yet
Development

Successfully merging this pull request may close these issues.

teams-for-linux: JavaScript TypeError "properties of undefined (reading 'play')"
3 participants