You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by la...@apache.org on 2023/06/13 05:43:36 UTC
[kudu] 14/15: Make dep_extract.py work with Python3
This is an automated email from the ASF dual-hosted git repository.
laiyingchun pushed a commit to branch branch-1.17.x
in repository https://gitbox.apache.org/repos/asf/kudu.git
commit 9fff4e3ed9f9858f7d6f69a986adbec46a66544d
Author: Marton Greber <gr...@gmail.com>
AuthorDate: Sun Jun 11 12:50:58 2023 +0000
Make dep_extract.py work with Python3
build_mini_cluster_binaries.sh does not work on RHEL9 at the moment, as
dep_extract.py - an underlying script - is not comaptible with Python3.
This issue came up as RHEL9 ships only with Python3.
Change-Id: I2f2534155cb88efa16cac02adf70ee795bf2a428
Reviewed-on: http://gerrit.cloudera.org:8080/20035
Tested-by: Kudu Jenkins
Reviewed-by: Yingchun Lai <la...@apache.org>
Reviewed-by: Zoltan Chovan <zc...@cloudera.com>
(cherry picked from commit 14c326461c2132d74706b284a2eee827ace73af1)
Reviewed-on: http://gerrit.cloudera.org:8080/20060
---
build-support/dep_extract.py | 3 +++
1 file changed, 3 insertions(+)
diff --git a/build-support/dep_extract.py b/build-support/dep_extract.py
index ba454b635..b2515f20c 100644
--- a/build-support/dep_extract.py
+++ b/build-support/dep_extract.py
@@ -21,6 +21,7 @@ import logging
import os
import re
import subprocess
+import sys
# Matches the output lines from the 'ldd' tool. For example:
# libcrypto.so.10 => /path/to/usr/lib64/libcrypto.so.10 (0x00007fb0cb0a5000)
@@ -105,6 +106,8 @@ class DependencyExtractor(object):
deps = []
for line in out.splitlines():
+ if sys.version_info.major >= 3:
+ line = line.decode('utf-8')
match = LDD_RE.match(line)
if not match:
continue