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

Support option -d to set xapian index database via CLI #1520

Conversation

liaham
Copy link
Contributor

@liaham liaham commented Apr 2, 2024

For running xapian indexer it is necessary to configure it to use the same database for indexing as defined in the user interface on plugin settings.

In order to make the configuration of xapian indexer some kind of persistence the command line options in redmine_dmsf/extra/xapian_indexer.rb could be extended by providing a further option for the index database path:

ruby redmine_dmsf/extra/xapian_indexer.rb -d <path/to/my/custom/index/database>

With this additional option it is possible to move the index database outside of redmine root to keep it even after running updates. Furthermore, custom configuration in redmine_dmsf/extra/xapian_indexer.rb won't be overridden when updating this plugin.

For running xapian indexer it is necessary to configure it to use the same database for indexing as defined in the user interface on plugin settings.

In order to make the configuration of xapian indexer some kind of persistence the command line options in `redmine_dmsf/extra/xapian_indexer.rb` could be extended by providing a further option for the index database path:

```shell
ruby redmine_dmsf/extra/xapian_indexer.rb -d <path/to/my/custom/index/database>
```

With this additional option it is possible to move the index database outside of redmine root to keep it even after running updates. Furthermore, custom configuration in `redmine_dmsf/extra/xapian_indexer.rb` won't be overridden when updating this plugin.
@picman picman self-requested a review April 2, 2024 13:43
@picman
Copy link
Collaborator

picman commented Apr 2, 2024

Please fix the Rubocop error about a too long line.

@picman picman merged commit a252e60 into danmunn:devel Apr 2, 2024
3 checks passed
@picman
Copy link
Collaborator

picman commented Apr 2, 2024

Thank you!

@liaham liaham deleted the xapian-indexing-with-further-option-for-db-setting branch January 21, 2025 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants