Daniel Shahaf <d...@daniel.shahaf.name> writes: > Quoting https://sqlite.org/compile.html#_options_to_omit_features: > > Important Note: The SQLITE_OMIT_* options may not work with > the amalgamation. SQLITE_OMIT_* compile-time options usually > work correctly only when SQLite is built from canonical > source files. > > and a bit later: > > Also, not all SQLITE_OMIT_* options are tested. Some > SQLITE_OMIT_* options might cause SQLite to malfunction and/or > provide incorrect answers. > > I think we should: > > - Revert r1865523
I see that this option has been supported from its introduction in 2010, and that it appears to be covered in tests. Also, since it only affects the journaling mode, apparently it does not participate in the parser/keyword generation, which is listed as the reason of not using it with amalgamation: https://github.com/sqlite/sqlite/blob/cc3f3d1f055/src/parse.y https://github.com/sqlite/sqlite/blob/cc3f3d1f055/tool/mkkeywordhash.c https://github.com/sqlite/sqlite/blob/cc3f3d1f055/test/attach4.test#L82 https://github.com/sqlite/sqlite/commit/bc6b8d73592ad72e674b10152012170a01c31c62 Thanks, Evgeny Kotkov
