diff --git a/hb_internal/build/build.py b/hb_internal/build/build.py index f0b70f6177fb5cdcf1e284ab45c8cfe418e794dd..e10ecf5d76d75a85b85e5d27564db5147fc96140 100644 --- a/hb_internal/build/build.py +++ b/hb_internal/build/build.py @@ -131,7 +131,7 @@ def add_options(parser): default='default') parser.add_argument('--build-variant', help='specifies device operating mode', - default='user') + default='root') parser.add_argument('--share-ccache', default="", help='It is customized path to place ccache, which allow' diff --git a/hb_internal/build/build_process.py b/hb_internal/build/build_process.py index c2fbaec19f1f8a82ac5b109d7dc981fb48ffe822..c8e9f7a3a3cdf37b834f92bc191a579bd5a402bf 100644 --- a/hb_internal/build/build_process.py +++ b/hb_internal/build/build_process.py @@ -158,7 +158,6 @@ class Build(): else: if not cmd_args.get('disable_post_build'): post_build = PostBuild(self.config) - post_build.patch_ohos_para(cmd_args) if not cmd_args.get('disable_package_image'): post_build.package_image() if not disable_post_build_args or 'output_part_rom_status' not in disable_post_build_args: @@ -236,6 +235,8 @@ class Build(): os_level = self.config.os_level if cmd_args.get('build_variant'): self.register_args('build_variant', cmd_args.get('build_variant')) + if cmd_args.get('device_type'): + self.register_args('device_type', cmd_args.get('device_type')) gn_cmd = [ gn_path, 'gen', diff --git a/hb_internal/common/misc.py b/hb_internal/common/misc.py index d239a35e8df184f62b2eba2f0b24b6d1db048691..ce1d1642731180329fcc260c053db708df86493e 100644 --- a/hb_internal/common/misc.py +++ b/hb_internal/common/misc.py @@ -20,7 +20,7 @@ import os import subprocess from datetime import datetime from distutils.spawn import find_executable -from hb_internal.common.utils import OHOSException, exec_command +from hb_internal.common.utils import exec_command from hb_internal.common.utils import hb_warning @@ -108,39 +108,6 @@ class PostBuild: self.get_warning_list() if not disable_post_build_args or 'compute_overlap_rate' not in disable_post_build_args: self.compute_overlap_rate() - - def patch_ohos_para(self, cmd_args): - ohos_para_data = [] - ohos_para_file_path = os.path.join(self._out_path, 'packages/phone/system/etc/param/ohos.para') - if not os.path.exists(ohos_para_file_path): - return - with open(ohos_para_file_path, 'r', encoding='utf-8') as ohos_para_file: - for line in ohos_para_file: - ohos_para_data.append(line) - if cmd_args.get('device_type') and cmd_args.get('device_type') != 'default': - for i, line in enumerate(ohos_para_data): - if ohos_para_data[i].__contains__('const.build.characteristics'): - ohos_para_data[i] = 'const.build.characteristics=' + cmd_args.get('device_type') + '\n' - break - if cmd_args.get('build_variant'): - if cmd_args.get('build_variant') != 'user' and cmd_args.get('build_variant') != 'root': - raise OHOSException('Error: --build-variant support user or root mode currently') - for i, line in enumerate(ohos_para_data): - if ohos_para_data[i].__contains__('const.secure'): - if cmd_args.get('build_variant') == 'user': - ohos_para_data[i] = 'const.secure=1\n' - else: - ohos_para_data[i] = 'const.secure=0\n' - if ohos_para_data[i].__contains__('const.debuggable'): - if cmd_args.get('build_variant') == 'user': - ohos_para_data[i] = 'const.debuggable=0\n' - else: - ohos_para_data[i] = 'const.debuggable=1\n' - data = '' - for line in ohos_para_data: - data += line - with open(ohos_para_file_path, 'w', encoding='utf-8') as ohos_para_file: - ohos_para_file.write(data) def package_image(self): image_path = os.path.join(self._out_path, 'packages/phone/images/')