已合并
[feature] 支持Unix Makefiles构建 #51
wiyr0创建于 3月24日
[feature] 支持Unix Makefiles构建 #51
已合并
wiyr0创建于 3月24日
wiyr0
wiyr0成员
3月24日

1. 修改描述

  • 修改原因: 当前编译要求msdebug一定要用ninja版本
  • 修改方案: msdebug实际上可以用make编译的,这里优先使用ninja,如果不存在ninja就使用make
  • 修改内容:
  1. CMakelists.txt里如果检测到当前是用ninja编译的,先判断ninja是否存在以及版本正确性,否则报错
  2. build.py里判断当前ninja是否存在,否则就用Unix Makefiles

2. 功能验证

ninja不存在:
image.png
image.png


3. 代码检视

  • 要求:
    • 合入功能代码大于 200 行,需要sig会议申报代码检视议题,并在PR中标注会议。
    • committer评估是否需要在sig会议进行代码检视。
    • 参与检视的committer人员名单与检视时间。
    • 大于 1000 行代码原则上不允许合入,需进行备案。
  • 检视committer人员名单与检视时间:

4. 资料修改自检

  • 资料修改: NA

likedislike
Pull Request已成功合入, 合并人@ascend-robot
(感谢 wiyr0 的贡献)
ascend-robotascend-robot成员
3月24日 添加了label:stat/needs-squash
ascend-robot
ascend-robot成员
3月24日 评论:

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/msdebug gong-siwei, cycoe, went_code (3/2) gong-siwei (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

wiyr0, thanks for your pull request. All authors of the commits have signed the CLA. 👍

likedislike
ascend-robotascend-robot成员
3月24日 添加了label:ascend-cla/yes
ascend-robot
ascend-robot成员
3月24日 评论:

Ascend docs pipeline is running...

likedislike
ascend-robotascend-robot成员
3月24日 添加了label:docs-ci-pipeline-running
此处折叠了62条消息 查看更多
ascend-robotascend-robot成员
3月24日 添加了label:lgtm
gong-siwei成员
3月25日 评论:

/lgtm
/approve

likedislike
ascend-robotascend-robot成员
3月25日 添加了label:approved
ascend-robot
ascend-robot成员
3月25日 评论:

Review Guide

This pull-request passes review.
Committers who wrote a comment of /approve are: gong-siwei.
Reviewers who wrote a comment of /lgtm are: cycoe, gong-siwei, went_code.

likedislike
ascend-robotascend-robot成员
3月25日 合入了pull request