On 2018/04/09 12:14, Gonzalo L. Rodriguez wrote:
> On [08/04/18] [10:30P], Stuart Henderson wrote:
> > CVSROOT:    /cvs
> > Module name:        ports
> > Changes by: st...@cvs.openbsd.org   2018/04/08 04:30:32
> > 
> > Modified files:
> >     sysutils/apachetop: Makefile
> > Added files:
> >     sysutils/apachetop/patches: patch-src_apachetop_cc
> >                                 patch-src_display_cc
> >                                 patch-src_hits_circle_cc
> >                                 patch-src_log_cc
> > 
> > Log message:
> > add clang6 patches from Matthew Martin
> > 
> > update homepage while there, someone should look at updating this,
> > the newest release (2017-04) includes a buffer overflow fix
> > 
> 
> Yes, I was working on that.
> 
> Moving to git, and update to the last version with the clang6 fix.
> 
> Diff attached.
> 
> OK? Comments?
> 
> Cheers.-
> 
> -- 
> Sending from my toaster.

> Index: Makefile
> ===================================================================
> RCS file: /cvs/ports/sysutils/apachetop/Makefile,v
> retrieving revision 1.11
> diff -u -p -r1.11 Makefile
> --- Makefile  8 Apr 2018 10:30:32 -0000       1.11
> +++ Makefile  9 Apr 2018 10:13:05 -0000
> @@ -2,8 +2,9 @@
>  
>  COMMENT =            top-like monitor for Apache
>  
> -DISTNAME =           apachetop-0.12.6
> -REVISION =           3
> +GH_ACCOUNT =         tessus
> +GH_PROJECT =         apachetop
> +GH_TAGNAME =         0.17.4
>  CATEGORIES =         sysutils
>  
>  MAINTAINER =         Gonzalo L. R. <gonz...@openbsd.org>
> @@ -15,12 +16,16 @@ PERMIT_PACKAGE_CDROM=     Yes
>  
>  MASTER_SITES =               http://www.webta.org/apachetop/

It doesn't fetch like this.

apachetop-0.17.4.tar.gz isn't present at http://www.webta.org/apachetop/
and setting MASTER_SITES here overrides the automatic MASTER_SITES setting
used for GH_*.

But there is a proper uploaded release on github, so please use that instead.
You can use something like

V=              0.17.4
DISTNAME=       apachetop-$V
MASTER_SITES=   https://github.com/tessus/apachetop/releases/download/$V/

That also has autoconf files generated so you should be able to go to
CONFIGURE_STYLE=gnu and drop the autoconf dep.

Reply via email to