Re: [felix+deb...@gueux.org: Bug#955862: RFS: sphinx-autoapi/1.2.1-1 [ITP]]

2020-04-21 Thread Thomas Goirand
On 4/21/20 11:12 AM, Félix Sipma wrote: > dget -x > https://mentors.debian.net/debian/pool/main/s/sphinx-autoapi/sphinx-autoapi_1.3.0-1.dsc > && sbuild sphinx-autoapi_1.3.0-1.dsc Hi Felix, I believe I was downloading the previous version, 1.2.0-1. Just an advice for lazy sponsors like myself,

Re: [felix+deb...@gueux.org: Bug#955862: RFS: sphinx-autoapi/1.2.1-1 [ITP]]

2020-04-21 Thread Félix Sipma
On 2020-04-20 21:53+0200, Thomas Goirand wrote: Well, I'm just trying to sponsor what you've uploaded to mentors, and so far, it didn't work (I just tried). Please try downloading the .dsc with dget and try yourself. Cheers, Thomas Goirand (zigo) How did you build the package? I just tried

Re: [felix+deb...@gueux.org: Bug#955862: RFS: sphinx-autoapi/1.2.1-1 [ITP]]

2020-04-20 Thread Thomas Goirand
On 4/20/20 4:35 PM, Félix Sipma wrote: > I use `dgit sbuild` to build the package, and the other tests are > actually ran (I just tried again)... Well, I'm just trying to sponsor what you've uploaded to mentors, and so far, it didn't work (I just tried). Please try downloading the .dsc with dget

Re: [felix+deb...@gueux.org: Bug#955862: RFS: sphinx-autoapi/1.2.1-1 [ITP]]

2020-04-20 Thread Félix Sipma
On 2020-04-20 16:16+0200, Thomas Goirand wrote: On 4/20/20 2:45 PM, Félix Sipma wrote: On 2020-04-20 12:06+0200, Thomas Goirand wrote: I'm not sure, the doc still accounts for half of the package. But let's pretend it's ok. We are talking about 80KB for the whole package. If it's not a

Re: [felix+deb...@gueux.org: Bug#955862: RFS: sphinx-autoapi/1.2.1-1 [ITP]]

2020-04-20 Thread Thomas Goirand
On 4/20/20 2:45 PM, Félix Sipma wrote: > On 2020-04-20 12:06+0200, Thomas Goirand wrote: >> I'm not sure, the doc still accounts for half of the package. But let's >> pretend it's ok. > > We are talking about 80KB for the whole package. If it's not a blocker > for you I prefer keeping it like

Re: [felix+deb...@gueux.org: Bug#955862: RFS: sphinx-autoapi/1.2.1-1 [ITP]]

2020-04-20 Thread Félix Sipma
On 2020-04-20 12:06+0200, Thomas Goirand wrote: I'm not sure, the doc still accounts for half of the package. But let's pretend it's ok. We are talking about 80KB for the whole package. If it's not a blocker for you I prefer keeping it like that. Now, the package contains a test suite.

Re: [felix+deb...@gueux.org: Bug#955862: RFS: sphinx-autoapi/1.2.1-1 [ITP]]

2020-04-20 Thread Thomas Goirand
On 4/20/20 11:13 AM, Félix Sipma wrote: > On 2020-04-19 21:50+0200, Thomas Goirand wrote: >> On 4/19/20 5:24 PM, Félix Sipma wrote: >>> I hope I fixed the issues you found >> >> Not really... :( > > Let's try again, then... > >> Now the package is 2.7 MB, out of which the extracted library is

Re: [felix+deb...@gueux.org: Bug#955862: RFS: sphinx-autoapi/1.2.1-1 [ITP]]

2020-04-20 Thread Félix Sipma
On 2020-04-19 23:18+0300, Dmitry Shachnev wrote: Better not symlink anything manually but use dh_sphinxdoc which will do it automatically: - Replace “--with python3” with “--with python3,sphinxdoc” in debian/rules. - Make python-sphinx-autoapi-doc depend on ${sphinxdoc:Depends}. - Drop

Re: [felix+deb...@gueux.org: Bug#955862: RFS: sphinx-autoapi/1.2.1-1 [ITP]]

2020-04-20 Thread Félix Sipma
On 2020-04-19 21:50+0200, Thomas Goirand wrote: On 4/19/20 5:24 PM, Félix Sipma wrote: I hope I fixed the issues you found Not really... :( Let's try again, then... Now the package is 2.7 MB, out of which the extracted library is only 380 KB. Now, the extracted documentation is 3.5 MB and

Re: [felix+deb...@gueux.org: Bug#955862: RFS: sphinx-autoapi/1.2.1-1 [ITP]]

2020-04-19 Thread Dmitry Shachnev
Hi all, On Sun, Apr 19, 2020 at 09:50:34PM +0200, Thomas Goirand wrote: > Now the package is 2.7 MB, out of which the extracted library is only > 380 KB. Now, the extracted documentation is 3.5 MB and contains 3 MB of > fonts, like fontawesome and lato, which are already part of Debian. > > If

Re: [felix+deb...@gueux.org: Bug#955862: RFS: sphinx-autoapi/1.2.1-1 [ITP]]

2020-04-19 Thread Thomas Goirand
On 4/19/20 5:24 PM, Félix Sipma wrote: > I hope I fixed the issues you found Not really... :( Now the package is 2.7 MB, out of which the extracted library is only 380 KB. Now, the extracted documentation is 3.5 MB and contains 3 MB of fonts, like fontawesome and lato, which are already part of

Re: [felix+deb...@gueux.org: Bug#955862: RFS: sphinx-autoapi/1.2.1-1 [ITP]]

2020-04-19 Thread Félix Sipma
On 2020-04-19 15:39+, Scott Kitterman wrote: As a member of the FTP Team, I see this situation from time to time. It's usually not a serious problem, but I agree with Zigo on this. Where it becomes problematic is when code in debian/ becomes intermingled with the installed upstream code.

Re: [felix+deb...@gueux.org: Bug#955862: RFS: sphinx-autoapi/1.2.1-1 [ITP]]

2020-04-19 Thread Scott Kitterman
On April 19, 2020 3:24:17 PM UTC, "Félix Sipma" wrote: >Hi Thomas, > >Thanks for the review, and it's nice to see you found a number of >problems! I have to admit I did not prepare a new package since a long >time, probably forgot a lot of things, and copied others from other >packages of

Re: [felix+deb...@gueux.org: Bug#955862: RFS: sphinx-autoapi/1.2.1-1 [ITP]]

2020-04-19 Thread Félix Sipma
Hi Thomas, Thanks for the review, and it's nice to see you found a number of problems! I have to admit I did not prepare a new package since a long time, probably forgot a lot of things, and copied others from other packages of mine... On 2020-04-19 14:09+0200, Thomas Goirand wrote: Hi

Re: [felix+deb...@gueux.org: Bug#955862: RFS: sphinx-autoapi/1.2.1-1 [ITP]]

2020-04-19 Thread Thomas Goirand
Hi Felix, Thanks for working on this. Here's my comments. I'm sorry that there's a lot to say on what you did... :/ On 4/19/20 11:53 AM, Félix Sipma wrote: > dget -x > https://mentors.debian.net/debian/pool/main/s/sphinx-autoapi/sphinx-autoapi_1.2.1-1.dsc Looking at your

[felix+deb...@gueux.org: Bug#955862: RFS: sphinx-autoapi/1.2.1-1 [ITP]]

2020-04-19 Thread Félix Sipma
Hi, Nobody answered on mentors, so I'm trying debian-python... If someone could have a look to this package and push it to the NEW queue, it would allow me to get a new version of khard in sid. Thanks, -- Félix --- Begin Message --- Package: sponsorship-requests Severity: normal Dear