Rafael Sadowski <raf...@sizeofvoid.org> writes: > Hi @ports, > > below is an update diff to net/lftp 4.7.4. Tested on amd64. > > OK, comments?
Please see below, > Kind regards, > > Rafael Sadowski > > > Index: Makefile > =================================================================== > RCS file: /cvs/ports/net/lftp/Makefile,v > retrieving revision 1.110 > diff -u -p -u -p -r1.110 Makefile > --- Makefile 7 Aug 2016 21:40:20 -0000 1.110 > +++ Makefile 19 Nov 2016 20:31:17 -0000 > @@ -2,10 +2,10 @@ > > COMMENT= shell-like command line ftp and sftp client > > -DISTNAME= lftp-4.7.3 > +DISTNAME= lftp-4.7.4 > CATEGORIES= net > > -HOMEPAGE= https://lftp.yar.ru/ > +HOMEPAGE= https://lftp.tech/ > > MAINTAINER= Rafael Sadowski <raf...@sizeofvoid.org> > > @@ -20,6 +20,10 @@ USE_GROFF= Yes > # lftp needs add_history_time() > LIB_DEPENDS= devel/libidn \ > devel/readline>=6.1p2 \ > + > +RUN_DEPENDS= devel/gettext I do not understand this. gettext could be part of LIB_DEPENDS, but why would it be a RUN_DEPENDS? Even if lftp directly uses gettext, it seems that the preferred approach is not to explicitely add it to LIB_DEPENDS, as libidn already brings it. > +BUILD_DEPENDS = devel/gettext-tools I see that there are tests for these tools at configure time, however they don't seem to be strictly needed to build a proper package. No objection if you want to keep this in BUILD_DEPENDS, if you think it might prevent failures in bulks one day. > MASTER_SITES= https://lftp.yar.ru/ftp/ There is a problem with MASTER_SITES and redirections here. ritchie /usr/ports/net/lftp$ ftp https://lftp.yar.ru/ftp/lftp-4.7.4.tar.gz Trying 213.187.99.145... Requesting https://lftp.yar.ru/ftp/lftp-4.7.4.tar.gz Redirected to https://lftp.tech/ftp/lftp-4.7.4.tar.gz Trying 213.187.99.145... Requesting https://lftp.tech/ftp/lftp-4.7.4.tar.gz Redirected to https://lftp.tech/get.html Trying 213.187.99.145... Requesting https://lftp.tech/get.html 100% |********[...]***************| 10156 00:00 10156 bytes received in 0.02 seconds (399.32 KB/s) ritchie /usr/ports/net/lftp$ ftp http://lftp.tech/ftp/lftp-4.7.4.tar.gz Trying 213.187.99.145... Requesting http://lftp.tech/ftp/lftp-4.7.4.tar.gz Redirected to http://lftp.tech/get.html Trying 213.187.99.145... Requesting http://lftp.tech/get.html 100% |********[...]***************| 10156 00:00 10156 bytes received in 0.02 seconds (412.92 KB/s) Did you use ''make fetch'' to grab the tarball? In the meantime we could add another mirror to MASTER_SITES, as in the updated diff below (which also does rm patches/patch-src_Resolver_cc, as we have getaddrinfo). Index: Makefile =================================================================== RCS file: /d/cvs/ports/net/lftp/Makefile,v retrieving revision 1.110 diff -u -p -r1.110 Makefile --- Makefile 7 Aug 2016 21:40:20 -0000 1.110 +++ Makefile 22 Nov 2016 03:20:41 -0000 @@ -2,10 +2,10 @@ COMMENT= shell-like command line ftp and sftp client -DISTNAME= lftp-4.7.3 +DISTNAME= lftp-4.7.4 CATEGORIES= net -HOMEPAGE= https://lftp.yar.ru/ +HOMEPAGE= https://lftp.tech/ MAINTAINER= Rafael Sadowski <raf...@sizeofvoid.org> @@ -17,11 +17,13 @@ WANTLIB += stdc++ util z USE_GROFF= Yes +BUILD_DEPENDS = devel/gettext-tools # lftp needs add_history_time() LIB_DEPENDS= devel/libidn \ devel/readline>=6.1p2 \ -MASTER_SITES= https://lftp.yar.ru/ftp/ +MASTER_SITES= https://lftp.yar.ru/ftp/ \ + ftp://ftp.st.ryukoku.ac.jp/pub/network/ftp/lftp/ LIBTOOL_FLAGS= --tag=disable-shared CONFIGURE_STYLE= gnu Index: distinfo =================================================================== RCS file: /d/cvs/ports/net/lftp/distinfo,v retrieving revision 1.69 diff -u -p -r1.69 distinfo --- distinfo 7 Aug 2016 21:40:20 -0000 1.69 +++ distinfo 22 Nov 2016 02:57:25 -0000 @@ -1,2 +1,2 @@ -SHA256 (lftp-4.7.3.tar.gz) = zmUZgxYDwZws8uPBC0HW3ch6FgSbmTg+e5t3+8dwchQ= -SIZE (lftp-4.7.3.tar.gz) = 2715817 +SHA256 (lftp-4.7.4.tar.gz) = j1XDfP37ziSpy0e2IcoSf0DTp95bTEn4xJ8Xv/co+zE= +SIZE (lftp-4.7.4.tar.gz) = 2766143 Index: patches/patch-configure =================================================================== RCS file: /d/cvs/ports/net/lftp/patches/patch-configure,v retrieving revision 1.14 diff -u -p -r1.14 patch-configure --- patches/patch-configure 7 Aug 2016 21:40:20 -0000 1.14 +++ patches/patch-configure 22 Nov 2016 02:57:25 -0000 @@ -2,9 +2,9 @@ $OpenBSD: patch-configure,v 1.14 2016/08 Use GNU readline as our base readline lacks add_history_time(). ---- configure.orig Fri Jul 15 12:43:04 2016 -+++ configure Thu Jul 21 05:20:48 2016 -@@ -48435,7 +48435,7 @@ fi +--- configure.orig Wed Nov 16 14:11:30 2016 ++++ configure Thu Nov 17 18:51:19 2016 +@@ -48854,7 +48854,7 @@ fi readline_include_dir="$readline_include_dir/readline" fi readline_ld_flags="-L$readline_prefix/lib" Index: patches/patch-src_Makefile_in =================================================================== RCS file: /d/cvs/ports/net/lftp/patches/patch-src_Makefile_in,v retrieving revision 1.12 diff -u -p -r1.12 patch-src_Makefile_in --- patches/patch-src_Makefile_in 27 May 2016 12:08:46 -0000 1.12 +++ patches/patch-src_Makefile_in 22 Nov 2016 02:57:25 -0000 @@ -1,7 +1,7 @@ $OpenBSD: patch-src_Makefile_in,v 1.12 2016/05/27 12:08:46 jca Exp $ ---- src/Makefile.in.orig Mon May 23 17:34:28 2016 -+++ src/Makefile.in Mon May 23 17:35:10 2016 -@@ -235,7 +235,7 @@ am__installdirs = "$(DESTDIR)$(libdir)" "$(DESTDIR)$(p +--- src/Makefile.in.orig Wed Nov 16 14:11:29 2016 ++++ src/Makefile.in Thu Nov 17 18:51:19 2016 +@@ -237,7 +237,7 @@ am__installdirs = "$(DESTDIR)$(libdir)" "$(DESTDIR)$(p "$(DESTDIR)$(bindir)" "$(DESTDIR)$(bindir)" \ "$(DESTDIR)$(pkgdatadir)" LTLIBRARIES = $(lib_LTLIBRARIES) $(pkgverlib_LTLIBRARIES) Index: patches/patch-src_Resolver_cc =================================================================== RCS file: patches/patch-src_Resolver_cc diff -N patches/patch-src_Resolver_cc --- patches/patch-src_Resolver_cc 16 Sep 2014 12:35:18 -0000 1.12 +++ /dev/null 1 Jan 1970 00:00:00 -0000 @@ -1,12 +0,0 @@ -$OpenBSD: patch-src_Resolver_cc,v 1.12 2014/09/16 12:35:18 sthen Exp $ ---- src/Resolver.cc.orig Wed Aug 27 09:34:02 2014 -+++ src/Resolver.cc Tue Sep 16 13:21:48 2014 -@@ -801,8 +801,6 @@ void Resolver::LookupOne(const char *name) - # undef h_errno // it could be a macro, but we want it to be local variable. - int h_errno=0; - ha=getipnodebyname(name,af,0,&h_errno); --# elif defined(HAVE_GETHOSTBYNAME2) -- ha=gethostbyname2(name,af); - # else - if(af==AF_INET) - ha=gethostbyname(name); Index: patches/patch-src_lftp_rl_c =================================================================== RCS file: patches/patch-src_lftp_rl_c diff -N patches/patch-src_lftp_rl_c --- patches/patch-src_lftp_rl_c 27 May 2016 12:09:20 -0000 1.1 +++ /dev/null 1 Jan 1970 00:00:00 -0000 @@ -1,14 +0,0 @@ -$OpenBSD: patch-src_lftp_rl_c,v 1.1 2016/05/27 12:09:20 jca Exp $ ---- src/lftp_rl.c.orig Mon May 23 17:46:56 2016 -+++ src/lftp_rl.c Mon May 23 17:47:33 2016 -@@ -19,8 +19,8 @@ - - #include <config.h> - #include <stdio.h> --#include <readline.h> --#include <history.h> -+#include <readline/readline.h> -+#include <readline/history.h> - #include <stdlib.h> - #include <string.h> - #include "lftp_rl.h" -- jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF DDCC 0DFA 74AE 1524 E7EE
