Re: Certbot DNS Dependencies

2018-03-11 Thread 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 <a...@netstat.org.uk> 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

2018-03-02 Thread 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 <a...@netstat.org.uk> 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 <a...@netstat.org.uk> 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

2018-02-25 Thread Harlan Lieberman-Berg
On Sun, Feb 25, 2018 at 16:06 Ana C. Custura <a...@netstat.org.uk> 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


Certbot DNS Dependencies

2018-02-23 Thread 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



Request to Join

2016-01-21 Thread Harlan Lieberman-Berg
Hello python modules team!

I'd like to request to join this team to help maintain backports of the Let's 
Encrypt dependencies, especially python-sphinx.

I've read and agree to the Python Modules Team Policy. My Alioth username is 
hlieberman-guest.

Sincerely,
-- 
Harlan Lieberman-Berg
~hlieberman