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

Fix k4run -l #126

Merged
merged 1 commit into from
Jul 25, 2023
Merged

Fix k4run -l #126

merged 1 commit into from
Jul 25, 2023

Conversation

jmcarcell
Copy link
Contributor

@jmcarcell jmcarcell commented Jul 21, 2023

BEGINRELEASENOTES

  • Fix k4run -l. Currently it fails because the parsing always wants a config file.

ENDRELEASENOTES

Current behavior (for example in the nightlies):

$ k4run -l
Error: missing gaudi options file.
Usage: k4run <options_file.py>, use --help to get a complete list of arguments

After this PR:

$ k4run -l
[k4run] Available components:
=====================
  AbortEventAlg (from GaudiExamples),
		 path: /Gaudi/install/python/GaudiExamples/__init__.py

  AlgContextAuditor (from GaudiAud),
		 path: /Gaudi/install/python/GaudiAud/__init__.py

  AlgContextSvc (from GaudiCommonSvc),
  ...

Passing no arguments keeps working as expected:

$ k4run
Error: missing gaudi options file.
Usage: k4run <options_file.py>, use --help to get a complete list of arguments

@atolosadelgado @BrieucF

@jmcarcell
Copy link
Contributor Author

Merging to get this into the nightlies since it seems no one has any complaints about this and it fixed the issue for me

@jmcarcell jmcarcell merged commit d10ca31 into master Jul 25, 2023
4 of 9 checks passed
@jmcarcell jmcarcell deleted the fix-k4run branch July 25, 2023 16:36
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