方舟编译器在开源的第三天,开始接受代码合并请求。第一个被合并的代码请求就是文档中的“GN_AR”问题。
“GN_AR”问题是一个很小的问题,起源于编译方舟编译器源码的时候,需要修改build/config/BUILDCONFIG.gn中的
GN_AR_COMPILER = "${MAPLE_ROOT}/tools/clang_llvm-8.0.0-x86_64-linux-gnu-ubuntu-16.04/bin/llvm-ar"
配置选项。但是,方舟的官方文档Development_Preparation.md在介绍该配置步骤的时候,说的是“GN_AR”,并且文档中引用build/config/BUILDCONFIG.gn中的代码的时候也是:
GN_AR = "${MAPLE_ROOT}/tools/clang_llvm-8.0.0-x86_64-linux-gnu-ubuntu-16.04/bin/llvm-ar"
第一个合并请求就是要将这两个GN_AR修改为GN_AR_COMPILER。
这个合并请求合并的过程之中,也引发了另外一个问题。那就是fork出来的repo会自动添加一个README.md文件。官方网站上的issue96描述了这个过程。所以,第一个合并请求有Initial commit(添加README.md)、修改文档、Revert "Initial commit"(删除README.md),共三个提交记录。
另外,第一个PR的提交者是“nobodxbodon”,合并PR的是方舟开源时候上传代码的账号“BinaryFZ”。第一个PR被接受了,那么其他PR也不会远了,有兴趣提交修改的同学可以开始尝试提PR了。