Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

clarify changes required to dnspython library #17

Closed
anarcat opened this issue May 18, 2016 · 10 comments
Closed

clarify changes required to dnspython library #17

anarcat opened this issue May 18, 2016 · 10 comments
Assignees
Labels

Comments

@anarcat
Copy link

anarcat commented May 18, 2016

Hi!

I am looking at packaging this for Debian and since the DNSpython library is already packaged in Debian, it is unclear how I will need to proceed.

What exactly are the changes required? Looking at the submodule, i see the following three commits:

... but those seem to be based on the upstream python3 branch, which doesn't seem to be merged with a release upstream.

Is that accurate?

Would it be possible to make a pull request that would regularize the situation? :)

Thanks!

@farrokhi farrokhi self-assigned this May 25, 2016
@farrokhi
Copy link
Owner

I created a pull request and awaiting for the merge.

@farrokhi
Copy link
Owner

Author was not interested to accept the pull request: rthalley/dnspython#163

@anarcat
Copy link
Author

anarcat commented May 29, 2016

i think i was able to somewhat change his mind - but now it requires more changes to dnsdiag, i think. is that something for which i should open a feature request or something?

thanks for following up!

@farrokhi
Copy link
Owner

I will make the changes based on the same ticket.

On 30 May 2016, at 1:50, anarcat wrote:

i think i was able to somewhat change his mind - but now it requires more changes to dnsdiag, i think. is that something for which i should open a feature request or something?

thanks for following up!


You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub:
#17 (comment)

@anarcat
Copy link
Author

anarcat commented May 30, 2016 via email

@farrokhi
Copy link
Owner

farrokhi commented Jun 1, 2016

@anarcat updated on dev branch. I am waiting for dnspython maintainer to upload new package to pypi so that I can merge with master.

@ana-cc
Copy link

ana-cc commented Jun 18, 2016

Hi, I currently have the ITP bug for dnsdiag in Debian- just waiting for this to be resolved before I proceed.

@farrokhi
Copy link
Owner

I am waiting for dnspython maintainer to upload latest version to pypi. I queried them and haven't receive a response yet.

@farrokhi
Copy link
Owner

@anarcat dnspython people updated their package on pypi with all the changes we needed. So I removed the local modified version of dnspython and now we depend on official dnspython from pypi. I pushed the changes and I believe you can package it for debian now.

@anarcat
Copy link
Author

anarcat commented Oct 18, 2016

@farrokhi awesome, thanks! turns out that @ana-cc has taken over my "RFP" (Request For Package) and turned it into an "ITP" (Intend To Package) which is great news: someone else than me will take care of this. :)

The ITP is here, if anyone wants to follow that progress, and I will notify that issue that this is fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants