Bug#954362: ncmpc documentation should be split out

2020-04-07 Thread Daniel Kahn Gillmor
Hi Kaliko--

Thanks for your thoughtful and detailed reply!

On Tue 2020-04-07 16:18:57 +0200, kaliko wrote:
> Here are my propositions.
>
>  * Keep ncmpc-lyrics a separated package
>
> Either
>   1) Move ${sphinxdoc:Depends} to Recommends (as suggested by Daniel)
>   2) Disable HTML build altogether ("html_manual=false" build option)
>
> I'm in favor of 2). Actually the html documentation is nothing more than the 
> html
> version of the man page. As Daniel mentioned ncmpc targets lightweight 
> (non-graphical)
> environment (and advanced users), the regular manual is enough IMHO.

This all looks reasonable to me.  I'm assuming that if you go with (2)
you'd drop sphinxdoc:Depends entirely, reducing the footprint of the
dependencies even further.

Many thanks for maintaining ncmpc in debian.  I've been using it for
years and it Just Works™ -- i haven't had to worry about it or fiddle
with it or anything.  Very much appreciated to have something nice to
use, simple and functional in easy reach!

   --dkg


signature.asc
Description: PGP signature


Bug#954362: ncmpc documentation should be split out

2020-04-07 Thread kaliko
Hi

Thanks for your report Daniel,

20/03/2020 , Daniel Kahn Gillmor wrote:
> ncmpc pulls in a bunch of javascript packages, which i think are just to
> render the documentation:

Right.

20/03/2020 , Daniel Kahn Gillmor wrote:
> Since ncmpc is good for use in a lightweight (non-graphical)
> environment […]
> I recommend either moving ncmpc documentation to a separate package, or
> making these dependencies into Recommends instead.

Here is a bit of context regarding documentation as a separated package.

When we took over maintenance I did split the documentation in a dedicated 
package. But
in order to have the package in time for Buster release we decided not to split 
the
package. Here is the original message :

06/01/2019 , Florian Schlichting wrote:
> - building a new binary package ncmpc-doc will cause the upload to go to
>   NEW for ftp-master approval. This may take several weeks and carries a
>   high risk of missing buster. My advice is to not split off the
>   documentation - not now because of release timing, but also more
>   generally because it's not actually that big (123k installed). In fact
>   I think the ncmpc-lyrics package should be re-integrated with ncmpc
>   (the additional changelog.Debian.gz is twice the size of all the
>   plugins together) but I suggest to do this after the buster release.

The reason for shipping documentation (and lyrics plugin as well) in a 
separated package
is not only a matter of size but also the extra dependencies it pulls.

ncmpc-doc (html only)  168k
libjs-sphinxdoc144k
   ↘ libjs-jquery  746k
   ↘ javascript-common  70k
   ↘ libjs-underscore  302k
TOTAL 1400k

In case of ncmpc-lyrics it's worse since it pulls the ruby interpreter (15M for 
libruby
alone).

Here are my propositions.

 * Keep ncmpc-lyrics a separated package

Either
  1) Move ${sphinxdoc:Depends} to Recommends (as suggested by Daniel)
  2) Disable HTML build altogether ("html_manual=false" build option)

I'm in favor of 2). Actually the html documentation is nothing more than the 
html
version of the man page. As Daniel mentioned ncmpc targets lightweight 
(non-graphical)
environment (and advanced users), the regular manual is enough IMHO.

I'll update the package in a couple of week.

Please comment if you think we should do otherwise or if I forgot something.

Cheers
k



signature.asc
Description: OpenPGP digital signature


Bug#954362: ncmpc documentation should be split out

2020-03-20 Thread Daniel Kahn Gillmor
Package: ncmpc
Version: 0.35-1

ncmpc pulls in a bunch of javascript packages, which i think are just to
render the documentation:

  javascript-common libjs-jquery libjs-sphinxdoc libjs-underscore

Since ncmpc is good for use in a lightweight (non-graphical)
environment, it doesn't mak a lot of sense to pull in these other
packages.

I recommend either moving ncmpc documentation to a separate package, or
making these dependencies into Recommends instead.

Thanks for maintaining ncmpc in debian!

   --dkg


signature.asc
Description: PGP signature