search

CNSRE / ABTestingGateway

MIT License
2.35k stars 767 forks source link

关于依赖软件这块的文档提个建议 #58

Open chimisu opened 7 years ago

chimisu commented 7 years ago

1、首先文档上写着：建议选用openresty最新版，但是从openresty-1.9.15.1开始，lua-resty-core有些api变更，因此建议先使用openresty-1.9.7.5及以下。这句话让人很糊涂，先说让用最新版，后说建议先使用openresty-1.9.7.5及以下，容易让人误解。我在准备环境的时候安装的是openresty-1.9.7.5。 2、我按照文档上的步骤执行utils下的demo，到第9步开始遇到 #48 ，然后我按照 #27 中提供的解决方案，我先是把Luajit替换成openresty维护的版本，然后从/usr/local/oprensty/lualib中将ngx和resty目录 copy 到ABTestingGateway的lib/lua-resty-core目录下替换，仍旧报 #48 ，我现在迷糊了。 3、建议依赖的软件指定具体的版本号， #48 问题建议在readme中给出能用的解决方案。

BG2BKK commented 7 years ago

你好，收到建议。我的想法一直是跟着春哥，openresty社区总是建议使用最新版，所以我写下了建议。关于openresty的一些api变更，https://github.com/CNSRE/ABTestingGateway/issues/48 和 https://github.com/CNSRE/ABTestingGateway/issues/27 里有相应解释，所以选择适合自己的就好，必要时修改下代码就好了。有时候个人能力所限，确实让用户体验不好，这个我也很抱歉，所以有什么想法可以提mr，我很乐意改进。

chimisu commented 7 years ago

@BG2BKK 文档还是让人比较糊涂，可能跟我没接触到这块有关系。经过我的尝试，我发现只需安装openresty(1.9.7.5)和lua-cjson就够了，启动nginx的时候直接启动openresty下的ngxin就行了，我最开始以为要单独安装nginx，结果我就一直报 #48 ，直到我看了abtesting.sh中的脚本内容我才清楚，建议文档更新下这块。