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

[MacOS | homebrew | 0.8.6] Error after update from 0.8.5: TOML parse error missing field is_font #152

Closed
dangnhdev opened this issue Jun 28, 2024 · 3 comments

Comments

@dangnhdev
Copy link

dangnhdev commented Jun 28, 2024

Terminal output:

fselect
TOML parse error at line 1, column 1
  |
1 | no_color = false
  | ^^^^^^^^^^^^^^^^
missing field `is_font`

Tried to find config.toml at ~/.config/fselect as in the doc but the folder doesn't exist.
May I ask where is the location of config.toml on Mac (installed via homebrew)?
Update: There is argument for setting config file location as well but I can't find any example .toml file in this project to copy from 😔

The fselect command still show result and in the mean time I just cut first 6 lines from its stderr to temporary use it:

fselect path, is_dir where "(name != '*.md' and is_dir = false) or (is_dir and name = '.*')" order by is_dir 2>&1 | tail -n +6
@jeff-arnold
Copy link

Environment: Mac OS X 12.7.5, fselect 0.8.6 installed via MacPorts

I was seeing the same error message.

The fselect config.toml file is found at ~/Library/Application\ Support/jhspetersson.fselect/config.toml.

The last line in that file was

is_fond = [".eot", ".fon", ".otc", ".otf", ".ttc", ".ttf", ".woff", ".woff2"]

with no newline at the end of the file.

I changed "is_fond" to "is_font" and added a newline and the error has gone away.

I don't know where this file comes from. It doesn't appear to be provided by the MacPorts fselect package.

@JonathanDoughty
Copy link

I can confirm that adding the is_font definition to ~/Library/Application\ Support/jhspetersson.fselect/config.toml - which was entirely missing in my macOS brew installation - was successful in eliminating the TOML parse error issue for me.

@jhspetersson
Copy link
Owner

jhspetersson commented Dec 18, 2024

Thanks for the feedback! With 0.8.7 release config parsing has been improved not to fail on missing parameters (in case of upgrade or something).

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

No branches or pull requests

4 participants