archlinuxcn / lilac

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

now can specific the path to config.ini as an argument #30

Closed petronny closed 7 years ago

petronny commented 8 years ago

如题

lilydjwg commented 8 years ago

我们需要一种更好的代码组织方式,来干掉这些全局变量。

petronny commented 8 years ago

我还没写过带setup.py的东西。。。 并不清楚要怎么做。。。

petronny commented 8 years ago

现在只有指定config.ini功能了

petronny commented 8 years ago

写了一个初步的版本。。 完全没有调试过。。

大约是这个意思吧?

petronny commented 8 years ago

又改了一波。把顶上的那3个和Other settings下面的变量恢复成全局变量了。 还有其他要改的吗

lilydjwg commented 8 years ago

有冲突……

petronny commented 8 years ago

好了 不过我没跑过不知道对不对。。

petronny commented 7 years ago

我觉得咱还是尽快解决用参数指定config.ini然后 mv lilac/config.ini repo/lilac.ini吧

lilydjwg commented 7 years ago

啊我都忘记了这个 pr……

lilydjwg commented 7 years ago

我合并了,看看待会会不会报错……

lilydjwg commented 7 years ago

唉,合并了你的代码,lilac 又挂了…………

petronny commented 7 years ago

囧。。那我去调试调试。。