已开启
build: 支持多框架版本编译 #20
build: 支持多框架版本编译 #20
已开启
linsanjiu创建于 19 天前
linsanjiu
linsanjiu
19 天前

根据README版本表,将buildEnv.sh的FLUTTER_SDK_VERSION由单版本改为多框架版本数组,支持多个Flutter框架版本门禁编译。

Signed-off-by: xileilei xi_leilei@hoperun.com

likedislike
合并受阻
openharmony_ci
openharmony_ci成员
19 天前 评论:

感谢提交 Pull Requests!
Thanks for submitting a pull request.

likedislike
openharmony_ciopenharmony_ci成员
19 天前 添加了label:dco检查成功
cpf-manager
cpf-manager成员19 天前进行代码检视1
buildEnv.sh
@@ -1,2 +1,3 @@
11#!/bin/bash
22BUILD_PKG_DIR=example
3-FLUTTER_SDK_VERSION=3.22.0-ohos
3+FLUTTER_SDK_VERSION=("br_3.7.12-ohos-1.1.3" "oh-3.22.3-release" "oh-3.27.0-release" "oh-3.35.7-release")
cpf-manager
cpf-manager19 天前评论:

【AI-Review】【严重】【基础代码问题】【代码逻辑错误】FLUTTER_SDK_VERSION 由标量改为数组,与同组织 20 仓标量约定不一致,门禁可能静默跳过 3 个版本

● 问题:buildEnv.sh 第 3 行将 FLUTTER_SDK_VERSION 从标量字符串 3.22.0-ohos 改为 bash 数组 ("br_3.7.12-ohos-1.1.3" "oh-3.22.3-release" "oh-3.27.0-release" "oh-3.35.7-release")。经核查 CPF-Flutter 组织下 20 个同类插件仓(fluttertpc_add_2_calendar、fluttertpc_tts_plugin、fluttertpc_velocity_x、fluttertpc_webview_cookie_manager、fluttertpc_super_native_extensions、fluttertpc_flutter_yfree 等)的 buildEnv.sh,该变量一致使用标量格式(如 FLUTTER_SDK_VERSION=oh-3.35.7-devFLUTTER_SDK_VERSION=3.22.0-ohos),本仓为首个改为数组格式的仓库。bash 中对数组变量进行标量引用 $FLUTTER_SDK_VERSION 只返回首元素 br_3.7.12-ohos-1.1.3(已通过本地 source 脚本实测验证)。该变量在本仓内仅被定义、无任何内部消费方(已对全仓 grep 确认),实际消费方为外部门禁 CI(tester 为 openharmony_ci)。

● 影响:严重。若门禁 CI 以 source buildEnv.sh 加载并按标量 $FLUTTER_SDK_VERSION 读取(与其余 20 仓的处理方式一致),则 oh-3.22.3-releaseoh-3.27.0-releaseoh-3.35.7-release 三个版本将被静默跳过,PR 声明的“支持多个Flutter框架版本门禁编译”目的无法达成,且无任何报错提示,问题难以被发现。

● 建议:提交前确认门禁 CI 是否通过 "${FLUTTER_SDK_VERSION[@]}" 遍历数组。若门禁尚未支持数组遍历,应先升级门禁脚本再合入此变更;若门禁已支持,建议在 buildEnv.sh 中补充注释说明该变量为数组、需通过 ${FLUTTER_SDK_VERSION[@]} 消费,避免后续维护者按标量误用。

likedislike