Hi Sean, On Wed, Dec 7, 2016 at 8:47 AM, Sean Whitton <spwhit...@spwhitton.name> 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: > > Copyright: 2008-2013 Michael McNamara and Wilson Snyder > > IANAL but I think there might be a legal distinction between these two, > so it's best to follow what upstream wrote in the file header.
Fixed. And following description also has same issue, which is fixed. ``` Files: 0test.el batch_prof.el batch_test.el batch_test.pl Copyright: 2008-2013, Michael McNamara and Wilson Snyder ``` > 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 > us. Don't forget a DEP-3 patch header. Umm... I don't think so. I should not install the info file into the verilog-mode package, with your advice. Because I believe that Debian source package having big patches is a bad manner. Already the installation of the info file is removed at the git repo. Could you review the git repo, again? Best regards, -- Kiwamu Okabe at METASEPI DESIGN
