Skip to content

enable submodule#92

Merged
KenyuKurimoto merged 4 commits intomainfrom
feature/checkout_recursively
May 18, 2022
Merged

enable submodule#92
KenyuKurimoto merged 4 commits intomainfrom
feature/checkout_recursively

Conversation

@jsupratman13
Copy link
Contributor

Summary

Detail

Impact

Test

Attention

@jsupratman13 jsupratman13 changed the title Feature/checkout recursively enable submodule May 18, 2022
@jsupratman13 jsupratman13 marked this pull request as ready for review May 18, 2022 08:47
@KenyuKurimoto
Copy link
Contributor

(参考までに)gitのバージョンが低かったのはros-buildをどこで走らせたとき?

@jsupratman13
Copy link
Contributor Author

(参考までに)gitのバージョンが低かったのはros-buildをどこで走らせたとき?

- name: Check out source repository
uses: actions/checkout@v2
with:
path: ros/src/${{ github.event.repository.name }}
submodules: recursive
timeout-minutes: 1
ここです。

@KenyuKurimoto
Copy link
Contributor

KenyuKurimoto commented May 18, 2022

(参考までに)gitのバージョンが低かったのはros-buildをどこで走らせたとき?

- name: Check out source repository
uses: actions/checkout@v2
with:
path: ros/src/${{ github.event.repository.name }}
submodules: recursive
timeout-minutes: 1

ここです。

ごめん、そういう意味ではなく、どの環境の上で走らせたときにgitのバージョンが低かったのかという疑問

普段走らせているself-hosted-runnerに該当するうちのサーバーのgitバージョンは確認したら2.36になっていて、普段使う仮想環境も2.36に元からなっているはずなので
https://github.com/actions/virtual-environments/blob/main/images/linux/Ubuntu1804-Readme.md

@jsupratman13
Copy link
Contributor Author

(参考までに)gitのバージョンが低かったのはros-buildをどこで走らせたとき?

- name: Check out source repository
uses: actions/checkout@v2
with:
path: ros/src/${{ github.event.repository.name }}
submodules: recursive
timeout-minutes: 1

ここです。

ごめん、そういう意味ではなく、どの環境の上で走らせたときにgitのバージョンが低かったのかという疑問

普段走らせているself-hosted-runnerに該当するうちのサーバーのgitバージョンは確認したら2.36になっていて、普段使う仮想環境も2.36に元からなっているはずなので https://github.com/actions/virtual-environments/blob/main/images/linux/Ubuntu1804-Readme.md

I don't think ros-build.yml runs on self-hosted runner, I believe it was docker?

@KenyuKurimoto
Copy link
Contributor

うーん、なんで低いバージョンのgitが使われたんだろ…

@KenyuKurimoto
Copy link
Contributor

#92 に関してはこれで解決したということで良い?

@jsupratman13 jsupratman13 requested a review from Tacha-S May 18, 2022 09:17
@Tacha-S
Copy link
Contributor

Tacha-S commented May 18, 2022

うーん、なんで低いバージョンのgitが使われたんだろ…

dockerのros:melodicイメージでCI走らせてます。

docker run -it ros:melodic bash
root@22fe9b462e38:/# git --version
git version 2.17.1

@KenyuKurimoto
Copy link
Contributor

うーん、なんで低いバージョンのgitが使われたんだろ…

dockerのros:melodicイメージでCI走らせてます。

docker run -it ros:melodic bash
root@22fe9b462e38:/# git --version
git version 2.17.1

なるほど。ありがとうございます。

@KenyuKurimoto KenyuKurimoto merged commit 8468ac2 into main May 18, 2022
@KenyuKurimoto KenyuKurimoto deleted the feature/checkout_recursively branch May 18, 2022 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

submoduleは考慮している?

3 participants