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

Running indexer via CLI fails due to unpopulated config #188

Closed
ctrlaltf24 opened this issue Oct 5, 2024 · 1 comment · Fixed by #195
Closed

Running indexer via CLI fails due to unpopulated config #188

ctrlaltf24 opened this issue Oct 5, 2024 · 1 comment · Fixed by #195

Comments

@ctrlaltf24
Copy link
Contributor

Will be making PR shortly

$ ./main.py --debug --run-indexing
2024-10-05 15:59:50 INFO: Running function: index
Exception in thread <function LogosManager.index.<locals>.run_indexing at 0x75e415ba5800>:
Traceback (most recent call last):
  File "/opt/lib/python3.12/threading.py", line 1075, in _bootstrap_inner
    self.run()
  File "/opt/lib/python3.12/threading.py", line 1012, in run
    self._target(*self._args, **self._kwargs)
TypeError: LogosManager.index.<locals>.run_indexing() got an unexpected keyword argument 'daemon'
Traceback (most recent call last):
  File "/opt/LogosLinuxInstaller/main.py", line 444, in <module>
    main()
  File "/opt/LogosLinuxInstaller/main.py", line 421, in main
    config.ACTION()
  File "/opt/LogosLinuxInstaller/logos.py", line 163, in index
    process = config.processes[config.logos_indexer_exe]
              ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^
KeyError: None
ctrlaltf24 added a commit to ctrlaltf24/LogosLinuxInstaller that referenced this issue Oct 5, 2024
And fix start_thread paramater from daemon (old) to daemon_bool (as it exists in the code today)

Fixes: FaithLife-Community#188
@thw26
Copy link
Collaborator

thw26 commented Oct 6, 2024

Please see #161 for ongoing work to fix the CLI optargs.

thw26 added a commit that referenced this issue Oct 7, 2024
@thw26 thw26 mentioned this issue Oct 7, 2024
thw26 added a commit that referenced this issue Oct 8, 2024
thw26 added a commit that referenced this issue Oct 8, 2024
@thw26 thw26 closed this as completed in #195 Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants