-
Notifications
You must be signed in to change notification settings - Fork 495
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
Using local config file with CLI mode #368
Comments
No, currently in cli mode config files are not read, only flags you pass on command line matter. wxWidgets has (or had) a bug where config reader does not work properly if gui is not initialized. I would probably need a hand-written parser to make config files work in cli mode. |
I will just add that this is something that would also benefit the Fusion/Eagle workflow, for which I've only ever used the CLI mode. For instance, I understand why 'include tracks' and 'include nets' are disabled in ibom by default, but for my personal use I always want them enabled and I'd like to be able to flip that switch in the config file and never need to pass the extra command-line arguments again. Ditto for things like certain extra fields that I might want to always include. I'm willing to help implement it, I just hadn't touched it before because the most obvious implementation would apply to all parsers, not just the Eagle/Fusion ones, and I was concerned about unintended side effects since I'm not at all experienced with kicad workflow(s). |
I there an example on how to pass those flags through the command line? I couldn't find them. I was thinking to force a simple parser if the option of no display is detected in the environment variables. I'll try to add it and if it works I'll submit a pull request. |
My 2 cents: |
If anyone wants to take a stab at it, be my guest. Only requirement is to not use any packages outside of standard python installation and to test your code at least on windows and linux. I can help test on windows if you can't.
Launch the script with
wxWidgets config lib was convenient and there wasn't a good reason to switch. Like you said, for cli it's easy to write a wrapper script to set all the flags you want. |
qu1ck, I do have a good starting point for this, and I have created a patch if you are interested. I could create a branch and push, but I have not done any thorough testing. It works ok for what we need. |
You can open a draft pull request, I can at least leave some comments on whether you are going in a direction that can be merged and what work needs to be done, if any, for that. |
The local project configuration will allow a good integration with the new jobset output manager of KiCad. just running iBOM as as CLI command. |
I'm running the script from command line and I set the environment variable INTERACTIVE_HTML_BOM_NO_DISPLAY.
This works ok, but even though the ibom.config.ini file is present and also the general config.ini file is located at the InteractiveHtmlBom folder, neither of those is read in this NO_DISPLAY mode.
I there a way to force it to use either one of the config files?
I'm using latest (at the moment) commit 1b551f5 and KiCad 7.0.1
Thanks
The text was updated successfully, but these errors were encountered: