Re: python-social-auth 0.2.19-1 review

2016-06-08 Thread Tiago Ilieve
Hi Dmitry, (Sorry for taking so long to reply.) On 1 June 2016 at 08:40, Dmitry Shachnev wrote: > Usually one would do both things using: > > git-dpm import-new-upstream --pristine-tar-commit /path/to/tarball > > In your case .git-dpm was inconsistent with upstream branch, so I had to > pass t

Re: python-social-auth 0.2.19-1 review

2016-06-01 Thread Dmitry Shachnev
Hi Tiago, On Mon, May 30, 2016 at 03:00:05AM -0300, Tiago Ilieve wrote: > At first I thought this would be hard[1], but following the example > from "python-docutils"[2][3] proved to be quite straightforward. I've > pushed the changes[4], but there are two related things that I didn't > figured ho

Re: python-social-auth 0.2.19-1 review

2016-05-29 Thread Tiago Ilieve
Dmitry and Michael, On 24 May 2016 at 07:08, Tiago Ilieve wrote: > Thinking about this, the best solution seems to be drop the patch and > remove the "site/" folder, repacking it as a DSFG-compatible tarball. > But if there isn't a strictly legal requirement (e.g. to include > sources for files t

Re: python-social-auth 0.2.19-1 review

2016-05-24 Thread Tiago Ilieve
Hi Dmitry, On 21 May 2016 at 05:47, Dmitry Shachnev wrote: > I think it's better to put the missing sources in debian/missing-sources/ > directory rather than patching them in. > > (That is also suggested by the Lintian error description[1], and should > make that error disappear.) The pedantic

Re: python-social-auth 0.2.19-1 review

2016-05-21 Thread Dmitry Shachnev
Hi Tiago, On Thu, May 19, 2016 at 01:19:46PM -0300, Tiago Ilieve wrote: >> - lintian complains about site/js/bootstrap.min.js, AFAIKT the "site" >> folder contains the project's website. Maybe you would like to remove >> it by repacking the source tarball. > > There's a patch[1] adding sources fo

Re: python-social-auth 0.2.19-1 review

2016-05-19 Thread Tiago Ilieve
Hi Michael, On 19 May 2016 at 08:41, Michael Fladischer wrote: > my quick review after building it: > > - lintian complains about site/js/bootstrap.min.js, AFAIKT the "site" > folder contains the project's website. Maybe you would like to remove > it by repacking the source tarball. There's a p

Re: python-social-auth 0.2.19-1 review

2016-05-19 Thread Michael Fladischer
Hi Tiago, On Tue, 2016-05-17 at 02:14 -0300, Tiago Ilieve wrote: > In my first task as a member of the Debian Python Modules Team, I've > prepared an upload of "python-social-auth"[1]. It was updated to a > newer upstream release (0.2.13 to 0.2.19) and some fixes were also > done. > > Is there an

python-social-auth 0.2.19-1 review

2016-05-16 Thread Tiago Ilieve
Hi DPMT, In my first task as a member of the Debian Python Modules Team, I've prepared an upload of "python-social-auth"[1]. It was updated to a newer upstream release (0.2.13 to 0.2.19) and some fixes were also done. Is there anyone here able to review/sponsor those changes? Regards, Tiago. [1