[Tarantool-patches] [PATCH v3 1/9] build: add Christian Hansen c-dt to the build
Vladislav Shpilevoy
v.shpilevoy at tarantool.org
Thu Aug 5 02:58:21 MSK 2021
Hi! Thanks for the patch!
On 02.08.2021 02:40, Timur Safin via Tarantool-patches wrote:
> * Integrated chansen/c-dt parser as 3rd party module to the
> Tarantool cmake build process.
> * Points to tsafin/c-dt instead iof original chansen/c-dt to
> have easier build integration, because there is additional
> commit which integrated cmake support
As I said in the previous review, it points at
https://github.com/tarantool/c-dt.git, not at tsafin/c-dt. Why
do you keep saying the contrary? Look:
url = https://github.com/tarantool/c-dt.git
It is 'tarantool', not 'tsafin' here.
Also, like on the previous review, when I call 'make' and do
'git status', I see:
Changes not staged for commit:
(use "git add <file>..." to update what will be committed)
(use "git restore <file>..." to discard changes in working directory)
(commit or discard the untracked or modified content in submodules)
modified: third_party/c-dt (modified content, untracked content)
You still didn't fix it. Please, do.
More information about the Tarantool-patches
mailing list