On Tue, May 07, 2019 at 06:08:00PM -0400, Jeremie Courreges-Anglas wrote:
> On Tue, May 07 2019, Stuart Henderson <[email protected]> wrote:
> > On 2019/05/07 13:29, Ian Darwin wrote:
> >> CVSROOT:   /cvs
> >> Module name:       ports
> >> Changes by:        [email protected]    2019/05/07 13:29:41
> >> 
> >> Removed files:
> >>    java/junit     : Makefile distinfo 
> >>    java/junit/pkg : DESCR PLIST 
> >> 
> >> Log message:
> >> Remove junit, no benefit to packaging, OK jca@
> >> 
> >
> > geo/jeoip still uses this
> 
> Indeed, boo. :-/
> 
> I proposed this diff to Ian, we'll see how he prefers to tackle this.

It's weird to have both TEST_DEPENDS and NO_TEST, no?

> Index: Makefile
> ===================================================================
> RCS file: /cvs/ports/geo/jeoip/Makefile,v
> retrieving revision 1.28
> diff -u -p -r1.28 Makefile
> --- Makefile  24 Mar 2019 22:24:13 -0000      1.28
> +++ Makefile  7 May 2019 21:02:08 -0000
> @@ -20,14 +20,14 @@ MODJAVA_VER=              1.8+
>  MODJAVA_BUILD=               ant
>  MODJAVA_BUILD_TARGET_NAME =  jar docs
>  
> -BUILD_DEPENDS=               java/junit
> -TEST_DEPENDS=                net/GeoIP,-db \
> -                     java/junit
> +TEST_DEPENDS=                net/GeoIP,-db
>  RUN_DEPENDS=         net/GeoIP,-db \
>                       java/javaPathHelper
>  MAKE_ENV=            
> CLASSPATH=${LOCALBASE}/share/java/classes/junit/junit.jar
>  
>  WRKDIST=             ${WRKDIR}/jeoip
> +
> +NO_TEST=             Yes
>  
>  do-install:
>       ${INSTALL_DATA_DIR} ${MODJAVA_JAR_DIR}
> 
> 
> -- 
> jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF  DDCC 0DFA 74AE 1524 E7EE
> 

-- 
Antoine

Reply via email to