已合并
[msprof]msprof的run包增加--uninstall参数 #84
yuliangbin创建于 2月26日
[msprof]msprof的run包增加--uninstall参数 #84
已合并
Pull Request已成功合入, 合并人@ascend-robot
（感谢 yuliangbin 的贡献）
ascend-robot
2月26日 评论：
2月26日 评论：
ascend-robot
2月26日 评论：
2月26日 评论：
Thanks for your pull-request.
The full list of commands accepted by me can be found at here。
You can get sig-info at here
PR Approval Progress
✅ Congratulations! All modules have met the lgtm and approve requirements.
Module Approval Details
| module | lgtm status | approve status |
|---|---|---|
| repo-Ascend/msprof | ✅ Seanesmhxocism, fanhong (2/2) | ✅ fanhong (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
2月26日 添加了label：ascend-cla/yes
yuliangbin
2月26日 评论：
2月26日 评论：
compile
2月26日 添加了label：ci-pipeline-running
2月26日 删除了label：ci-pipeline-running
2月26日 添加了label：ci-pipeline-passed
ascend-robot
2月26日 评论：
2月26日 评论：
2月26日 强制推送 1 个提交:7c8a9e17-add uninstall sh
2月26日 删除了label：ci-pipeline-passed
ascend-robot
2月26日 评论：
2月26日 评论：
yuliangbin
2月26日 评论：
2月26日 评论：
compile
2月26日 添加了label：ci-pipeline-running
2月26日 强制推送 1 个提交:fd33d853-add uninstall sh
yuliangbin
2月26日 评论：
2月26日 评论：
compile
2月26日 删除了label：ci-pipeline-running
ascend-robot
2月26日 评论：
2月26日 评论：
2月26日 添加了label：ci-pipeline-running
此处折叠了5条事件消息 查看更多
2月26日 添加了label：ci-pipeline-passed
ascend-robot
2月26日 评论：
2月26日 评论：
chenhao_1209
2月26日 评论：
2月26日 评论：
/approve
2月26日 添加了label：approved
scripts/run_script/install.sh
已过期
@@ -142,3 +165,4 @@
142165chmod_ini_file
143166set_libmsprofiler_right
144167chmod_libmsprofiler
[review] 缩进问题，建议和其他的保持一致，避免格式不统一。
系统消息
2月26日 评论：
2月26日 评论：
yuliangbin
2月26日 评论：
2月26日 评论：
2月26日 强制推送 1 个提交:77669aac-add uninstall sh
2月26日 删除了label：ci-pipeline-passed
2月26日 删除了label：approved
ascend-robot
2月26日 评论：
2月26日 评论：
ascend-robot
2月26日 评论：
2月26日 评论：
2月26日 强制推送 1 个提交:20d211ab-add uninstall sh
2月26日 强制推送 1 个提交:8c0bbad1-add uninstall sh
ascend-robot
2月26日 评论：
2月26日 评论：
yuliangbin
2月26日 评论：
2月26日 评论：
compile
2月26日 解决了最后一个问题
2月26日 添加了label：ci-pipeline-running
2月26日 删除了label：ci-pipeline-running
2月26日 添加了label：ci-pipeline-passed
ascend-robot
2月26日 评论：
2月26日 评论：
wangzixuan
2月26日 评论：
2月26日 评论：
/lgtm
xfeng
2月26日 评论：
2月26日 评论：
[review] [建议] [功能问题]
问题：验证不充分
原因：需要验证一下是否注册成功，可以通过查看cann_install.sh里面有没有新增修改；需要手动执行一下share/info/msprof/script下面的uninstall.sh脚本
将截图补充到描述信息中
2月26日 修改了pull request 的描述
fanhong
2月26日 评论：
2月26日 评论：
/approve
2月26日 添加了label：approvedlgtm
ascend-robot
2月26日 评论：
2月26日 评论：
Review Guide
This pull-request passes review.
Committers who wrote a comment of /approve are: fanhong.
Reviewers who wrote a comment of /lgtm are: fanhong, Seanesmhxocism.
2月26日 合入了pull request
PR 合入模板
注：经过自检不涉及的可标注“不涉及”或直接打勾，特殊情况请文字备注。不符合规范的 PR 不允许合入，请（后备）commit 注意。
1. 修改描述
需要新增卸载脚本、卸载命令行适配CANN整包打包
2. 功能验证
3. 分支合并要求
3. 代码检视
要求：
是否经过代码检视
是否具备 UT 测试用例看护 （如不符合，请说明原因：____________________）
检视意见数：____ 条 （请填写本次检视的意见总数，用于commit合入前审视）
4. 安全自检
Python、C++：
C++：
5. 变更知会