Bug#840424: RFS: verilog-mode

2016-12-09 Thread Sean Whitton
Hello Kiwamu, On Fri, Dec 09, 2016 at 10:51:05PM +0900, Kiwamu Okabe wrote: > OK. I think I should learn "pristine-tar" before the next day. Or you can just share the tarball via any web server. I would suggest reading dgit-sponsorship(7), a tutorial manpage (apt-get install dgit). -- Sean

Bug#840424: RFS: verilog-mode

2016-12-09 Thread Sean Whitton
Thanks for fixing this, Gianfranco. Fwiw it build in sbuild on my machine. -- Sean Whitton signature.asc Description: PGP signature

Bug#840424: RFS: verilog-mode

2016-12-07 Thread Kiwamu Okabe
Hi Gianfranco, On Thu, Dec 8, 2016 at 11:10 AM, Sean Whitton wrote: > You don't need to: just provide Gianfranco with a tarball to upload, or > tell him how to produce it from your git repository. Thanks for your advice. Could you upload my package with following? >

Bug#840424: RFS: verilog-mode

2016-12-07 Thread Sean Whitton
Dear Kiwamu, On Thu, Dec 08, 2016 at 10:22:00AM +0900, Kiwamu Okabe wrote: > Could you advice me how to use the command in this situation? You don't need to: just provide Gianfranco with a tarball to upload, or tell him how to produce it from your git repository. -- Sean Whitton

Bug#840424: RFS: verilog-mode

2016-12-07 Thread Kiwamu Okabe
Hi all, On Thu, Dec 8, 2016 at 9:07 AM, Sean Whitton wrote: >> verilog-mode (master) $ gbp buildpackage -S -d >> gbp:warning: Pristine-tar branch "pristine-tar" not found >> gbp:error: Pristine-tar couldn't checkout >> "verilog-mode_20161124.fd230e6.orig.tar.gz":

Bug#840424: RFS: verilog-mode

2016-12-07 Thread Sean Whitton
Hello, On Wed, Dec 07, 2016 at 11:46:19PM +, Gianfranco Costamagna wrote: > verilog-mode (master) $ gbp buildpackage -S -d > gbp:warning: Pristine-tar branch "pristine-tar" not found > gbp:error: Pristine-tar couldn't checkout > "verilog-mode_20161124.fd230e6.orig.tar.gz": pristine-tar: no

Bug#840424: RFS: verilog-mode

2016-12-07 Thread Gianfranco Costamagna
control: owner -1 ! control: tags -1 moreinfo Hi >I consider git revision 4ba937fbc958fb5037e65fe351d18f2da8e0b988 ready >for upload. > >Thank you for your patience, Kiwamu. tarball? verilog-mode (master) $ gbp buildpackage -S -d gbp:warning: Pristine-tar branch "pristine-tar" not found

Bug#840424: RFS: verilog-mode

2016-12-07 Thread Sean Whitton
control: tag -1 -moreinfo +confirmed control: noowner -1 I consider git revision 4ba937fbc958fb5037e65fe351d18f2da8e0b988 ready for upload. Thank you for your patience, Kiwamu. -- Sean Whitton signature.asc Description: PGP signature

Bug#840424: RFS: verilog-mode

2016-12-07 Thread Kiwamu Okabe
Dear Sean, On Thu, Dec 8, 2016 at 12:42 AM, Sean Whitton wrote: >> I think "Forwarded: no" is valid, because >> http://dep.debian.net/deps/dep3/ say following: > > Right, but the convention is that "no" means "I should forward this but > I haven't forwarded it yet" and

Bug#840424: RFS: verilog-mode

2016-12-07 Thread Sean Whitton
Dear Kiwamu, On Wed, Dec 07, 2016 at 01:48:37PM +0900, Kiwamu Okabe wrote: > On Wed, Dec 7, 2016 at 1:14 PM, Sean Whitton wrote: > > Looks good, but you need "Forwarded: not-needed" not "Forwarded: no". > > I think "Forwarded: no" is valid, because >

Bug#840424: RFS: verilog-mode

2016-12-06 Thread Kiwamu Okabe
Dear Sean, On Wed, Dec 7, 2016 at 1:14 PM, Sean Whitton wrote: > Looks good, but you need "Forwarded: not-needed" not "Forwarded: no". I think "Forwarded: no" is valid, because http://dep.debian.net/deps/dep3/ say following: ``` Forwarded (optional) Any value other

Bug#840424: RFS: verilog-mode

2016-12-06 Thread Sean Whitton
Dear Kiwamu, On Wed, Dec 07, 2016 at 12:45:08PM +0900, Kiwamu Okabe wrote: > Could you review it? Looks good, but you need "Forwarded: not-needed" not "Forwarded: no". -- Sean Whitton signature.asc Description: PGP signature

Bug#840424: RFS: verilog-mode

2016-12-06 Thread Kiwamu Okabe
Dear Sean, On Wed, Dec 7, 2016 at 11:47 AM, Sean Whitton wrote: >> > 2. You should patch the "New versions" and "Installation" sections out >> > of the texinfo file, as these could be confusing to Debian users who >> > already have the package installed and will obtain

Bug#840424: RFS: verilog-mode

2016-12-06 Thread Sean Whitton
Dear Kiwamu, On Wed, Dec 07, 2016 at 11:01:46AM +0900, Kiwamu Okabe wrote: > > 2. You should patch the "New versions" and "Installation" sections out > > of the texinfo file, as these could be confusing to Debian users who > > already have the package installed and will obtain new versions from >

Bug#840424: RFS: verilog-mode

2016-12-06 Thread Kiwamu Okabe
Hi Sean, On Wed, Dec 7, 2016 at 8:47 AM, Sean Whitton wrote: > 1. I think you should tweak the copyright for Makefile. You wrote > > Copyright: 2008-2013, Michael McNamara > 2008-2013, Wilson Snyder > > but I think the following is more accurate: > >

Bug#840424: RFS: verilog-mode

2016-12-06 Thread Sean Whitton
Dear Kiwamu, On Tue, Dec 06, 2016 at 01:53:25PM +0900, Kiwamu Okabe wrote: > Should I find a Debian Developer in > pkg-emacsen-add...@lists.alioth.debian.org to dput the verilog-mode > package? Sorry, but I have a few more items... 1. I think you should tweak the copyright for Makefile. You

Bug#840424: RFS: verilog-mode

2016-12-05 Thread Kiwamu Okabe
Dear Sean, On Tue, Dec 6, 2016 at 3:09 AM, Sean Whitton wrote: > No, it's not Lintian's mistake. > > You have: > > Files: 0test.el, batch_prof.el, batch_test.el, batch_test.pl > > You should have: > > Files: 0test.el batch_prof.el batch_test.el batch_test.pl

Bug#840424: RFS: verilog-mode

2016-12-05 Thread Sean Whitton
Dear Kiwamu, On Tue, Dec 06, 2016 at 12:11:46AM +0900, Kiwamu Okabe wrote: > And also some lintian errors in debian/copyright are fixed. But I can't fix > following a lintian error. Is it a issue on the side of lintian > command? No, it's not Lintian's mistake. You have: Files: 0test.el,

Bug#840424: RFS: verilog-mode

2016-12-05 Thread Kiwamu Okabe
Dear Sean, On Sun, Dec 4, 2016 at 7:47 AM, Sean Whitton wrote: >> > 1. The line "Only support emacs and xemacs" doesn't make sense (what >> > else would you be supporting?). What were you trying to say? >> >> Fixed. It's my simple mistake. I should say "Support emacs

Bug#840424: RFS: verilog-mode

2016-12-03 Thread Sean Whitton
Dear Kiwamu, On Fri, Nov 25, 2016 at 09:41:16PM +0900, Kiwamu Okabe wrote: > On Sun, Nov 20, 2016 at 5:15 AM, Sean Whitton > wrote: > > Thank you for your work to bring this new package to Debian! As I said, > > I can't sponsor the upload, but I hope this more

Bug#840424: RFS: verilog-mode

2016-11-25 Thread Kiwamu Okabe
Dear Sean, On Sun, Nov 20, 2016 at 5:15 AM, Sean Whitton wrote: > Thank you for your work to bring this new package to Debian! As I said, > I can't sponsor the upload, but I hope this more detailed review is > useful to you. Of course, your precisive review is very

Bug#840424: RFS: verilog-mode

2016-11-19 Thread Sean Whitton
control: retitle -1 RFS: verilog-mode/20160910.debfc6d-1 [ITP] Dear Kiwamu, Thank you for your work to bring this new package to Debian! As I said, I can't sponsor the upload, but I hope this more detailed review is useful to you. I've split it into two sections: things that I would consider

Bug#840424: RFS: verilog-mode/0:20160910.debfc6d.vpo-1

2016-11-13 Thread Kiwamu Okabe
Dear Sean, On Mon, Nov 14, 2016 at 9:38 AM, Kiwamu Okabe wrote: > I would like to support xemacs with verilog-mode, in the future. > However xemacs-mule package is not installable, today. Indeed, > http://www.xemacs.org seems like shutdown... > > I think I should use dh_elpa

Bug#840424: RFS: verilog-mode/0:20160910.debfc6d.vpo-1

2016-11-13 Thread Sean Whitton
control: owner -1 ! Dear Kiwamu, On Wed, Nov 09, 2016 at 07:42:56PM +0900, Kiwamu Okabe wrote: > Could you again review my package? It says in the changelog that you are not supporting xemacs anymore. This was the reason for not using dh_elpa. Is this a temporary problem? Will you be able to

Bug#840424: RFS: verilog-mode/0:20160910.debfc6d.vpo-1

2016-10-26 Thread Sean Whitton
Dear Kiwamu, On Sun, Oct 23, 2016 at 09:54:39PM +0900, Kiwamu Okabe wrote: > $ git log | head -1 > commit 0c1dd008d6813ad90c74d1f83d6c728d219cecaf I haven't fully reviewed the package yet because it fails to build. Please see the attached build log. Also, I see several problems that would be

Bug#840424: RFS: verilog-mode/0:20160910.debfc6d.vpo-1

2016-10-23 Thread Sean Whitton
Dear Kiwamu, On Sun, Oct 23, 2016 at 09:54:39PM +0900, Kiwamu Okabe wrote: > Hi Sean, > > I think that the verilog-mode package is ready to upload. > Could you check and dput it? I'm not a Debian Developer, so I can't upload it, but I'll try to review it for you at some point soon. -- Sean

Bug#840424: RFS: verilog-mode/0:20160910.debfc6d.vpo-1

2016-10-23 Thread Kiwamu Okabe
Hi Sean, I think that the verilog-mode package is ready to upload. Could you check and dput it? $ pwd /home/kiwamu/deb/verilog-mode $ cat .git/config | grep url | tail -1 url = ssh://kiwamu-gu...@git.debian.org/git/pkg-emacsen/pkg/verilog-mode.git $ git log | head -1 commit

Bug#840424: RFS: verilog-mode/0:20160910.debfc6d.vpo-1

2016-10-22 Thread Kiwamu Okabe
Hi Sean, On Mon, Oct 17, 2016 at 11:32 PM, Sean Whitton wrote: > As the manpage says: > >> Certain Debian upstream version strings cannot be translated into >> version strings Emacs will accept (see the docstring for the Emacs >> function `version-to-list' for details).

Bug#840424: RFS: verilog-mode/0:20160910.debfc6d.vpo-1

2016-10-17 Thread Sean Whitton
Hello Kiwamu, On Mon, Oct 17, 2016 at 09:07:51AM +0900, Kiwamu Okabe wrote: > E: The Debian version 20160910.debfc6d.vpo cannot be used as an ELPA > version. > See dh_elpa(1) HINTS for how to deal with this. > debian/rules:7: recipe for target 'binary' failed > > How to fix it? As the

Bug#840424: RFS: verilog-mode/0:20160910.debfc6d.vpo-1

2016-10-17 Thread Kiwamu Okabe
Hi Sean, On Mon, Oct 17, 2016 at 10:59 PM, Sean Whitton wrote: > It sounds like you can just add this to d/rules: > > override_dh_auto_install > /bin/true I try to do that: ``` $ git diff | cat diff --git a/debian/rules b/debian/rules index

Bug#840424: RFS: verilog-mode/0:20160910.debfc6d.vpo-1

2016-10-17 Thread Sean Whitton
Hello Kiwamu, It sounds like you can just add this to d/rules: override_dh_auto_install /bin/true -- Sean Whitton

Bug#840424: RFS: verilog-mode/0:20160910.debfc6d.vpo-1

2016-10-16 Thread Kiwamu Okabe
On Sun, Oct 16, 2016 at 1:56 PM, Kiwamu Okabe wrote: > On Sun, Oct 16, 2016 at 1:53 PM, Dmitry Bogatov wrote: >> In general with debhelper you do following in your d/rules: >> >> override_dh_auto_install: >> do-stuff > > Yes. You are

Bug#840424: RFS: verilog-mode/0:20160910.debfc6d.vpo-1

2016-10-15 Thread Dmitry Bogatov
> The "dh_binary" tries to run "make install", but the make target is > for upstream author's release process and not for installing elisps > into destdir. I would like to override "dh_auto_install" and run some > process under "debian/elpa" settings. How to do that? In general with debhelper

Bug#840424: RFS: verilog-mode/0:20160910.debfc6d.vpo-1

2016-10-15 Thread Kiwamu Okabe
On Sun, Oct 16, 2016 at 1:53 PM, Dmitry Bogatov wrote: > In general with debhelper you do following in your d/rules: > > override_dh_auto_install: > do-stuff Yes. You are right. However, how to manage "debian/elpa" file at "override_dh_auto_install"

Bug#840424: RFS: verilog-mode/0:20160910.debfc6d.vpo-1

2016-10-15 Thread Kiwamu Okabe
Hi all, How to install elisp files of verilog-mode by "dh_elpa" style? If you build my repo now, you'll see following: $ pwd /home/kiwamu/deb/verilog-mode $ git log | head -1 commit 0321d9aeb34ff6e20bcb586b820653d30c92b031 $ cat debian/elpa e/*.el $ gbp buildpackage -us -uc

Bug#840424: RFS: verilog-mode/0:20160910.debfc6d.vpo-1

2016-10-13 Thread Kiwamu Okabe
On Fri, Oct 14, 2016 at 12:54 PM, Kiwamu Okabe wrote: > However, I can't see following web page: > > https://anonscm.debian.org/cgit/pkg-emacsen/pkg/verilog-mode.git/ I see the page, now! I think I should simply wait someone finding that. Thanks, -- Kiwamu Okabe at

Bug#840424: RFS: verilog-mode/0:20160910.debfc6d.vpo-1

2016-10-13 Thread Kiwamu Okabe
Hi Sean, I simply move the verilog-mode.git repo as following: $ slogin kiwamu-gu...@git.debian.org git.debian.org$ set -u git.debian.org$ umask 002 git.debian.org$ cd /git/pkg-emacsen/pkg git.debian.org$ mv /git/collab-maint/verilog-mode.git . Now I can clone the code from git repo: $ git

Bug#840424: RFS: verilog-mode/0:20160910.debfc6d.vpo-1

2016-10-11 Thread Kiwamu Okabe
Hi Sean, On Tue, Oct 11, 2016 at 10:56 PM, Sean Whitton wrote: > I'd like to ask you to package this addon using the new dh-elpa > infrastructure. The pkg-emacsen team are slowly working to convert all > existing Emacs Lisp addon packages, such as major modes, to use

Bug#840424: RFS: verilog-mode/0:20160910.debfc6d.vpo-1

2016-10-11 Thread Sean Whitton
control: tag -1 +moreinfo Dear Kiwamu, I'd like to ask you to package this addon using the new dh-elpa infrastructure. The pkg-emacsen team are slowly working to convert all existing Emacs Lisp addon packages, such as major modes, to use dh-elpa. Here is what you should do if you'd like to

Bug#840424: RFS: verilog-mode/0:20160910.debfc6d.vpo-1

2016-10-11 Thread Kiwamu Okabe
Package: sponsorship-requests Severity: normal Dear mentors, I am looking for a sponsor for my package "verilog-mode" Package name: verilog-mode Version : 0:20160910.debfc6d.vpo-1 Upstream Author : Michael McNamara , Wilson Snyder URL