Closed geaaru closed 5 years ago
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
:memo: Please visit https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here (e.g. I signed it!
) and we'll verify it.
ℹ️ Googlers: Go here for more info.
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
:memo: Please visit https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here (e.g. I signed it!
) and we'll verify it.
ℹ️ Googlers: Go here for more info.
I signed it!
Hey, I can't reproduce this issue, can you describe what you did to test this change? I'm a little confused because you should be able to write the command with spaces in between, after the "foreach" instead of needing to list it all as one command.
Maybe we need a "--" ?
I never used to hit this in my projects that have submodules but I hit it now. The only thing that has changed on my end is my git version. I upgraded git via brew a few days ago:
$ git --version
git version 2.22.0
My dep version hasn't changed:
$ dep version
dep:
version : v0.5.3
build date : 2019-05-19
git hash : 192eb44
go version : go1.12.5
go compiler : gc
platform : darwin/amd64
features : ImportDuringSolve=false
Hi,
in my case I have:
$ git --version
git version 2.21.0
You could reproduce it directly with git and the repository that I wrote:
$# git clone https://github.com/openSUSE/umoci.git
$# git submodule foreach --recursive git clean -x -d -f -f
Entering '.site/themes/hugo-theme-learn'
error: unknown switch `x'
$# # right way with avoid use of clean option directly
$# git submodule foreach --recursive "git clean -x -d -f -f"
Entering '.site/themes/hugo-theme-learn'
In summary, if you avoid assembling the git clean options then the options are seen as git submodule options and this break the execution.
OK
OK
Fix execution with the vendor with submodules.
What does this do / why do we need it?
This fix error: