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

vscx/ms-vsliveshare-vsliveshare: 1.0.2902 -> 1.0.3912 #115277

Merged
merged 1 commit into from
Mar 7, 2021

Conversation

deviant
Copy link
Member

@deviant deviant commented Mar 6, 2021

This was broken on vscode >= 1.53.0.

Cleans up a bunch of things:

  • uses substitution over sed & wrappers, messing with $PATH less and making everything more clear
  • substitutes xprop in the correct place (this was never actually used in the launcher scripts)
  • substitutes references to xsel — although this isn't strictly necessary, as it bundles the binary(!)
  • substitutes hardcoded /bin/bash (the vsls scheme handler cannot have actually worked before on NixOS)

Supersedes #112909.

@deviant deviant requested a review from jonringer as a code owner March 6, 2021 18:25
@ofborg ofborg bot requested a review from jraygauthier March 6, 2021 18:35
@ofborg ofborg bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 1-10 10.rebuild-linux: 1 labels Mar 6, 2021
@SuperSandro2000
Copy link
Member

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.

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

1 package built:
  • vscode-extensions.ms-vsliveshare.vsliveshare

@teto
Copy link
Member

teto commented Mar 6, 2021

I've just tested it and it fixed for me. Now I wonder if #115249 wont break it again ?

@deviant
Copy link
Member Author

deviant commented Mar 7, 2021

@teto: I would hope not! This is the latest version as per mp.vs/MS-vsliveshare.vsliveshare.

The extension was broken due to not having been updated in nixpkgs for months, while depending on an unstable "proposed" API that was later removed. Having automatic update PRs for VS Code extensions would help prevent future issues of this sort cropping up (and more closely reflect how they're used by normal people). @ryantm, is this something that would be simple to add to your bot?

This is the place for API experiments and proposals.
These API are NOT stable and subject to change. They are only available in the Insiders
distribution and CANNOT be used in published extensions.

Perhaps Microsoft should read their own documentation :)

@teto teto merged commit f956dd3 into NixOS:master Mar 7, 2021
@deviant deviant deleted the bump-vsls branch March 7, 2021 10:47
@jraygauthier
Copy link
Member

@deviant: Excellent cleanup of the original draft. 👍

@jraygauthier
Copy link
Member

Works fine too here on nixos, using vscode 1.53.2, with extension 1.03912 both local instances built from latest nixpkgs:

$ cd /path/to/latest/nixpkgs
$ nix-build -I nixpkgs=. -E 'with import <nixpkgs> {config={allowUnfree = true;};}; vscode-with-extensions.override {vscodeExtensions = with vscode-extensions; [ms-vsliveshare.vsliveshare];}'
$ ./result/bin/code .

JesusMtnez added a commit to JesusMtnez/dotfiles that referenced this pull request Mar 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 1-10 10.rebuild-linux: 1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants