<div dir="ltr"><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)">Hi Gianfranco,<br><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">review done and changes pushed on git :)<br>
<br>
Please upstream patches<br></blockquote><div><br></div><div>Patches sent :) (via email)<br><br><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
changes:<br>
- Make installation configurable<br>
<br>
- Remove useless debian/dirs<br>
- Put flags at the end, to avoid link failures when Wl, asneeded is the default<br>
<br>
(e.g. Ubuntu)<br>
<br>
- Fix build installation<br>
<br>
let me know if you like them or not!<br></blockquote><div><br></div><div>So for so good, i had absolutely no idea about that problem with "WL, asneeded" and flag order, i should be paying attention for that in the future. <br></div></div><div> <br></div><div>All the changes looks good to me, i've renamed one patch you made from fix-install.patch to fix_install.patch in order to follow other patch's names.<br></div><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">BTW also lintian should be fixed in a future upload:<br>
P: mdk3 source: source-contains-svn-control-di<wbr>r osdep/radiotap/.svn<br>
<br>
and licensecheck<br>
<br>
licensecheck * -r<br>
<br>
*: GPL (v2 or later) (with incorrect FSF address)<br></blockquote>
<span class="gmail-m_-7049295969911659009gmail-im"><br></span></div><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)"><span class="gmail-m_-7049295969911659009gmail-im">If i understood correctly, debhelper doesn't ship these .svn files, so its more of a cosmetic change, right? Don't get me wrong, this ~should~ be fixed, but we can wait for an upstream fix.<br><br></span></div><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)"><span class="gmail-m_-7049295969911659009gmail-im">I've just commited a patch that fixes the FSF address, so that's one problem less.<br></span><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">awesome, you are now DM for t50, please do source-only uploads, and give
 me good packages if you want DM permissions for something else<br>
(I have to sponsor each package once for you before granting them, sorry!)<br></blockquote><div><br></div><div>Awesome it is! Thanks for the dak and advocacy Gianfranco, i understand your worry about granting permission on another packages.<br><br></div><div>I believe we're ready to upload mdk3, if you agree with my last changes, please feel free to send it up.<br><br></div><div>Regards,<br></div></div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><span style="font-family:verdana,sans-serif"><span style="color:rgb(102,102,102)">Samuel Henrique <samueloph></span></span></div></div></div></div></div></div></div>
<br><div class="gmail_quote">2016-10-17 17:12 GMT-02:00 Gianfranco Costamagna <span dir="ltr"><<a href="mailto:locutusofborg@debian.org" target="_blank">locutusofborg@debian.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Samuel!<br>
<span class=""><br>
>I've just finished the packaging of mdk3 and would like someone to review and upload the package.<br>
<br>
<br>
</span>review done and changes pushed on git :)<br>
<br>
Please upstream patches<br>
<br>
changes:<br>
- Make installation configurable<br>
<br>
- Remove useless debian/dirs<br>
- Put flags at the end, to avoid link failures when Wl, asneeded is the default<br>
<br>
(e.g. Ubuntu)<br>
<br>
- Fix build installation<br>
<br>
let me know if you like them or not!<br>
<br>
BTW also lintian should be fixed in a future upload:<br>
P: mdk3 source: source-contains-svn-control-<wbr>dir osdep/radiotap/.svn<br>
<br>
and licensecheck<br>
<br>
licensecheck * -r<br>
<br>
*: GPL (v2 or later) (with incorrect FSF address)<br>
<span class=""><br>
>My key got added to the keyring about one week ago, so if someone could grant me upload rights to the packages i maintain or only the packages inside the team, that >would be great.<br>
<br>
<br>
</span>awesome, you are now DM for t50, please do source-only uploads, and give me good packages if you want DM permissions for something else<br>
(I have to sponsor each package once for you before granting them, sorry!)<br>
<br>
please let me know :)<br>
(feel free to revert if you don't like them :) )<br>
<br>
thanks!<br>
<span class="HOEnZb"><font color="#888888"><br>
G.<br>
</font></span></blockquote></div><br></div>