Bug#847458: RFS: clues-emacs/0~2014.09.23.69d873c-1 ITP

2016-12-19 Thread Sean Whitton
On Fri, Dec 16, 2016 at 05:40:05AM +0300, Dmitry Bogatov wrote: > I upgraded to next upstream version (purely cosmetic) 1.0.1 and made dch -r. 24b068ac68848a73bd7c45337057ea5668ad9af1 LGTM. -- Sean Whitton signature.asc Description: PGP signature

Bug#847458: RFS: clues-emacs/0~2014.09.23.69d873c-1 ITP

2016-12-15 Thread Dmitry Bogatov
[2016-12-14 15:51] Sean Whitton > > part 1 text/plain 441 > On Wed, Dec 14, 2016 at 05:06:39AM +0300, Dmitry Bogatov wrote: > > I received reply from upstream, so now version is oficially 1.0.0. One > > more review, please. > > You need `dch -r`.

Bug#847458: RFS: clues-emacs/0~2014.09.23.69d873c-1 ITP

2016-12-14 Thread Sean Whitton
On Wed, Dec 14, 2016 at 05:06:39AM +0300, Dmitry Bogatov wrote: > I received reply from upstream, so now version is oficially 1.0.0. One > more review, please. You need `dch -r`. Otherwise, cab58bc0530e73165bf01d66a37966d08ca26280 LGTM. > About d/watch -- seems I used old version of

Bug#847458: RFS: clues-emacs/0~2014.09.23.69d873c-1 ITP

2016-12-13 Thread Dmitry Bogatov
I received reply from upstream, so now version is oficially 1.0.0. One more review, please. About d/watch -- seems I used old version of dh-make-elpa. Latest version correctly creates d/watch with https:// links in both modes (with and without --pkg-emacsen). -- X-Web-Site:

Bug#847458: RFS: clues-emacs/0~2014.09.23.69d873c-1 ITP

2016-12-11 Thread Sean Whitton
Hello Dmitry, On Sun, Dec 11, 2016 at 08:03:37AM +0300, Dmitry Bogatov wrote: > > [2016-12-10 09:15] Sean Whitton > > For normal packages, dh-elpa autoloads functions, so you don't need to > > call (package-initialize). However, AIUI, you can't autoload themes. > > So

Bug#847458: RFS: clues-emacs/0~2014.09.23.69d873c-1 ITP

2016-12-10 Thread Sean Whitton
control: tag -1 +confirmed control: noowner -1 f95bd4552f6c9922754ac72deec139ab70fa5d28 ready for upload. On Sat, Dec 10, 2016 at 12:11:29PM +0300, Dmitry Bogatov wrote: > I did not checked, that (package-initialize) is really needed, but I > assume you are right. > > But probably it should be

Bug#847458: RFS: clues-emacs/0~2014.09.23.69d873c-1 ITP

2016-12-10 Thread Dmitry Bogatov
control: tag -1 -moreinfo [2016-12-08 11:23] Sean Whitton > Here's a review for you. I also made two commits to your repository -- > please see the commit messages for an explanation. Hopefully you will > agree with the changes ;) I did not checked, that

Bug#847458: RFS: clues-emacs/0~2014.09.23.69d873c-1 ITP

2016-12-08 Thread Sean Whitton
control: owner -1 ! control: tag -1 +moreinfo Hello Dmitry, Here's a review for you. I also made two commits to your repository -- please see the commit messages for an explanation. Hopefully you will agree with the changes ;) Must-fixes == 1. Your most recent changes are newer than

Bug#847458: RFS: clues-emacs/0~2014.09.23.69d873c-1 ITP

2016-12-08 Thread Dmitry Bogatov
Package: sponsorship-requests Severity: wishlist Dear mentors, I am looking for a sponsor for my package "clues-emacs" * Package name: clues-emacs Version : 0~2014.09.23.69d873c-1 Upstream Author : Jason Milkins * Url :