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

matterbridge, modules/matterbridge: init at 1.1.0 #28892

Merged
merged 1 commit into from Sep 16, 2017

Conversation

ryantm
Copy link
Member

@ryantm ryantm commented Sep 2, 2017

This adds the matterbridge service to nixos and nixpkgs https://github.com/42wim/matterbridge

@@ -297,6 +297,7 @@
clickhouse = 278;
rslsync = 279;
minio = 280;
matterbridge = 281;
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is a dynamic uid insufficient?

@ryantm
Copy link
Member Author

ryantm commented Sep 10, 2017

@joachifm You're right; I think I just need the user to exist not be a particular constant ID.

@joachifm joachifm merged commit c0616a3 into NixOS:master Sep 16, 2017
@joachifm
Copy link
Contributor

Thank you. Note that, ideally, package additions and module additions should be in separate commits.

@joachifm
Copy link
Contributor

@ryantm the package expression appears to be broken: https://hydra.nixos.org/build/61154969

@orivej
Copy link
Contributor

orivej commented Sep 17, 2017

I have built it successfully a few days ago, but then I've collected garbage. @ryantm, if you still have an archive that hashes to the old hash, it would be interesting to see.

@ryantm
Copy link
Member Author

ryantm commented Sep 18, 2017

@orivej It looks like it is some issue with the file meta data being different:

ryantm@nixos-dev ~/p/nixpkgs$ ls -lah /nix/store/*v1.1.0*
-r--r--r-- 1 root    root   2.3K Dec 31  1969 /nix/store/an68r6n3x7wpdygszdmkp1ic76fpx3bn-v1.1.0.tar.gz.drv
-r--r--r-- 1 root    root   2.3K Dec 31  1969 /nix/store/f78n0ivgzbps4hwl41h4ri7fvcns0bcn-v1.1.0.tar.gz.drv
-r--r--r-- 1 root    root   4.8M Dec 31  1969 /nix/store/h7h6mkh11945a7j0xypmw106fya6h39q-v1.1.0.tar.gz
-rw-r--r-- 1 nixbld1 nixbld 4.8M Sep  1 16:03 /nix/store/xc2w0fz8wmpgsa5b8qf6zsmli7pshd0l-v1.1.0.tar.gz
-rw------- 1 root    root      0 Sep  1 16:03 /nix/store/xc2w0fz8wmpgsa5b8qf6zsmli7pshd0l-v1.1.0.tar.gz.lock
ryantm@nixos-dev ~/p/nixpkgs$ sha256sum /nix/store/h7h6mkh11945a7j0xypmw106fya6h39q-v1.1.0.tar.gz
73ec9fed6cbb93b3aa34b54b179604103d36ab69dcdcafd09cb749cf9348c35a  /nix/store/h7h6mkh11945a7j0xypmw106fya6h39q-v1.1.0.tar.gz
ryantm@nixos-dev ~/p/nixpkgs$ sha256sum  /nix/store/xc2w0fz8wmpgsa5b8qf6zsmli7pshd0l-v1.1.0.tar.gz
73ec9fed6cbb93b3aa34b54b179604103d36ab69dcdcafd09cb749cf9348c35a  /nix/store/xc2w0fz8wmpgsa5b8qf6zsmli7pshd0l-v1.1.0.tar.gz

I'll make a PR to fix the matterbridge.

Ryan

ryantm added a commit to ryantm/nixpkgs that referenced this pull request Sep 18, 2017
I used the wrong source hash because of some kind of metadata problem.
See NixOS#28892 (comment)
for details.
fpletz pushed a commit that referenced this pull request Sep 18, 2017
I used the wrong source hash because of some kind of metadata problem.
See #28892 (comment)
for details.
fpletz pushed a commit that referenced this pull request Sep 19, 2017
I used the wrong source hash because of some kind of metadata problem.
See #28892 (comment)
for details.

(cherry picked from commit 20f9493)
@ryantm ryantm deleted the matterbridge2 branch November 17, 2017 01:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants