Re: Certbot DNS Dependencies

[Ana C. Custura]

Hi Harlan,

On Sun, Mar 11, 2018, at 8:43 PM, Harlan Lieberman-Berg wrote:
> I took a look; thanks for the update.  Can you move the publicsuffix
> local URL to the top of the list? Based on its current position, it
> will (essentially) never be used.Ok, changed and uploaded. 

> 
> I also created the repos under the DMPT heirarchy and granted you
> explicit access to them; once you join the DPMT team, you'll inherit
> the permissions from there in addition.> 
> https://salsa.debian.org/python-team/modules/python-tldextract
> https://salsa.debian.org/python-team/modules/python-lexicon
Thank you for this, and I've already sent out a request to join DPMT!

Regards,
Ana
> 
> Sincerely,
> 
> On Mon, Mar 5, 2018 at 12:21 AM, Ana C. Custura
>  wrote:>> __
>> Hi Harlan,
>> 
>> Thank you for the feedback!
>> 
>> 
>> On Sat, Mar 3, 2018, at 1:48 AM, Harlan Lieberman-Berg wrote:
>>> For the VCSs', you have two options; if you want to join DPMT, you
>>> could put them under their hierarchy, or you could use the Debian
>>> heirarchy and I can grant you permissions for the repos.  Let me
>>> know which one you'd prefer to do.>> 
>> The latter for now please. I will fire an email to DPMT to request
>> joining the team, as I'd like to start working with them on the
>> other ITPs.>> 
>>> 
>>> I do have one tweak I'd like to see for python-tldextract; I think
>>> we should have it Recommend `publicsuffix` and add that to the list
>>> of PUBLIC_SUFFIX_LIST_URLs.  Please test it to make sure it works
>>> even without that file; reading the code, it looks like it should,
>>> but I'd like to double check.>> 
>> Done, new version uploaded on mentors. Have tested the relevant code,
>> it works with and without
>> /usr/share/publicsuffix/effective_tld_names.dat.>> 
>>> 
>>> For python-lexicon, it's generally considered good practice to use
>>> the same license for the work under debian/ as the rest of the
>>> package is.  If the difference was intentional, I'd want to know
>>> more about why you feel the difference is important.>> 
>> Ah, I forgot to change it from the dh_make default. Thank you for
>> pointing it out, new version on mentors!>> 
>> Let me know if there's anything else to do.
>> 
>> Regards,
>> Ana
>> 
> 
> 
> 
> -- 
> 
> Harlan Lieberman-Berg
> ~hlieberman

Re: Certbot DNS Dependencies

[Harlan Lieberman-Berg]

Hi Ana!

Sorry for the delay; I've been away for work for a bit.

I took a look; thanks for the update.  Can you move the publicsuffix local
URL to the top of the list? Based on its current position, it will
(essentially) never be used.

I also created the repos under the DMPT heirarchy and granted you explicit
access to them; once you join the DPMT team, you'll inherit the permissions
from there in addition.

https://salsa.debian.org/python-team/modules/python-tldextract
https://salsa.debian.org/python-team/modules/python-lexicon

Other than that, tldextract looks good and is ready for an upload as soon
as that tweak is done.  Lexicon is fine as-is; I'll push that up when
tldextract gets uploaded.

Sincerely,

On Mon, Mar 5, 2018 at 12:21 AM, Ana C. Custura  wrote:

> Hi Harlan,
>
> Thank you for the feedback!
>
> On Sat, Mar 3, 2018, at 1:48 AM, Harlan Lieberman-Berg wrote:
>
> For the VCSs', you have two options; if you want to join DPMT, you could
> put them under their hierarchy, or you could use the Debian heirarchy and I
> can grant you permissions for the repos.  Let me know which one you'd
> prefer to do.
>
> The latter for now please. I will fire an email to DPMT to request joining
> the team, as I'd like to start working with them on the other ITPs.
>
>
> I do have one tweak I'd like to see for python-tldextract; I think we
> should have it Recommend `publicsuffix` and add that to the list of
> PUBLIC_SUFFIX_LIST_URLs.  Please test it to make sure it works even without
> that file; reading the code, it looks like it should, but I'd like to
> double check.
>
> Done, new version uploaded on mentors. Have tested the relevant code,  it
> works with and without /usr/share/publicsuffix/effective_tld_names.dat.
>
>
> For python-lexicon, it's generally considered good practice to use the
> same license for the work under debian/ as the rest of the package is.  If
> the difference was intentional, I'd want to know more about why you feel
> the difference is important.
>
> Ah, I forgot to change it from the dh_make default. Thank you for pointing
> it out, new version on mentors!
>
> Let me know if there's anything else to do.
>
> Regards,
> Ana
>
>


-- 
Harlan Lieberman-Berg
~hlieberman

Re: Certbot DNS Dependencies

[Ana C. Custura]

Hi Harlan,

Thank you for the feedback!

On Sat, Mar 3, 2018, at 1:48 AM, Harlan Lieberman-Berg wrote:
> For the VCSs', you have two options; if you want to join DPMT, you
> could put them under their hierarchy, or you could use the Debian
> heirarchy and I can grant you permissions for the repos.  Let me know
> which one you'd prefer to do.The latter for now please. I will fire an email 
> to DPMT to request
joining the team, as I'd like to start working with them on the
other ITPs.> 
> I do have one tweak I'd like to see for python-tldextract; I think we
> should have it Recommend `publicsuffix` and add that to the list of
> PUBLIC_SUFFIX_LIST_URLs.  Please test it to make sure it works even
> without that file; reading the code, it looks like it should, but I'd
> like to double check.Done, new version uploaded on mentors. Have tested the 
> relevant code,
it works with and without
/usr/share/publicsuffix/effective_tld_names.dat.> 
> For python-lexicon, it's generally considered good practice to use the
> same license for the work under debian/ as the rest of the package is.
> If the difference was intentional, I'd want to know more about why you
> feel the difference is important.Ah, I forgot to change it from the dh_make 
> default. Thank you for
pointing it out, new version on mentors!
Let me know if there's anything else to do.

Regards,
Ana

Re: Certbot DNS Dependencies

[Harlan Lieberman-Berg]

Hello Ana!

These look to be in good shape in general.  For the VCSs', you have two
options; if you want to join DPMT, you could put them under their
hierarchy, or you could use the Debian heirarchy and I can grant you
permissions for the repos.  Let me know which one you'd prefer to do.

I do have one tweak I'd like to see for python-tldextract; I think we
should have it Recommend `publicsuffix` and add that to the list of
PUBLIC_SUFFIX_LIST_URLs.  Please test it to make sure it works even without
that file; reading the code, it looks like it should, but I'd like to
double check.

For python-lexicon, it's generally considered good practice to use the same
license for the work under debian/ as the rest of the package is.  If the
difference was intentional, I'd want to know more about why you feel the
difference is important.

Let me know if you have questions; a couple of minor changes and I think
they're about ready to ship!


On Fri, Mar 2, 2018 at 7:37 PM, Ana C. Custura  wrote:

> Hi Harlan,
>
> Thank you for your email,  I've had some time to look at them and both are
> now on mentors:
> https://mentors.debian.net/package/tldextract
> https://mentors.debian.net/package/lexicon
>
> I would also like to push to salsa, but need help with getting access to
> repositories and/or teams. I've left the vcs fields pointing to
> salsa.debian.org/debian for now.
>
> Some notes:
> tldextract:
> -uses a writable cache file which ends up in usr/share  with the wrong
> permissions during the installation. I moved this to the user's homedir.
>
> lexicon:
> - some optional dependencies still need to be packaged to make some
> provider-specific functionality work; I filed ITPs for python3-pynamecheap
> and python3-transip [ #891944 and #891945 ]
> - it looks like tests fail with pybuild because they need to run from a
> specific directory,  so dh_auto_test is overridden with a custom command.
>
> All feedback is much appreciated, and thanks again!
>
> Ana
>
> On Sun, Feb 25, 2018, at 9:27 PM, Harlan Lieberman-Berg wrote:
>
> On Sun, Feb 25, 2018 at 16:06 Ana C. Custura  wrote:
>
> Thank you for the email!
> Yes I'm almost there with tldextract and I've partly packaged lexicon too,
> but forgot pick up the ITP for it.  I'm not a DD yet, so my for new
> packages I need a sponsor - would you like to be the one to review them and
> upload them?
>
>
> That’s awesome! I just took care of python-cloudflare last night, so
> that’s the list.
>
> Happy to sponsor them for you. Push them up to mentors and ping me when
> you’re ready!
>
> Sincerely,
>
>
> --
> Harlan Lieberman-Berg
> ~hlieberman
>
>
>


-- 
Harlan Lieberman-Berg
~hlieberman

Re: Certbot DNS Dependencies

[Ana C. Custura]

Hi Harlan,

Thank you for your  email,  I've had some time to look at them and both
are now on mentors:https://mentors.debian.net/package/tldextract
https://mentors.debian.net/package/lexicon

I would also like to push to salsa, but need help with getting access to
repositories and/or  teams. I've left the vcs fields pointing to
salsa.debian.org/debian for now.
Some notes:
tldextract:
-uses a writable cache file which ends up in usr/share  with the wrong
permissions during the installation. I moved this to the user's homedir.
lexicon:
- some optional dependencies still need to be packaged to make some provider-
  specific functionality work; I filed ITPs for python3-pynamecheap and
  python3-transip [ #891944 and #891945 ]- it looks like tests fail with 
pybuild because they need to run
  from a specific directory,  so dh_auto_test is overridden with a
  custom command.
All feedback is much appreciated, and thanks again!

Ana

On Sun, Feb 25, 2018, at 9:27 PM, Harlan Lieberman-Berg wrote:
> On Sun, Feb 25, 2018 at 16:06 Ana C. Custura
>  wrote:>> Thank you for the email!
>>  Yes I'm almost there with tldextract and I've partly packaged
>>  lexicon too, but forgot pick up the ITP for it.  I'm not a DD yet,
>>  so my for new packages I need a sponsor - would you like to be the
>>  one to review them and upload them?> 
> That’s awesome! I just took care of python-cloudflare last night, so
> that’s the list.> 
> Happy to sponsor them for you. Push them up to mentors and ping me
> when you’re ready!> 
> Sincerely,
> 
>> 
> -- 
> Harlan Lieberman-Berg
> ~hlieberman

Re: Certbot DNS Dependencies

[Harlan Lieberman-Berg]

On Sun, Feb 25, 2018 at 16:06 Ana C. Custura  wrote:

> Thank you for the email!
> Yes I'm almost there with tldextract and I've partly packaged lexicon too,
> but forgot pick up the ITP for it.  I'm not a DD yet, so my for new
> packages I need a sponsor - would you like to be the one to review them and
> upload them?


That’s awesome! I just took care of python-cloudflare last night, so that’s
the list.

Happy to sponsor them for you. Push them up to mentors and ping me when
you’re ready!

Sincerely,

-- 
Harlan Lieberman-Berg
~hlieberman

Re: Certbot DNS Dependencies

[Ana C. Custura]

Hi Harlan,

Thank you for the email!
Yes I'm almost there with tldextract and I've partly packaged lexicon too, but 
forgot pick up the ITP for it.  I'm not a DD yet, so my for new packages I need 
a sponsor - would you like to be the one to review them and upload them? 

Thank you,
Ana



On Fri, Feb 23, 2018, at 11:24 PM, Harlan Lieberman-Berg wrote:
> On Fri, Feb 23, 2018 at 6:14 PM, Harlan Lieberman-Berg
>  wrote:
> > In order to package some of the certbot DNS plugins, there are a
> > couple of dependencies that currently aren't packaged for Debian.
> 
> 
> Looks like python-tldextract has already been picked up by Ana
> already; thanks Ana!  Please let me know if there's anything I can do
> to help.
> 
> Sincerely,
> -- 
> Harlan Lieberman-Berg
> ~hlieberman

Re: Certbot DNS Dependencies

[Scott Kitterman]

On February 23, 2018 11:14:50 PM UTC, Harlan Lieberman-Berg 
 wrote:
>Hello DPMT!
>
>In order to package some of the certbot DNS plugins, there are a
>couple of dependencies that currently aren't packaged for Debian.
>
>I'm happy to do the leg work for their initial packaging, but I wanted
>to check in to see if you'd be willing to have them maintained by your
>team in terms of updates, with the Let's Encrypt team acting as an
>uploader to help out here and there.
>
>Specifically, the packages are:
>
>python-cloudflare (RFP: #888799)
>python-dns-lexicon (RFP: #00)
>python-tldextract (RFP: #01)
>
>Let me know!

I think adding the packages is fine, but plan on doing most of the work on them 
yourself.  We don't have very many people working on general package updates.

Scott K

Certbot DNS Dependencies

[Harlan Lieberman-Berg]

Hello DPMT!

In order to package some of the certbot DNS plugins, there are a
couple of dependencies that currently aren't packaged for Debian.

I'm happy to do the leg work for their initial packaging, but I wanted
to check in to see if you'd be willing to have them maintained by your
team in terms of updates, with the Let's Encrypt team acting as an
uploader to help out here and there.

Specifically, the packages are:

python-cloudflare (RFP: #888799)
python-dns-lexicon (RFP: #00)
python-tldextract (RFP: #01)

Let me know!

Sincerely,
-- 
Harlan Lieberman-Berg
~hlieberman