From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:51747)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <Debian-debbugs@debbugs.gnu.org>) id 1d8jeo-0006SM-2i
	for guix-patches@gnu.org; Thu, 11 May 2017 04:41:11 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <Debian-debbugs@debbugs.gnu.org>) id 1d8jeg-0003tI-2n
	for guix-patches@gnu.org; Thu, 11 May 2017 04:41:09 -0400
Received: from debbugs.gnu.org ([208.118.235.43]:34958)
	by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16)
	(Exim 4.71) (envelope-from <Debian-debbugs@debbugs.gnu.org>)
	id 1d8jef-0003tE-V4
	for guix-patches@gnu.org; Thu, 11 May 2017 04:41:01 -0400
Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2)
	(envelope-from <Debian-debbugs@debbugs.gnu.org>) id 1d8jef-0005S9-Ma
	for guix-patches@gnu.org; Thu, 11 May 2017 04:41:01 -0400
Subject: bug#26803: [PATCH 34/36] gnu: Add java-commons-net.
Resent-Message-ID: <handler.26803.B26803.149449203420918@debbugs.gnu.org>
References: <20170506153617.3074-1-rekado@elephly.net>
	<20170506153617.3074-34-rekado@elephly.net>
From: Roel Janssen <roel@gnu.org>
In-reply-to: <20170506153617.3074-34-rekado@elephly.net>
Date: Thu, 11 May 2017 10:40:19 +0200
Message-ID: <rbuvap7pz0s.fsf@gnu.org>
MIME-Version: 1.0
Content-Type: text/plain
List-Id: <guix-patches.gnu.org>
List-Unsubscribe: <https://lists.gnu.org/mailman/options/guix-patches>,
	<mailto:guix-patches-request@gnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/archive/html/guix-patches/>
List-Post: <mailto:guix-patches@gnu.org>
List-Help: <mailto:guix-patches-request@gnu.org?subject=help>
List-Subscribe: <https://lists.gnu.org/mailman/listinfo/guix-patches>,
	<mailto:guix-patches-request@gnu.org?subject=subscribe>
Errors-To: guix-patches-bounces+kyle=kyleam.com@gnu.org
Sender: "Guix-patches" <guix-patches-bounces+kyle=kyleam.com@gnu.org>
To: Ricardo Wurmus <rekado@elephly.net>
Cc: 26803@debbugs.gnu.org


Ricardo Wurmus writes:

> * gnu/packages/java.scm (java-commons-net): New variable.
> ---
>  gnu/packages/java.scm | 28 ++++++++++++++++++++++++++++
>  1 file changed, 28 insertions(+)
>
> diff --git a/gnu/packages/java.scm b/gnu/packages/java.scm
> index 1639570f9..287a7ae69 100644
> --- a/gnu/packages/java.scm
> +++ b/gnu/packages/java.scm
> @@ -2219,6 +2219,34 @@ project in its HttpClient and HttpCore modules, which offer better performance
>  and more flexibility.")
>      (license license:asl2.0)))
>  
> +(define-public java-commons-net
> +  (package
> +    (name "java-commons-net")
> +    (version "3.6")
> +    (source (origin
> +              (method url-fetch)
> +              (uri (string-append "mirror://apache/commons/net/source/"
> +                                  "commons-net-" version "-src.tar.gz"))
> +              (sha256
> +               (base32
> +                "0n0cmnddk9qdqhjvka8pc6hd9mn2qi3166f1s6xk32h7rfy1adxr"))))
> +    (build-system ant-build-system)
> +    (arguments
> +     `(;; FIXME: MainTest.java tries to read "examples.properties" (which
> +       ;; should be "resources/examples/examples.properties"), but gets "null"
> +       ;; instead.

This sounds fixable in a substitute construct.  But maybe I am wrong
about it.  Would you like to try that first?

> +       #:tests? #f
> +       #:jar-name "commons-net.jar"))
> +    (native-inputs
> +     `(("java-junit" ,java-junit)
> +       ("java-hamcrest-core" ,java-hamcrest-core)))
> +    (home-page "http://commons.apache.org/net/")
> +    (synopsis "Client library for many basic Internet protocols")
> +    (description "The Apache Commons Net library implements the client side of
> +many basic Internet protocols.  The purpose of the library is to provide
> +fundamental protocol access, not higher-level abstractions.")
> +    (license license:asl2.0)))
> +
>  (define-public java-commons-cli
>    (package
>      (name "java-commons-cli")

Otherwise, LGTM.

Kind regards,
Roel Janssen