clone with inaccessible submodule fails the clone process (cartodb-private) about cartodb

[Oreilles]

In case anyone stumble into this, here's the solution:

git clone https://github.com/cartodb/cartodb.git
cd cartodb
git rm private
git submodule update --init --recursive

[Stivens73]

@sabman, you can also run it with -n flag for a shallow copy.

upd: okay, this might not be a good solution, -n as shallow copy will basically create an empty dir

[ilbambino]

the private submodule adds some things mostly for the CI as you said. For different business reasons we have decided not to release those in the open source version.
we are sorry for any inconveniences caused, but we considered different options and the submodule was the less intrusive one.
You will have to apply any of those workarounds for the time being. We will add a note in the README to clarify it. But unfortunately we cannot remove it, or it will break our current CI setup.

[Stivens73]

Also encountered this issue from one of Dockerized carto images. Seems like this is an error, at least I hope so.

[sabman]

the PR that originally added the submodule is here:
#16023

I am wondering if there are any ideas on a better way to manage this? For now i could either clone without the --recursive flag and also comment out the private submodule from the .submodulefile.

sed -i '.bak' 's/\[submodule "private"\]/#&/' .gitmodules
sed -i '.bak' 's/^[[:space:]]*url = [email protected]:CartoDB\/cartodb-private\.git/#&/' .gitmodules
sed -i '.bak' 's/^[[:space:]]*path = private/#&/' .gitmodules

I'm sure there's a better way though, any suggestions?

[sabman]

looking more closely at the PR where private submodule was added it seems it's about the new CI. So it's possible that the tagged release will exclude the private submodule (ideally that's how it should be done).

[sutanAE]

In case anyone stumble into this, here's the solution:

git clone https://github.com/cartodb/cartodb.git
cd cartodb
git rm private
git submodule update --init --recursive

hi @Oreilles it seems like this does not work anymore?