RFR: 7903920: Support building jtreg on FreeBSD

Jonathan Gibbons jjg at openjdk.org
Fri Jan 24 23:04:01 UTC 2025


On Thu, 23 Jan 2025 20:06:27 GMT, snake66 <duke at openjdk.org> wrote:

>> Okay, people have apparently varying opinions on this. :) Don't want to put you in a "damned if you do, damned if you don't" situation, so just ignore my comment and go with your solution.
>
> I don't have any strong opinions about this, but for FreeBSD's part it would actually be slightly better to let the shell find the binaries in $PATH. But it's not important for my progress now, so I figured I'd deal with it if it comes up :)

Maybe the concern is becoming obsolete, but the general design focus of this and similar scripts was to lock down everything, and avoid any variation that might occur as a result of individuals setting non-standard values for PATH and similar path-related variables -- CLASSPATH anyone?

But, a reasonable compromise would be to limit the use of `which` to this file (`Defs.gmk`) and maybe to within suitable `ifeq ($(SYSTEM_UNAME), FreeBSD)` constructs.

That all being said, I would defer to the expertise and style-advice of folk like @magicus on the Build team,  but with some amount of caveat for overall style consistency, as much as is reasonable.

-------------

PR Review Comment: https://git.openjdk.org/jtreg/pull/237#discussion_r1929341033


More information about the jtreg-dev mailing list