search

archlinuxcn / lilac

Lilac is the build bot for archlinuxcn
GNU General Public License v3.0
114 stars 40 forks source link

bugs fix #56

Closed petronny closed 6 years ago

petronny commented 6 years ago

首先正确去除了报错邮件里的\r

报错的时候现在也有build_output了 不过感觉这里问题很大

lilydjwg commented 6 years ago

这样做是为什么呢？

petronny commented 6 years ago

就是如果build出错了,out就走 raise CalledProcessError(code, cmd, out)而不是return out了

现在都是基于拿到了call_lilac_build里面返回的out再做的，得在返回或raise之前就修好out 或者在raise的时候也有out

lilydjwg commented 6 years ago

那就在处理 CalledProcessError 那里再处理呗。

petronny commented 6 years ago

呃有道理，改了一下，感觉问题没那么大了

petronny commented 6 years ago

这个还有啥问题不？