开源改变世界

在自动更新机器配置期间忽略注释行 #565

推推 grbl 3年前 (2023-01-29) 200次浏览
关闭
lorenzncode 打开了这个问题 2019 年 3 月 4 日 · 7条评论
关闭

在自动更新机器配置期间忽略注释行#565

lorenzncode 打开了这个问题 2019 年 3 月 4 日 · 7条评论

注释

在自动更新机器配置期间忽略注释行 #565
贡献者

以下是我重现该问题所遵循的步骤:

  1. 从配置 [EMC]VERSION = 1.0 开始.hal 文件在注释行 #net spindle-relay motion.spindle-on hm2_5i25.0.gpio
    中包含一个过时的引脚,例如“motion.spindle-on” 。
    033.out

  2. 启动 Linux CNC (current master)

  3. 在“确认自动更新”弹出窗口中选择“是”(更新至 [EMC] VERSION=1.1)

这是我期望发生的事情:

.hal 文件中的注释行不会更新。

这是发生了什么:

.hal 文件注释行已从:
#net spindle-relay motion.spindle-on hm2_5i25.0.gpio.033.out 更新

#net spindle-relay spindle.0.on hm2_5i25.0.gpio.033.out

这是一个小问题,但说我有与旧配置版本相关的评论。自动更新后,评论可能没有任何意义。我建议不要在自动更新中更新注释行。

在我的例子中,我手动更新了我的配置以更正旧的 pin motion.spindle-on。然后我稍后运行自动更新并对文件进行差异化,并注意到注释行已更改。

在自动更新机器配置期间忽略注释行 #565
合作者

我可以看到两种情况:

  1. 评论是集成商的文档,在这种情况下更新没有意义
  2. 注释掉的行是实际的代码行,将来可能会取消注释,在这种情况下,将其变成功能行是有意义的。

也就是说,我可以看到两种方式的争论,并且更愿意看到更多的意见。

在自动更新机器配置期间忽略注释行 #565
合作者
c-莫利 评论了 2019 年 3 月 4 日 通过电子邮件
在自动更新机器配置期间忽略注释行 #565
合作者

当检测到问题时让转换程序询问用户会不会很困难?

是的。它(可能)需要两套完整的正则表达式来处理这两种选择。

在自动更新机器配置期间忽略注释行 #565
合作者

你能两者都做吗?保留旧评论并添加适合掌握的新行?

在自动更新机器配置期间忽略注释行 #565
合作者

jethornton:目前它只是一个大的搜索和替换。对评论的特殊处理将需要更多的工作。

通过更改正则表达式,我可以很容易地排除评论,但我真的不确定这是否有意义。无论如何,原始评论都备份在存档文件夹中。

在自动更新机器配置期间忽略注释行 #565
合作者

@andypugh 可能努力不值得结果。
@manymachines 你能关闭它吗?

在自动更新机器配置期间忽略注释行 #565
贡献者作者

我知道注释是有意修改的,以避免在用户取消注释该行时出现未来错误。