From: Leo Famulari <leo@famulari.name>
To: Danny Milosavljevic <dannym@scratchpost.org>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH v5 04/17] gnu: Add perl-net-dns-resolver-programmable.
Date: Mon, 8 Aug 2016 15:41:15 -0400 [thread overview]
Message-ID: <20160808194115.GA18650@jasmine> (raw)
In-Reply-To: <20160728213833.22057-5-dannym@scratchpost.org>
On Thu, Jul 28, 2016 at 11:38:20PM +0200, Danny Milosavljevic wrote:
>
> gnu: Add perl-net-dns-resolver-programmable.
>
> * gnu/packages/networking.scm (perl-net-dns-resolver-programmable): New variable.
I'm going through v5 of this patch series. This is the first patch I
have a question about.
> +(define-public perl-net-dns-resolver-programmable
> + (package
> + (name "perl-net-dns-resolver-programmable")
> + (version "v0.003")
> + (source
> + (origin
> + (method url-fetch)
> + (uri (string-append
> + "mirror://cpan/authors/id/J/JM/JMEHNLE/net-dns-resolver-programmable/"
> + "Net-DNS-Resolver-Programmable-" version ".tar.gz"))
> + (sha256
> + (base32
> + "1v3nl2kaj4fs55n1617n53q8sa3mir06898vpy1rq98zjih24h4d"))
> + (patches
> + (list (origin
> + (method url-fetch)
> + (uri "https://rt.cpan.org/Public/Ticket/Attachment/1575108/841078/patch.txt")
> + (sha256
> + (base32
> + "027fzq1dryqwhkprz4r32vj78qfdnx1mhjx6piag7p62p5npmdic")))))))
Can you send a comment explaining why we have to apply this patch? Its
URL does not enable me to "go back" to the relevant discussion and learn
the context or identity of the author.
Assuming we keep the upstream patch, I will download it and add it to
'gnu/packages/patches', and apply it using the method seen in the cpio
package definition. I can do that on my own, or you can submit a v6 of
this patch.
next prev parent reply other threads:[~2016-08-08 19:41 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-07-28 21:38 [PATCH v5 00/17] Add missing dependencies of Spamassassin Danny Milosavljevic
2016-07-28 21:38 ` [PATCH v5 01/17] gnu: Add perl-db-file Danny Milosavljevic
2016-07-31 11:39 ` Danny Milosavljevic
2016-07-28 21:38 ` [PATCH v5 02/17] gnu: Add perl-net-dns Danny Milosavljevic
2016-07-28 21:38 ` [PATCH v5 03/17] gnu: Add perl-socket6 Danny Milosavljevic
2016-07-28 21:38 ` [PATCH v5 04/17] gnu: Add perl-net-dns-resolver-programmable Danny Milosavljevic
2016-08-08 19:41 ` Leo Famulari [this message]
2016-08-08 21:08 ` Danny Milosavljevic
2016-07-28 21:38 ` [PATCH v5 05/17] gnu: Add perl-netaddr-ip Danny Milosavljevic
2016-07-28 21:38 ` [PATCH v5 06/17] gnu: Add perl-net-patricia Danny Milosavljevic
2016-08-08 19:44 ` Leo Famulari
2016-08-08 21:09 ` Danny Milosavljevic
2016-07-28 21:38 ` [PATCH v5 07/17] gnu: Add perl-file-find-object Danny Milosavljevic
2016-07-28 21:38 ` [PATCH v5 08/17] gnu: Add perl-file-find-object-rule Danny Milosavljevic
2016-07-28 21:38 ` [PATCH v5 09/17] gnu: Add perl-test-trailingspace Danny Milosavljevic
2016-07-28 21:38 ` [PATCH v5 10/17] gnu: Add perl-encode-detect Danny Milosavljevic
2016-07-28 21:38 ` [PATCH v5 11/17] gnu: Add perl-net-cidr-lite Danny Milosavljevic
2016-07-28 21:38 ` [PATCH v5 12/17] gnu: Add perl-geo-ip Danny Milosavljevic
2016-08-08 19:46 ` Leo Famulari
2016-08-08 22:50 ` Danny Milosavljevic
2016-07-28 21:38 ` [PATCH v5 13/17] gnu: Add perl-crypt-openssl-rsa Danny Milosavljevic
2016-07-28 21:38 ` [PATCH v5 14/17] gnu: Add perl-crypt-openssl-bignum Danny Milosavljevic
2016-08-08 19:54 ` Leo Famulari
2016-08-08 21:18 ` Danny Milosavljevic
2016-08-09 1:06 ` Leo Famulari
2016-07-28 21:38 ` [PATCH v5 15/17] gnu: Add perl-mail-spf Danny Milosavljevic
2016-07-28 21:38 ` [PATCH v5 16/17] gnu: Add perl-io-socket-inet6 Danny Milosavljevic
2016-07-28 21:38 ` [PATCH v5 17/17] gnu: Add perl-crypt-openssl-random Danny Milosavljevic
2016-08-08 19:54 ` [PATCH v5 00/17] Add missing dependencies of Spamassassin Leo Famulari
2016-08-09 2:38 ` Leo Famulari
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://guix.gnu.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20160808194115.GA18650@jasmine \
--to=leo@famulari.name \
--cc=dannym@scratchpost.org \
--cc=guix-devel@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/guix.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).