Any further comments, besides REVISION=0 ?
Jan
> > On 2012/03/24 07:09, Jan Stary wrote:
> > > SoX uses wget(1) to play(1) remote files such as streaming radios.
> > > That is in fact a bug in the port (RUN_DEPENDS). Rather than
> > > adding wget to DEPENDS, the small patch below replaces that
> > > functionality with the base ftp(1).
>
> Index: Makefile
> ===================================================================
> RCS file: /cvs/ports/audio/sox/Makefile,v
> retrieving revision 1.50
> diff -u -p -u -p -r1.50 Makefile
> --- Makefile 23 Mar 2012 13:16:41 -0000 1.50
> +++ Makefile 25 Mar 2012 16:27:03 -0000
> @@ -8,6 +8,7 @@ SHARED_LIBS += sox 3.0 # .2.0
> CATEGORIES= audio
> HOMEPAGE= http://sox.sourceforge.net/
> MAINTAINER= Jan Stary <[email protected]>
> +REVISION= 1
>
> # GPLv2+
> PERMIT_PACKAGE_CDROM= Yes
> Index: patches/patch-src_formats_c
> ===================================================================
> RCS file: patches/patch-src_formats_c
> diff -N patches/patch-src_formats_c
> --- /dev/null 1 Jan 1970 00:00:00 -0000
> +++ patches/patch-src_formats_c 25 Mar 2012 16:27:03 -0000
> @@ -0,0 +1,21 @@
> +$OpenBSD$
> +--- src/formats.c.orig Sat Mar 24 06:29:07 2012
> ++++ src/formats.c Sat Mar 24 06:29:53 2012
> +@@ -348,7 +348,7 @@ static int sox_checkformat(sox_format_t * ft)
> + return SOX_SUCCESS;
> + }
> +
> +-static sox_bool is_url(char const * text) /* detects only wget-supported
> URLs */
> ++static sox_bool is_url(char const * text)
> + {
> + return !(
> + strncasecmp(text, "http:" , (size_t)5) &&
> +@@ -385,7 +385,7 @@ static FILE * xfopen(char const * identifier, char con
> + else if (is_url(identifier)) {
> + FILE * f = NULL;
> + #ifdef HAVE_POPEN
> +- char const * const command_format = "wget --no-check-certificate -q -O-
> \"%s\"";
> ++ char const * const command_format = "ftp -a -V -o - \"%s\"";
> + char * command = lsx_malloc(strlen(command_format) +
> strlen(identifier));
> + sprintf(command, command_format, identifier);
> + f = popen(command, POPEN_MODE);
