-
-
Notifications
You must be signed in to change notification settings - Fork 14k
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.53 → 1.0.59, add updateScript #222440
teams-for-linux: 1.0.53 → 1.0.59, add updateScript #222440
Conversation
3eef1db
to
77c56e3
Compare
77c56e3
to
c56fb30
Compare
Result of 1 package built:
|
c56fb30
to
88c6054
Compare
88c6054
to
60c5fd1
Compare
60c5fd1
to
2a2dfe5
Compare
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-ready-for-review/3032/2068 |
vendorhash() { | ||
(nix --extra-experimental-features nix-command build --impure --argstr nixpkgs "$nixpkgs" --argstr attr "$1" --expr '{ nixpkgs, attr }: let pkgs = import nixpkgs {}; in with pkgs.lib; (getAttrFromPath (splitString "." attr) pkgs).overrideAttrs (attrs: { outputHash = fakeHash; })' --no-link 2>&1 >/dev/null | tail -n3 | grep -F got: | cut -d: -f2- | stripwhitespace) 2>/dev/null || true | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should get this into nix-update if not already done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah this one's a bit of a special one since it's not using any standard Yarn builder atm and we would have to just tell it that a package having .offlineCache
means Yarn (which is a reasonably strong convention but isn't 100% consistent for packages like this since it's not a standard builder)
Once we have buildYarnPackage
though (#210814) we'll add support for it and yarnDeps
in nix-update
findpath() { | ||
path="$(nix --extra-experimental-features nix-command eval --json --impure -f "$nixpkgs" "$1.meta.position" | jq -r . | cut -d: -f1)" | ||
outpath="$(nix --extra-experimental-features nix-command eval --json --impure --expr "builtins.fetchGit \"$nixpkgs\"")" | ||
|
||
if [ -n "$outpath" ]; then | ||
path="${path/$(echo "$outpath" | jq -r .)/$nixpkgs}" | ||
fi | ||
|
||
echo "$path" | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tbh this can just be hardcoded
Description of changes
Add update script (based on existing one I use for
sonic-pi
, which is in a similar situation with a FOD that isn't directly supported by nix-update right now) and bump versionThe only changes in the 1.0.53 -> 1.0.56 bump is supporting a custom app title (which didn't come with release notes it seems...), a command line flag to disable the notification sound (notes), and several bug fixes (notes)
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)