bug#36213: [PATCH 4/4] gnu: Add pulseaudio-dlna.

[Ludovic Courtès]

Hi,

"Alex Griffin" <address@hidden> skribis:

> On Tue, Jul 2, 2019, at 3:54 PM, Ludovic Courtès wrote:
>> > +(define-public pulseaudio-dlna
>> > +  (let ((commit "4472928dd23f274193f14289f59daec411023ab0")
>> > +        (revision "1"))
>> 
>> Could you explain why you chose this commit rather than the latest
>> release?
>
> Honestly, I just noticed that Debian was building from git. I assumed they 
> had a good reason, so I didn't even try the latest release from 2016.

OK.  There have been quite a few commits since that 2016 release, so it
probably makes sense to use the latest one.

>> > +      (arguments `(#:python ,python-2))
>> 
>> Can it run on Python 3?
>> 
>> Since Python 2 is approaching EOL, we’re trying to avoid adding more
>> Python 2 packages.  If we could remove ‘python2-’ packages from this
>> patch series, that’d be great; if not, so be it!
>
> No, it doesn't run on Python 3. It's actually unmaintained right now, but I'm 
> not aware of any other free software projects that provide the same 
> functionality.

OK.

I took the liberty to add some of what you wrote as comments in the
code and pushed the whole series.

Thanks!

Ludo’.