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