Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 3 Dec 2017 20:32:49 +0100
From:      =?UTF-8?Q?Ren=c3=a9_Ladan?= <rene@freebsd.org>
To:        Gerald Pfeifer <gerald@pfeifer.com>, java@FreeBSD.org, Joe Nosay <superbisquit@gmail.com>
Subject:   Re: Remove dead code from java/cacao/Makefile
Message-ID:  <09dc739b-fef9-ca35-f930-0e9ca9720b23@freebsd.org>
In-Reply-To: <alpine.LSU.2.21.1712030948580.2983@anthias.pfeifer.com>
References:  <alpine.LSU.2.21.1705142235120.4477@anthias.pfeifer.com> <CA%2BWntOu5d7avfVhoa-Y2a7dghQBHU8PizyRsjN1eCHe49Sz6QA@mail.gmail.com> <alpine.LSU.2.21.1705280131360.3529@anthias.pfeifer.com> <alpine.LSU.2.21.1712030948580.2983@anthias.pfeifer.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On 03-12-2017 18:04, Gerald Pfeifer wrote:
> On Sun, 28 May 2017, Gerald Pfeifer wrote:
>> I did not propose to remove that option, I merely wanted to relay
>> that (a) currently this alternative codepath is simply broken and
>> (b) remove three lines of dead code at least.
> Let me follow-up on this again.
>
> Rene, I saw that you recently updated java/cacao.  Can you approve?
You would have to ask java@, I just updated the port to use a supported
version of bootstrap-openjdk and then fixed my previous commit.
> I verified that WITH_GCC has exactly two occurrences in the _entire_
> Ports tree, and that is this dead code in java/cacao/Makefile.
>
> Any real objections towards removing this _dead_ code?
So WITH_GCC is not defined anywhere in usr/ports/Mk/ nor in lang/gcc*,
which would indeed render the if statement useless. So yes, the change
should be fine. You could also remove the GCC_SUFX?=46 line?
> Gerald
>
> PS: There is more, but let's focus on this most straightforward one
> for now where I'm surprised (a) nobody else cared and (b) even this 
> was hard to get in.
>
> Index: java/cacao/Makefile
> ===================================================================
> --- java/cacao/Makefile	(revision 455410)
> +++ java/cacao/Makefile	(working copy)
> @@ -37,9 +37,6 @@
>  JAR=		${LOCALBASE}/bootstrap-openjdk6/bin/jar
>  JAVAC=		${LOCALBASE}/bootstrap-openjdk6/bin/javac
>  .else
> -.if defined(WITH_GCC)
> -GCC_SUFX=	${WITH_GCC:S/.//}
> -.endif
>  GCC_SUFX?=	46
>  BUILD_DEPENDS+=	gcj${GCC_SUFX}:lang/gcc${GCC_SUFX}
>  EXTRA_PATCHES=	${FILESDIR}/gcj.patch
>
René




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?09dc739b-fef9-ca35-f930-0e9ca9720b23>