Skip to content

tools/testbuild.sh: update makefunc to hardcode make/extra flags opti…#792

Merged
yamt merged 1 commit intoapache:masterfrom
liuguo09:testbuild
Apr 15, 2020
Merged

tools/testbuild.sh: update makefunc to hardcode make/extra flags opti…#792
yamt merged 1 commit intoapache:masterfrom
liuguo09:testbuild

Conversation

@liuguo09
Copy link
Copy Markdown
Contributor

…ons and redirection

Aim to let makefunc caller just pass in the target. So update makefunc to hardcode make/extra
flags options and redirection, meanwhile avoid the case in which makefunc would remove the quotes
for EXTRAFLAGS. Note that JOPTIONS should also hardcode once parallel build break finally fixed.

Signed-off-by: liuhaitao liuhaitao@xiaomi.com

…ons and redirection

Aim to let makefunc caller just pass in the target. So update makefunc to hardcode make/extra
flags options and redirection, meanwhile avoid the case in which makefunc would remove the quotes
for EXTRAFLAGS. Note that JOPTIONS should also hardcode once parallel build break finally fixed.

Signed-off-by: liuhaitao <liuhaitao@xiaomi.com>
Copy link
Copy Markdown
Contributor

@yamt yamt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@yamt yamt merged commit ac18fc0 into apache:master Apr 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants