RFR: 8233115: Protect ExecuteWithLog from running with redirection without a subshell [v3]

Magnus Ihse Bursie ihse at openjdk.org
Fri Sep 5 08:55:27 UTC 2025


On Thu, 4 Sep 2025 16:54:27 GMT, Erik Joelsson <erikj at openjdk.org> wrote:

>> Magnus Ihse Bursie has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - Break all lines to fit in 80 chars
>>  - Break long lines
>
> make/common/MakeBase.gmk line 306:
> 
>> 304:   $(call MakeDir, $(dir $(strip $1)) $(MAKESUPPORT_OUTPUTDIR)/failure-logs) \
>> 305:   $(call WriteFile, $2, $(strip $1).cmdline) \
>> 306:   ( $(RM) $(strip $1).log && $(if $(call is_redirect, $2),$(LEFT_PAREN) )$(strip $2)$(if $(call is_redirect, $2), $(RIGHT_PAREN)) > >($(TEE) -a $(strip $1).log) 2> >($(TEE) -a $(strip $1).log >&2) || \
> 
> Consider breaking these lines.

Good point. I turned all of them into 80 char max. It's both helpful and hurtful for readability, but let's stick to our standard.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27096#discussion_r2324505388


More information about the build-dev mailing list