Repository navigation
Conversation
|
👋 感谢您对 RT-Thread 的贡献！Thank you for your contribution to RT-Thread! 为确保代码符合 RT-Thread 的编码规范，请在你的仓库中执行以下步骤运行代码格式化工作流（如果格式化CI运行失败）。 🛠 操作步骤 | Steps
完成后，提交将自动更新至 如有问题欢迎联系我们，再次感谢您的贡献！💐 |
|
…failure on rebuild
|
ci有报错 |
|
tm4c123bsp的Sconstruct脚本中存留dist脚本导出的几行代码，已经删除:如下； # set RTT_ROOT
#if not os.getenv("RTT_ROOT"):
#RTT_ROOT="rt-thread"并修改本次pr的mkdist.py脚本，仅匹配未注释的脚本如下： #set RTT_ROOT
if not os.getenv("RTT_ROOT"):
RTT_ROOT="rt-thread" |
|
有个问题，为什么不直接删掉然后在dist出来一份。我认为dist出来这个工程是为了方便开发。这个文件应该不保留在原本的dist目录下了。其次这个文件如果被进程占用，也是删不掉的，只清理只读位应该也没有作用。 |
我看了这个脚本的代码，原先的意思就是如果dist目录下存在project，那么就优先删除project重新dist出来一份。 |
|
下述代码先查看要copy的存在不存在，不存在直接返回，之后看目标目录存不存在要copy的文件夹，如果存在，那么就先删除，如果报错了就不删除以及直接返回后续copy也不直接进行； def do_copy_folder(src_dir, dst_dir, ignore=None):
import shutil
# check source directory
if not os.path.exists(src_dir):
return
try:
if os.path.exists(dst_dir):
shutil.rmtree(dst_dir)
except:
print('Deletes folder: %s failed.' % dst_dir)
return
shutil.copytree(src_dir, dst_dir, ignore = ignore)但是，项目中有些文件是只读文件：
def do_copy_folder(src_dir, dst_dir, ignore=None):
import shutil
# check source directory
if not os.path.exists(src_dir):
return
try:
if os.path.exists(dst_dir):
shutil.rmtree(dst_dir)
except Exception as e:
print('Deletes folder: %s failed: %s\n' % (dst_dir, e))
return
shutil.copytree(src_dir, dst_dir, ignore = ignore)
导致的结果是，第二次dist因为
成功删除只读文件，从而可以正常进行后续copy。 |
|
其次修改 bsp_update_sconstruct函数是因为 def bsp_update_sconstruct(dist_dir):
with open(os.path.join(dist_dir, 'SConstruct'), 'r') as f:
data = f.readlines()
with open(os.path.join(dist_dir, 'SConstruct'), 'w') as f:
for line in data:
if line.find('RTT_ROOT') != -1:
if line.find('sys.path') != -1:
f.write('# set RTT_ROOT\n')
f.write('if not os.getenv("RTT_ROOT"): \n RTT_ROOT="rt-thread"\n\n')
f.write(line)该函数会检测 # set RTT_ROOT
if not os.getenv("RTT_ROOT"):
RTT_ROOT="rt-thread"这就会导致该代码块会随之dist的次数增加而增加 
|
拉取/合并请求描述：(PR description)
[
为什么提交这份PR (why to submit this PR)
scons --dist在同一目录上重复执行、或目标 dist 目录里残留只读文件时，tools/mkdist.py表现异常：bsp_update_sconstruct重复注入RTT_ROOT块RTT_ROOT+sys.path的行时，函数只向流里前置写入 3行新代码并保留原行，原始
sys.path行永远不被移除；scons --dist都再注入一遍，导致dist/SConstruct里# set RTT_ROOT/if not os.getenv("RTT_ROOT"): ...块不断累积；do_copy_folder撞只读/被锁文件直接放弃整次 copypackages/CMSIS-Core-latest/.git/objects/pack/*.idx、杀毒残留、文件索引器占句柄等）时，
shutil.rmtree抛WinError 5 (拒绝访问)；except吞掉异常直接return，rmtree 失败后copytree也不执行，整次 dist 中断；rm -rf dist/project才能恢复，对开发体验影响很大。你的解决方案是什么 (what is your solution)
1.先读一边文档是否已经存在
# set RTT_ROOT；存在便直接返回2.对于捕获到无法删除的，先清除其只读位进行删除
请提供验证的bsp和config (provide the config and bsp)
]
当前拉取/合并请求的状态 Intent for your PR
必须选择一项 Choose one (Mandatory):
代码质量 Code Quality：
我在这个拉取/合并请求中已经考虑了 As part of this pull request, I've considered the following:
#if 0代码，不包含已经被注释了的代码 All redundant code is removed and cleaned up