-
-
Notifications
You must be signed in to change notification settings - Fork 14.2k
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
element-call: init at 0.5.16; lk-jwt-service: init at main #300966
Conversation
11d5f1b
to
a7788fe
Compare
+++ b/package.json | ||
@@ -1,4 +1,5 @@ | ||
{ | ||
+ "name": "element-call", |
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.
Please try to upstream this diff
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.
I opened a PR here
''; | ||
|
||
offlineCache = fetchYarnDeps { | ||
yarnLock = "${src}/yarn.lock"; |
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.
That could be triggering IFD, you are sure it does not?
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.
I'm not sure, though this is what the documentation recommends doing
yarnLock = src + "/yarn.lock"; |
I have updated the package and will wait for the CI to build the backages on aarch64 as well as darwin to get the corresponding hashes |
Description of changes
Package the element-call software, as well as the lk-jwt-service. Element Call is a web group calling application, which uses lk-jwt-service as it's authentication service.
As these packages are designed to work side by side, I thought it would make sense, to have only one PR. If necessary I can split those up.
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.