diff options
author | Alan Wu <XrXr@users.noreply.github.com> | 2022-04-28 17:52:21 -0400 |
---|---|---|
committer | Alan Wu <XrXr@users.noreply.github.com> | 2022-04-28 19:02:26 -0400 |
commit | 99281f513cf7d4873d2f672860f80e063f98b7cb (patch) | |
tree | 8f311ffff22a623338d9d694a9afa765afe4cd31 /test/-ext- | |
parent | 907b31d7b92fd0eb98507dca6411d07da0fecb46 (diff) | |
download | ruby-99281f513cf7d4873d2f672860f80e063f98b7cb.tar.gz |
YJIT: Handle spaces in build dir path using find(1) `-exec`
Previously, we relied on shell word splitting, which leads
to passing the wrong arguments when there are white spaces
in the path.
Avoiding command substitution also makes this script more
likely to work under Solaris 10, where `/bin/sh` is not
POSIX compliant [1]. (Thanks you, `@znz` for fixing the syntax
error in 4210ae2158b545beda908fb29e03d23994f262e3 though!)
The hack from c466f270b891962518763ad299f907beac0ebf62
doesn't actually work so this commit reverts it. The shell
still needs to parse through all of the code, maybe because
make doesn't in fact send newlines to the shell.
By the way, we also use the `-exec` option in the `ext/distclean` task.
[1]: https://docs.oracle.com/cd/E26505_01/html/816-5165/sh-1.html
Diffstat (limited to 'test/-ext-')
0 files changed, 0 insertions, 0 deletions