atcoder libraryに公式対応して欲しい(もしくはgit cloneしたライブラリで壊れないようにして欲しい)

[hotman78]

Description

確認不足で現状でもできるようなら申し訳ないです

Other notes

• Will you try to create a pull request?
  • 出来たらやるかもしれません

[kmyk]

#298 がたぶんこれと同じ動機の issue です。ACL 対応してほしい分かる
cc @jellc

[kmyk]

#298 ができたとしても ACL をうまく同時に使う方法についてのドキュメントとかは書かないといけない気がする。duplicate ではないということで

[jellc]

他のライブラリと同じくソースコード上に展開することを #298 では想定しています(他のコンテストサイトでの使用も前提にしているため)。
bundle ではオプションを付けることで ac-library を展開しないことを可能にし、verify の方ではコンパイル時に git pull してきた ac-library を渡せるようにする、といった形でしょうかね。

[kmyk]

@hotman78 とりあえずは以下のようにやれば動くと思います。
ためしてないけど、基本は -I path/to/ac-library を追加するだけなので (こういうことであってる？)

---

1. .verify-helper/config.toml に CXXFLAGS を設定する。以下のような感じで書く

[[languages.cpp.environments]]
CXX = "g++"
CXXFLAGS = ["-std=c++17", "-O2", "-I", "ac-library"]

参考: https://online-judge-tools.github.io/verification-helper/document.ja.html#:~:text=CXXFLAGS

2. https://github.com/atcoder/ac-library を submodule として追加する。$ git submodule add https://github.com/atcoder/ac-library を実行してコミットする

参考: https://git-scm.com/book/ja/v2/Git-%E3%81%AE%E3%81%95%E3%81%BE%E3%81%96%E3%81%BE%E3%81%AA%E3%83%84%E3%83%BC%E3%83%AB-%E3%82%B5%E3%83%96%E3%83%A2%E3%82%B8%E3%83%A5%E3%83%BC%E3%83%AB

3. .github/workflows/verify.yml の actions/checkout@v1 のところを修正して submodule も checkout するようにする

    - uses: actions/checkout@v1
      submodules: true

参考: https://github.com/actions/checkout#:~:text=submodules:%20%60true%60

[kmyk]

不親切なので設定に include_paths みたいな項目を足したさはある
デフォルトの CXX や CXXFLAGS はそれなりにかしこく設定されてるけど、これを自分で設定するとなると C++ に強くないときびしいし面倒なので。そもそも Windows と Linux でオプションを切り替えるとかは現状の設定ファイルだと書けない

なお、設定ファイルを TOML でなく Starlark にするとか oj-verify 全体を Bazel で書き直すとかするとかなり柔軟になるはず。検討してみてはいるけど、たいへんなのでたぶんやらない

[sash2104]

ためしてないけど、基本は -I path/to/ac-library を追加するだけなので (こういうことであってる？)

これを試していたのですが、oj-verify allの途中でエラーになってしまいました。
sash2104/library#51

エラーを回避する方法 (もしくは自分のPRの不備) って何かありますでしょうか？

[kmyk]

本体のバグだったので修正プルリク作りました #330
これがマージされたらすぐバージョンを上げる予定です

[sash2104]

5.2.0使ったらoj-verify allが最後まで実行されるようになりました。ありがとうございます。

• 実行ログ
• ドキュメント生成結果