[product suggestions] 无法通过编译的代码为什么会出现在主线!!??
Tofloor
poster avatar
wlly-lzh
deepin
2024-09-27 20:14
Author

有点难以想象,遇到了这种低级问题。

两次遇到 master分支的代码无法通过编译,我十分怀疑开发在上传代码之前没有在本地编译过代码。


第一次编译失败是缺少一个命名空间,见编译deepin-unioncode失败

src/plugins/codegeex/widgets/inputeditwidget.cpp中缺少一个命名空间。

可以查看这个文件相关的提交记录:2024/9/20(commit 859f569f246250ac3ba85dbe933c01767e33add8)

今天又编译失败了,原因竟然是因为缺少头文件

见:3rdparty/unioncode-qscintilla214/src/PlatQt.cpp (commit 24fc34dba1ace9b3f29b289aa830fefe36bc783e)

第512行:

 518 |     QPainterPath path;

缺少头文件:qpainterpath.h


哎……

不说别的了,在deepin干活应该很轻松吧。

Reply Favorite View the author
All Replies
忘记、过去
deepin
2024-09-27 20:38
#1

QPainterPath 在 Qt 5.15 里面确实需要单独引用头文件#include


但是,在 Qt 5.11.3 里面,只要 #include 就会间接引用到 ,不需要单独写...... doubt

所以有一种可能,代码在 UOS 上编译通过后就直接合并上传了。至于使用高版本 Qt 的 deepin 会不会同步测试,能不能编过就要看运气了 joy

反正咱也遇到过这种问题,已经被坑了好几次了......这种头文件变化写的时候也不一定能想的起来

Reply View the author
Oli
deepin
2024-09-27 20:49
#2

哈哈哈哈哈在deep干活应该很轻松吧,好有杀伤力又轻飘飘的吐槽啊

Reply View the author
gfdgd_xi
deepin
Ecological co-builder
2024-09-27 20:54
#3
忘记、过去

QPainterPath 在 Qt 5.15 里面确实需要单独引用头文件#include


但是,在 Qt 5.11.3 里面,只要 #include 就会间接引用到 ,不需要单独写...... doubt

所以有一种可能,代码在 UOS 上编译通过后就直接合并上传了。至于使用高版本 Qt 的 deepin 会不会同步测试,能不能编过就要看运气了 joy

反正咱也遇到过这种问题,已经被坑了好几次了......这种头文件变化写的时候也不一定能想的起来

joy 我就说为什么移植 DDE15 时很多都没有 #include

Reply View the author
wlly-lzh
deepin
2024-09-27 21:12
#4
忘记、过去

QPainterPath 在 Qt 5.15 里面确实需要单独引用头文件#include


但是,在 Qt 5.11.3 里面,只要 #include 就会间接引用到 ,不需要单独写...... doubt

所以有一种可能,代码在 UOS 上编译通过后就直接合并上传了。至于使用高版本 Qt 的 deepin 会不会同步测试,能不能编过就要看运气了 joy

反正咱也遇到过这种问题,已经被坑了好几次了......这种头文件变化写的时候也不一定能想的起来

有可能吧,这样的话就是开发环境与部署环境不同造成的问题了。

Reply View the author
神末shenmo
deepin
Spark-App
2024-09-28 00:59
#5
gfdgd_xi

joy 我就说为什么移植 DDE15 时很多都没有 #include

等debian 13吧,到时候调这些够我俩受的了

Reply View the author
神末shenmo
deepin
Spark-App
2024-09-28 01:00
#6
wlly-lzh

有可能吧,这样的话就是开发环境与部署环境不同造成的问题了。

活跃开发分支确实会这样的,nightly build跑不起来也正常

打了tag的跑不起来才算事故吧~

Reply View the author
raspbian
deepin
2024-09-28 10:11
#7

我每次都是checkout到一个最新的tag之后编译安装的

Reply View the author
Mozart
deepin
2024-09-29 10:43
#8

是的,开发环境目前是专业版,如果在社区版编译确实有可能出现问题。常规的做法是在提交代码后执行各版本编译任务,但IDE是没有添加的。 之前更多是内部开发,最近出现代码层面的反馈,说明大家也开始关注代码本身,所以添加CI提升大家的编码体验也是有必要的。

Reply View the author
新手来啦~
deepin
2024-10-02 22:41
#9

agree

Reply View the author
小鱼贝壳
deepin
2024-10-04 19:43
#10

applaud

Reply View the author