-
Notifications
You must be signed in to change notification settings - Fork 3.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[C++] Remove Perl dependency from cpp/build-support/run-test.sh #35361
Comments
kou
added a commit
to kou/arrow
that referenced
this issue
Apr 28, 2023
…un-test.sh We can re-implement the addr2line filter in other language such as Python or shell if we still need it.
kou
added a commit
that referenced
this issue
Apr 28, 2023
…t.sh (#35362) ### Rationale for this change Because Perl in conda doesn't work on Manjaro Linux. ### What changes are included in this PR? * Use `sed` instead of `perl`. * Remove `stacktrace_addr2line.pl`. If we still need it, we can re-implement the addr2line filter in other language such as Python or shell later. ### Are these changes tested? Yes by hand on Manjaro Linux. ### Are there any user-facing changes? No. * Closes: #35361 Authored-by: Sutou Kouhei <kou@clear-code.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
liujiacheng777
pushed a commit
to LoongArch-Python/arrow
that referenced
this issue
May 11, 2023
…un-test.sh (apache#35362) ### Rationale for this change Because Perl in conda doesn't work on Manjaro Linux. ### What changes are included in this PR? * Use `sed` instead of `perl`. * Remove `stacktrace_addr2line.pl`. If we still need it, we can re-implement the addr2line filter in other language such as Python or shell later. ### Are these changes tested? Yes by hand on Manjaro Linux. ### Are there any user-facing changes? No. * Closes: apache#35361 Authored-by: Sutou Kouhei <kou@clear-code.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
ArgusLi
pushed a commit
to Bit-Quill/arrow
that referenced
this issue
May 15, 2023
…un-test.sh (apache#35362) ### Rationale for this change Because Perl in conda doesn't work on Manjaro Linux. ### What changes are included in this PR? * Use `sed` instead of `perl`. * Remove `stacktrace_addr2line.pl`. If we still need it, we can re-implement the addr2line filter in other language such as Python or shell later. ### Are these changes tested? Yes by hand on Manjaro Linux. ### Are there any user-facing changes? No. * Closes: apache#35361 Authored-by: Sutou Kouhei <kou@clear-code.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
rtpsw
pushed a commit
to rtpsw/arrow
that referenced
this issue
May 16, 2023
…un-test.sh (apache#35362) ### Rationale for this change Because Perl in conda doesn't work on Manjaro Linux. ### What changes are included in this PR? * Use `sed` instead of `perl`. * Remove `stacktrace_addr2line.pl`. If we still need it, we can re-implement the addr2line filter in other language such as Python or shell later. ### Are these changes tested? Yes by hand on Manjaro Linux. ### Are there any user-facing changes? No. * Closes: apache#35361 Authored-by: Sutou Kouhei <kou@clear-code.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the enhancement requested
Perl may not be so portable as before
See also:
https://lists.apache.org/thread/ys0w82slp3d7w3ngprqnqjlwmzf5brqc
https://lists.apache.org/thread/w47852gv4h9ngkjdnfhnmnqnx6km5nlb
Component(s)
C++
The text was updated successfully, but these errors were encountered: