Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ensure consistent sqlite output mode #140

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

wabain
Copy link

@wabain wabain commented Apr 27, 2024

This is another variation on #112. With a non-default output mode in .sqliterc output can have a format that the scripts don't handle. For instance, in my case I have .mode column in my .sqliterc. It turns out this meant that when setting up a new machine the output of the HISTDB_SESSION query was an all-whitespace string aligned to the column width (' ') instead of the expected empty string, which meant that the output wasn't substituted with the default value 0.

The fix here is to add -list to the sqlite3 CLI arguments everywhere that #112 modified. This is obviously unfortunate and there are probably other ways that .sqliterc can break things, but unless I'm missing it there's no way to turn the rc file off completely.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant