RFR: 7903920: Support building jtreg on FreeBSD

Magnus Ihse Bursie ihse at openjdk.org
Sat Jan 25 20:40:58 UTC 2025


On Sat, 25 Jan 2025 20:37:41 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:

>> 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.
>
> If the path is hardcoded as e.g. `/usr/bin/foo`, I tend to agree with you. That means that an official system tool is needed and confirmed working. However, hard-coding `/usr/local/bin/foo` is another matter altogether. It means there is no standard system tool that is working, and we presume the user has installed a "correct" tool locally.

If the path is hardcoded as e.g. `/usr/bin/foo`, I tend to agree with you. That means that an official system tool is needed and confirmed working. However, hard-coding `/usr/local/bin/foo` is another matter altogether. It means there is no standard system tool that is working, and we presume the user has installed a "correct" tool locally.

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

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


More information about the jtreg-dev mailing list