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

Add poezio #8719

Merged
merged 3 commits into from
Jul 14, 2015
Merged

Add poezio #8719

merged 3 commits into from
Jul 14, 2015

Conversation

lsix
Copy link
Member

@lsix lsix commented Jul 9, 2015

No description provided.

@pSub
Copy link
Member

pSub commented Jul 9, 2015

As poezio is a cli application you might want to use namePrefix = "";.


meta = {
description = "A DNS toolkit for Python 3.x";
homepage = "http://www.dnspython.org";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You do not need quotes around urls.

@lsix
Copy link
Member Author

lsix commented Jul 10, 2015

Updated. Thanks for feedbacks.

@pSub
Copy link
Member

pSub commented Jul 10, 2015

Ah, one more thing. Do you want to add yourself as maintainer?

@lsix
Copy link
Member Author

lsix commented Jul 10, 2015

Done.

I also included a small fix, let me know if the patch files should be placed somewhere else.

lucabrunox pushed a commit that referenced this pull request Jul 14, 2015
@lucabrunox lucabrunox merged commit 6bf51d3 into NixOS:master Jul 14, 2015
@lucabrunox
Copy link
Contributor

Looks fine. Thanks and welcome.

@lsix lsix deleted the add_poezio branch December 12, 2015 10:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants