unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
From: jgart via Guix-patches via <guix-patches@gnu.org>
To: 47104@debbugs.gnu.org
Cc: raghavgururajan@disroot.org, Raghav Gururajan <rg@raghavgururajan.name>
Subject: [bug#47104] grumble status update
Date: Sun, 18 Apr 2021 17:25:06 +0000	[thread overview]
Message-ID: <e01f8d02625a4fb4a1a1d8d7daa3afef@dismail.de> (raw)
In-Reply-To: <cd89e2168da413f579b889f5d537e6d3@dismail.de>

Hi Leo,

> I know you mean this somewhat jokingly, but is there anything (apart
> maybe from the name of the binary), that would keep you from reusing
> murmur-service-type?

See here: 

https://github.com/mumble-voip/grumble/issues/21
https://github.com/mumble-voip/grumble/pull/26

There are more sources related to the grumble config that's currently implemented that I can't locate at the moment. 

I remember reading that they didn't necessarily want to maintain feature parity with the grumble config format.

> 1. Is this package in its current state usable?

I would say yes. We packaged grumble while talking over grumble. It feels pretty solid. 

Grumble also has an active fork as a library and being used by wahay: https://wahay.org/

It is currently 16 commits ahead of upstream:

https://github.com/digitalautonomy/grumble

> 2. Is it still maintained upstream?  It is a little stretch to say
> Grumble is undergoing active development after a year of no activity. 

It sounds like the project maintainers of the upstream grumble project are very slow to review pull requests. It sounds like they are too busy with other projects/work.

See the complaint here by one of the contributors that chimed in when I opened an issue:

https://github.com/mumble-voip/grumble/issues/76

> 3. https://github.com/mumble-voip/grumble#project-statuslists quite a
> few features that are lacking, but does it maybe contain features, that
> would make it worth packaging?

See https://github.com/mumble-voip/grumble/issues/76

"... Grumble has the distinguishing feature of native support for Websockets (because I was a lot worse at C++ back then and so I contributed a patch here instead), and Murmur will probably not have that for the foreseeable future. You could of course just configure a proxy in front of Murmur if you need this. A lot of the plans for work we were making a few years ago pointed towards Grumble being more focused on ease-of-use and these small workloads I talked about above. It makes sense: the Murmur static binary has issues and so a Grumble static (just how Go works) binary that you can download and run, trivially configure and easily negotiate certs over LE (unfortunately never happened due to LE issues, but it would be viable now), accessible over the Web could fulfil a sort of "batteries-included" user-friendly niche."

> If the answer is "no" to any of the above, I'm not too sure whether it
> would be wise to have this in Guix upstream.  If LibreMiami wanted to
> host grumble instances on Guix regardless, perhaps a channel might be a
> better fit?

We can put this in a LibreMiami channel with a service for it if you insist it not be included in upstream guix. 

If upstream grumble picks up development then I can send a patch again for review.

That said, can you take the patch for go-github-com-gorilla-websocket?

We will need go-github-com-gorilla-websocket for many other packages that we're working on. One of them being the hugo static site generator that we're working on with Ryan Prior.

Relatedly, we're planning on packaging wahay (https://wahay.org/). 

wahay depends on the fork of grumble that I linked above. 

Should we package only the fork of grumble in that case and not upstream grumble?

all the best,

jgart




  parent reply	other threads:[~2021-04-18 17:26 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-03-12 16:25 [bug#47104] [PATCH 1/2] gnu: Add grumble jgart via Guix-patches via
2021-03-12 20:37 ` Leo Prikler
2021-04-18 17:25 ` jgart via Guix-patches via [this message]
2021-04-18 18:31   ` [bug#47104] grumble status update Leo Prikler
2021-04-18 19:16   ` jgart via Guix-patches via
2022-06-22 18:51     ` bug#47104: [PATCH 1/2] gnu: Add grumble Maxim Cournoyer

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=e01f8d02625a4fb4a1a1d8d7daa3afef@dismail.de \
    --to=guix-patches@gnu.org \
    --cc=47104@debbugs.gnu.org \
    --cc=jgart@dismail.de \
    --cc=raghavgururajan@disroot.org \
    --cc=rg@raghavgururajan.name \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).