<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Le mer. 4 mars 2020 à 16:38, eamanu <<a href="mailto:eamanu@yaerobi.com">eamanu@yaerobi.com</a>> a écrit :<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 olivier,<br>
<br>
On 04/03/2020 12:13, olivier sallou wrote:<br>
> Hi,<br>
> I saw you started some work on python-ftputil to fix #952208.<br>
> <br>
> I just pushed to git repo a fix for that.<br>
<br>
I am a little confused, you just added description to the patch?<br></blockquote><div><br></div><div>No, I added a patch "debian/patches/fix_tests_future.patch" as explained in d/changelog</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">
<br>
Why do you  douplicate on d/changelog  the (Closes #bugnumber) sentence? [0]<br></blockquote><div><br></div><div>was temporary, just referring what my patch impacts and will close this issue.</div><div><br></div><div>The skip you made in your patch does not fix all issues, only workaround them.</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">
<br>
> <br>
> Will you push new patch release soon? Or should I do?<br>
<br>
I'm DM and I will need sponsorhip.<br></blockquote><div><br></div><div>I am DD, so I can upload it if you think it is "over" for you.</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">
> <br>
> I wonder however why you pushed a gbp.conf (not really expected usually)<br>
> setting branch as debian/master , which does not exists and default<br>
> being master.<br>
<br>
I push gbp.conf and set debian/master as default branch to be the<br>
repository according to the DPMT policy recommendation [1] and DEP-14 [2]<br></blockquote><div><br></div><div>ooh ok, I see a "new" debian/master branch, which did not exists on my local clone....</div><div>So I pushed to master branch instead of debian/master..... that's reason why we do not "see" everything...</div><div><br></div><div>I have updated debian/master branch accordingly</div><div>I pushed my patch and disabled your patch to skip the tests from d/patches/series</div><div>A gbp buildpackage works with all tests now.</div><div><br></div><div>Maybe master branch should be removed to avoid this kind of confusion</div><div><br></div><div><br></div><div>Olivier</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">
<br>
[0]<br>
<a href="https://salsa.debian.org/python-team/modules/python-ftputil/-/commit/a52fad2009b720882fed9bbed366b5e70e9154ae" rel="noreferrer" target="_blank">https://salsa.debian.org/python-team/modules/python-ftputil/-/commit/a52fad2009b720882fed9bbed366b5e70e9154ae</a><br>
[1]<br>
<a href="https://salsa.debian.org/python-team/tools/python-modules/blob/master/policy.rst#branch-names" rel="noreferrer" target="_blank">https://salsa.debian.org/python-team/tools/python-modules/blob/master/policy.rst#branch-names</a><br>
[2] <a href="https://dep-team.pages.debian.net/deps/dep14/" rel="noreferrer" target="_blank">https://dep-team.pages.debian.net/deps/dep14/</a><br>
<br>
Cheers!<br>
<br>
-- <br>
Emmanuel Arias<br>
@eamanu<br>
<a href="http://yaerobi.com" rel="noreferrer" target="_blank">yaerobi.com</a><br>
</blockquote></div></div>