<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Tue, Jul 31, 2018 at 6:04 PM Carsten Schoenert <<a href="mailto:c.schoenert@t-online.de">c.schoenert@t-online.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi,<br>
<br>
Am 31.07.18 um 18:28 schrieb Onkar Shinde:<br>
> Hi all,<br>
> <br>
> I have released a new version of gnusim8085 yesterday 1.4.1. This version<br>
> can now be built using GTK3 and GtkSourceView3 libraries.<br>
> I have updated the debian packaging files in the salsa git repo. I have<br>
> made sure that new version can be built using these files in sbuild chroot<br>
> for sid.<br>
> <br>
> If required I can upload the source package to <a href="http://mentors.debian.net" rel="noreferrer" target="_blank">mentors.debian.net</a>. Please<br>
> let me know you want me to do that.<br>
<br>
not needed, I guess most of the people here in group prefer going<br>
through history and source within a git tree.<br>
<br>
So far I've seen you have added all the changes within one single<br>
commit. Please don't do this, make single atomic commit that can be<br>
easily reviewed, reverted, later modified and so on. <br></blockquote><div><br></div><div>Will keep this in mind going forward. <br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Also the packaging should follow DEP14 naming conventions [14] and<br>
further on contain the source within a dedicated source branch. And we<br>
want / need pristine-tar data. So I would move over the master branch<br>
into debian/sid and create the others which are needed. I can do this if<br>
you agree.<br></blockquote><div><br></div><div>Please go ahead. I have never done this before so I don't want to mess things up. I would rather watch and learn. <br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
You bumped Standards-Versions to 4.1.5, you are sure there are no<br>
further changes needed? So please mention this then.<br></blockquote><div> </div><div>I verified the differences between the 3.9.1 and 4.1.5 and found nothing that needs to be taken care for this package.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Why stick with debhelper 9? Current version is 11.<br></blockquote><div><br></div><div>It was more of what is the minimum version requested by policy. I did not find anything specific in version 11 which I wanted to use and hence kept at 9. Also looking at [1], it sounds like package build will break if we bump to 11.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
The cdbs package is for sure not needed anymore. If yes get rid of this<br>
need.<br></blockquote><div><br></div><div>First attempt at packaging after so many years so did not want to major changes. I will consider getting rid of cdbs with next upstream release. <br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
debian/watch should be bumped to version 4.<br></blockquote><div><br></div><div>Will you do this for me if you are looking into importing upstream source </div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
And I personally suggest to make the package ready for maintaining it by<br>
git-buildpackage.<br></blockquote><div><br></div><div>Will try to learn about this. :-)<br></div></div><div class="gmail_quote"><br clear="all"></div><div>[1] <a href="https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=885407">https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=885407</a></div><div><br></div><div>Regards,</div><div>Onkar<br></div>-- <br><div dir="ltr" class="gmail_signature">Passion - Some people climb mountains - others write Free software.<br>Don't ask why - the reason is the same.</div></div>