问题反馈

[awkoo]

1：在bbg_patch.sh中硬编码../导致编译错误
2：bbg_patch.sh不应在Makefile中生成，考虑将其放置于内核scripts目录？

[AlexLiuDev233]

可以说一下内核版本和报错日志吗

[awkoo]

与内核版本无关，原因是“make O=”可以是任何目录

[AlexLiuDev233]

主要问题在于，必须在Selinux Makefile中执行sh patch，如果在baseband_guard本身的makefile中调用，有时候会导致patch晚于目标文件编译

[awkoo]

为什么一定要在编译的时候patch呢，也可以在setup中patch啊

[AlexLiuDev233]

唔，可以试一下，把bbg_patch.sh丢到项目根目录，setup.sh在clone repo之后进行调用

[awkoo]

就不能把patch扔setup里吗，这玩意只需要执行一次就够了呀，也没有动态生成的东西

[AlexLiuDev233]

？直接丢到setup.sh的话，那个文件就特别乱了，所以想在setup.sh阶段执行，需要拆分文件..

[awkoo]

行吧，记得执行完删掉就行

[AlexLiuDev233]

你还是没看懂，我的意思是作为repo的一部分提交...不是临时文件

[AlexLiuDev233]

取代现在的sepatch.txt

[awkoo]

总之把Makefile那段shell脚本生成分离出来就行，这不是Makefile该做的，它只负责调用执行

[AlexLiuDev233]

问了下 cctv18，setup.sh要大改，做patch的移除，升级，安装，所以等弄完你这个issue自然也解决了，还有您能不能不要用那种emoji，开发的人也是人，不是牛马不是机器...

[awkoo]

好的，抱歉

[awkoo]

awkoo/kernel_mi845_lineage@d70f82f
能否考虑以这种方式作为最终结果？

[AlexLiuDev233]

看 #38 ，有人已经给解决了）你这个commit应该是没有使用setup.sh然后手动集成的吧
能跑是能跑，但是你更新bbg会很崩溃的
原因嘛，我本地在写个东西，对于5.1-的内核需要加一个patch，解决一个已知的可能被利用的漏洞（虽然似乎现在没人发现）

[awkoo]

相对于移植susfs来说，ggb应该还算是简单，至于那条commit，我是手动执行生成后的patch.sh，再删掉Makefile那段patch生成

[AlexLiuDev233]

相对于移植susfs来说，ggb应该还算是简单，至于那条commit，我是手动执行生成后的patch.sh，再删掉Makefile那段patch生成

不不不，问题在于，你把Baseband-guard子模块删了，你打算后面怎么更新？

[awkoo]

不不不，问题在于，你把Baseband-guard子模块删了，你打算后面怎么更新？

手动对比commit，手动修改就行，毕竟这玩意代码量很少，重要文件就两个