已关闭
统一各仓 enable_hilog 判断条件到 build 仓 #12262
ElevenDuan创建于  7月13日关闭于  22 天前
ElevenDuan
ElevenDuan
7月13日 创建

问题

enable_hilog 在 arkcompiler 各仓(ets_runtime、ets_frontend、runtime_core、toolchain)中分别定义,判断条件不一致:

  • ets_runtime 的 enable_hilog 已包含 !is_qemu_runtime(及 !disable_hilog)条件;
  • ets_frontend、runtime_core、toolchain 三仓的 enable_hilog 判断条件中缺少 is_qemu_runtime

导致 qemu runtime 构建时,后三仓仍会启用 hilog,行为与 ets_runtime 不一致。

is_qemu_runtime / disable_hilog 当前仅在 ets_runtime 的 js_runtime_config.gni 中声明(declare_args),其它仓不可见。

方案

enable_hilog 的判断条件统一收敛到 build 仓 build/config/components/runtime_core/hilog_config.gni

  • 集中声明 is_qemu_runtime / disable_hilog(declare_args);
  • 提供两套共享值(保留原有两套语义):
    • enable_hilog_broad:ohos 设备 + windows/mac 预览器,供 ets_runtime、toolchain;
    • enable_hilog_device:ohos 标准系统 arm/arm64,供 runtime_core、ets_frontend;
  • 两套均包含 !is_qemu_runtime && !disable_hilog

各仓的 *_config.gni 改为 import 该共享定义(standalone 构建保留本地回退,恒为 false)。

涉及仓库

  • build:新增 build/config/components/runtime_core/hilog_config.gnistatic_core/ark_common_config.gni 引用共享定义。
  • arkcompiler_ets_runtime:js_runtime_config.gni
  • arkcompiler_ets_frontend:ark_config.gni
  • arkcompiler_runtime_core:ark_config.gnistatic_core/ark_config.gni
  • arkcompiler_toolchain:toolchain_config.gni

依赖顺序:build 仓需先合入,其余四仓 import 其 hilog_config.gni

likedislike
openharmony_ci
openharmony_ci成员
7月13日 评论:

感谢提交Issue!关于Issue的交互操作,请访问OpenHarmony社区支持命令清单。如果有问题,请联系 [@weng-changcheng](https://gitcode.com/weng-changcheng) [@godmiaozi](https://gitcode.com/godmiaozi) [@liyiming13](https://gitcode.com/liyiming13) [@lijin1039](https://gitcode.com/lijin1039) [@Prof](https://gitcode.com/Prof) [@dingding5](https://gitcode.com/dingding5) [@vpukhov](https://gitcode.com/vpukhov) [@v-cherkashin](https://gitcode.com/v-cherkashin) 。如果需要调整订阅PR、Issue的变更状态,请访问链接


Thanks for submitting the issue. For more commands, please visit OpenHarmony Command List. If you have any questions, please refer to committer gitcode for help. If you need to change the subscription of a Pull Request or Issue, please visit the link.

likedislike
openharmony_ciopenharmony_ci成员
7月13日 添加了label:waiting_for_assign
ElevenDuanElevenDuan
7月13日 关联了pull request:Unify enable_hilog build condition
ElevenDuanElevenDuan
7月13日 关联了pull request:Unify enable_hilog build condition
ElevenDuanElevenDuan
7月13日 关联了pull request:Unify enable_hilog build condition
ElevenDuanElevenDuan
7月13日 关联了pull request:Unify enable_hilog build condition
ElevenDuanElevenDuan
7月13日 关联了pull request:Unify enable_hilog build condition
ohci1ohci1成员
22 天前 关闭了 issue
ohci1ohci1成员
22 天前 issue状态由 待办的 改变为 已完成
Lyupa AnastasiaLyupa Anastasia
22 天前 关联了pull request:Unify enable_hilog build condition