datalad / datalad-deprecated

DataLad extension for functionality that has been phased out of the core package
Other
0 stars 3 forks source link

ls --json: causes subdatasets being installed #84

Open yarikoptic opened 1 year ago

yarikoptic commented 1 year ago

looking at the log of running the hooks/post-update which runs datalad ls -

[INFO] Directory: /srv/datasets.datalad.org/www/dbic/QA/sub-qa64/ses-20230705/func
[INFO] Directory: /srv/datasets.datalad.org/www/dbic/QA/sub-qa64/ses-20230705
[INFO] Directory: /srv/datasets.datalad.org/www/dbic/QA/sub-qa64
[INFO] Dataset: /srv/datasets.datalad.org/www/dbic/QA
subdataset(ok): code/containers (dataset)
subdataset(ok): code/reproin (dataset)
metadata(ok): . (dataset)
subdataset(ok): code/containers (dataset)
subdataset(ok): code/reproin (dataset)
Submodule 'code/containers' (https://datasets.datalad.org/repronim/containers/.git) registered for path 'code/containers'
Submodule 'code/reproin' (https://github.com/ReproNim/reproin) registered for path 'code/reproin'
Cloning into '/srv/datasets.datalad.org/www/dbic/QA/code/containers'...
Cloning into '/srv/datasets.datalad.org/www/dbic/QA/code/reproin'...
Submodule path 'code/containers': checked out 'b0d4dbf652f4ffa0bc358f19356a39f9e891abba'
fatal: remote error: upload-pack: not our ref d47e66272571eba5c2a1ab4d9df01281d1d77727
fatal: Fetched in submodule path 'code/reproin', but it did not contain d47e66272571eba5c2a1ab4d9df01281d1d77727. Direct fetching of that commit failed.

so it installed/attempted to install submodules which IMHO it should have not done.