matrix-synapse: init at 0.12.0

[roblabla]

Adds matrix.org's synapse server as a package and module.

I am currently using this package on my server without any problems.

[mention-bot]

By analyzing the blame information on this pull request, we identified @edolstra, @joachifm and @offlinehacker to be potential reviewers

[lucabrunox]

That's a huge addition, thanks!

Please check the travis error, daemonize doesn't build. Make sure you have enabled nix chroots. Also note that the uid 219 is no more available.

[roblabla]

I changed the uid to 223, and fixed a build error concerning pydenticon missing mock (caught that by enabling chroot builds).

Now I'm having a small problem. The sha256 returned by fetchgit seems to have changed from under my feets, even though the tag still points to the same commit. Any idea how this could happen ? https://github.com/roblabla/nixpkgs/blob/feature-initMatrixSynapse/pkgs/servers/matrix-synapse/default.nix#L17-L21

Yesterday : 29f3d95c0a5a0c902a1c37357dea1a9150d3177d17f1045e27a9d01692816af9
Today : 54ffa04166660fe59f475db04cfccc635fa299f5867312db7982229a6fe2f9f0

Is the reason it breaks because I'm using tag refs instead of a real commit hash ?

[lucabrunox]

@roblabla is fetchgit necessary? Use fetchFromGitHub if possible.

[roblabla]

Fixed, it's using fetchFromGitHub now. Also fixed a failure in pynacl due to missing pytest for the tests to pass. Hopefully, everything should pass now (I didn't catch any problem with daemonize locally).

[FRidh]

Unless you tested with Nox what the effects are of this upgrade I suggest you add atwisted_15 instead and use that. We can then later see what other packages can use this latest twisted. The same with simplejson since that's also a common package. Unless of course you run into collisions.

[roblabla]

@FRidh I updated twisted separately as suggested. Major bump, so that makes sense.

For simplejson, I went through the changelog and I don't see how any of the changes could break existing applications. They are either new API additions or bugfixes to edge-cases. Is it really okay to create a new package for a minor version bump ?

I also rebased branch on current master, the ids changed again.

[FRidh]

@roblabla nox-review wip or nox-review pr 12301 will tell whether a separate package might be needed or not.

[roblabla]

I get an error running nox-review (both commands) : https://gist.github.com/10f9638a07d367b1f3a0

[lucabrunox]

The travis error is related to pypy that is unable to compile, right?

[roblabla]

Seems like blist doesn't support pypy : DanielStutzbach/blist#68

I added disabled = isPyPy; to the package.

[lucabrunox]

Built locally, thanks.