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

CV: .filetags may contain mandatory options #17

Closed
novoid opened this issue Apr 29, 2018 · 3 comments
Closed

CV: .filetags may contain mandatory options #17

novoid opened this issue Apr 29, 2018 · 3 comments

Comments

@novoid
Copy link
Owner

novoid commented Apr 29, 2018

Controlled vocabulary files may contain a list of command line options that are mandatory.

Example: #options --recursive --filebrowser thunar always uses functions recursive and opens results in thunar (instead of a different default browser).

FIXXME:

  • What about contradicting CLI options?
    • example 1: --filebrowser thunar in .filetags and --filebrowser geeqie in command line?
    • example 2: --tagtrees-depth 3 in .filetags and user just wants to tag a file (which does not allow/need tagtrees at all and results in an error)
@tuxlifan
Copy link
Contributor

Yes, please!

ad ex.1: CLI options should win through higher specificity (with possibly a warning being displayed).

ad ex.2: filetags --tagtrees-depth 3 foo seems to work without problem to just add a tag here? In case of conflicting modes I think that again CLI input should win as it signifies the current intent of the user.

jm2c

@novoid
Copy link
Owner Author

novoid commented May 1, 2018

Short remark on ex.2: I plan to separate filetags from tagtrees using a separate Python script within the same repository. Tagtrees has been a small feature but got bigger and bigger.
See #8

@novoid
Copy link
Owner Author

novoid commented Dec 29, 2021

I'm going to close this because #16 is a much better solution although it doesn't allow "overriding" this config file in a .filetags file which is located in a deeper level within the file system hierarchy.

@novoid novoid closed this as completed Dec 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants