set time_limit in lilac.py

[petronny]

47

这样？ 没测过正确性。。。

[petronny]

呃，我看见错了。稍等 修好了，不过还是没测过。。。

[lilydjwg]

1. 这样我们好像加载了 lilac.py 两次？
2. 需要根据 build.log 来找出需要提升时间限制的包。不过目前并没有记录打包耗时，不太好找。

[petronny]

1. 对。。。感觉得把lilac.py的结构体存了才能解决，我再去看看lilac.py的加载机制
2. 我刚才还在build机上找build.log来着，不过应该在你home下没有读权限吧

[petronny]

1. 呃下面的lilac_build又加载了一次。。。所以是3次。。。估计那些被作为依赖的加载的更多

[lilydjwg]

build.log 在 ~lilydjwg/.lilac/build.log，已加权限。

[lilydjwg]

有读权限的，来我家里读就行。上层目录只有进入权限，所以 ls 不出来是对的。

2017年12月22日 13:47，"Jingbei Li" notifications@github.com写道：

1. 对。。。感觉得把lilac.py的结构体存了才能解决，我再去看看lilac.py的加载机制
2. 我刚才还在build机上找build.log来着，不过应该在你home下没有读权限吧

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/archlinuxcn/lilac/pull/48#issuecomment-353527163, or mute the thread https://github.com/notifications/unsubscribe-auth/AAa5VXTsrqhGpR4AOKflmxtNg1P5L57Oks5tC0KLgaJpZM4RKqLG .

[lilydjwg]

现在的代码，实现这个应该容易很多了。

[petronny]

这个也更新了

[lilydjwg]

不要用 assert 处理外部数据。assert 用来确定程序内部数据与预期一致，不是用来检查外部数据是否合法的。

另外，合并之前需要先把那些已经超过默认时限的包的 lilac.py 给修改了（可以查 build.log）。

最后，更新 docs 啦。

[petronny]

不要用 assert 处理外部数据。assert 用来确定程序内部数据与预期一致，不是用来检查外部数据是否合法的。

那是应该写成if ... raise吗

改成if ... raise并放到load mod的过程中了

[lilydjwg]

叫 time_limit_hours 会不会更清晰？

你要帮忙更新需要更新的 lilac.py 吗？

[petronny]

改名改好了，希望没有typo... 我把有时间记录的build log都看看吧

[petronny]

从2018-07-04 17:18:02之后的build.log看：

超过3600s的没有
接近3600s的只有scribus-svn(约3300s)
超过1800s的只有firefox-kde-opensuse(约2800s)和scribus-svn
接近1800s的只有qtwebkit(约1600s)和firefox-kde-opensuse, scribus-svn

之前报错的chromium-dev在2018-07-04 17:18:02之后一直failed...😓 我就改一下这个的lilac.py好了

另外不考虑一下默认超时半小时吗？