I wanted to give TinyORM a go but I got this error while compiling master branch:

I think I see the problem. Tabulate's Row_t is <code class="notransl

<div class="snippet-clipboard-content notranslate position-relative overflow-auto" data-snippet-clip

<a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="/us

Solved by <a href="https://github.com/silverqx/TinyORM/commit/ecc3ceb89e4a6e0f387ba167

compilation error in interactswithio.cpp about tinyorm HOT 7 CLOSED

nurettin commented on May 26, 2024

compilation error in interactswithio.cpp

from tinyorm.

Comments (7)

nurettin commented on May 26, 2024 1

Again? omg tabulate, the problem is they have that type hardcoded, they should export it through using so other libraries can use that using.

I'm closing it since you found a very nice solution.

from tinyorm.

nurettin commented on May 26, 2024

I think I see the problem.

Tabulate's Row_t is std::variant<std::string, char const*, string_view, tabulate::Table>
But TableRow is std::variant<std::string, char const*, tabulate::Table> (if Tabulate version is >= 1.3.0)

It looks like Tabulate 1.4.0 updated its code to include more string representations.

from tinyorm.

nurettin commented on May 26, 2024

[ 25%] Building CXX object CMakeFiles/TinyOrm.dir/tom/src/tom/concerns/interactswithio.cpp.o
[ 26%] Building CXX object CMakeFiles/TinyOrm.dir/tom/src/tom/concerns/pretendable.cpp.o
[ 27%] Building CXX object CMakeFiles/TinyOrm.dir/tom/src/tom/concerns/printsoptions.cpp.o
[ 28%] Building CXX object CMakeFiles/TinyOrm.dir/tom/src/tom/concerns/usingconnection.cpp.o
[ 29%] Building CXX object CMakeFiles/TinyOrm.dir/tom/src/tom/exceptions/tomlogicerror.cpp.o
[ 30%] Building CXX object CMakeFiles/TinyOrm.dir/tom/src/tom/exceptions/tomruntimeerror.cpp.o
[ 30%] Building CXX object CMakeFiles/TinyOrm.dir/tom/src/tom/migrationrepository.cpp.o
[ 31%] Building CXX object CMakeFiles/TinyOrm.dir/tom/src/tom/migrator.cpp.o
[ 32%] Building CXX object CMakeFiles/TinyOrm.dir/tom/src/tom/seeder.cpp.o
[ 33%] Building CXX object CMakeFiles/TinyOrm.dir/tom/src/tom/terminal.cpp.o
[ 34%] Building CXX object CMakeFiles/TinyOrm.dir/tom/src/tom/tomconstants_extern.cpp.o
[ 35%] Building CXX object CMakeFiles/TinyOrm.dir/tom/src/tom/tomutils.cpp.o
[ 36%] Linking CXX shared library libTinyOrm.so
[100%] Built target TinyOrm

I guess this deserves a pull request

from tinyorm.

nurettin commented on May 26, 2024

@silverqx would you consider merging this? It works for me

from tinyorm.