Attention is currently required from: Felix Singer, Thomas Heijligen.
7 comments:
Patchset:
As part of rebasing, I replaced --image (this option is deprecated) with --include
File Makefile:
Patch Set #9, Line 589: ACTIVE_PROGRAMMERS += internal
It took a little until I understood this programmer list is needed for the substitution. […]
Yes, it should be mentioned, thank you. Done.
Patch Set #9, Line 1032: $(PROGRAM).8 $(PROGRAM).8.html $(PROGRAM).bash $(BUILD_DETAILS_FILE)
One tab too much
What do you mean? I just added `$(PROGRAM).bash`
Patch Set #9, Line 1041: .bash
That's just the file in the source directory. […]
From [Bash Completion FAQ](https://github.com/scop/bash-completion/blob/master/README.md#faq):
```
The completion filename for command `foo` in this directory should be either `foo`, or `foo.bash`.
```
So, we only have two naming options available. I cannot use a name without the suffix due to a name conflict (As Felix mentioned above, the flashrom binary has the same name). Unfortunately, a custom prefix isn't supported, so we have no choice.
Felix, your idea is what I wanted to do initially. But there's a problem with meson, please have a look at another Thomas comment #646.
File meson.build:
Patch Set #9, Line 624: if get_option('classic_cli').disabled()
When `get_option('bash_completion').auto()` and `get_option('classic_cli'). […]
Thanks for catching it!
Patch Set #9, Line 646: .bash
same here
No way. If the output file is `flashrom`, the bash-completion file will be overwritten by the executable.
`install` command (in the makefile) lets me choose DESTINATION, and meson only copies the file in one place - `install_dir`/`output`.
File util/flashrom.bash-completion.tmpl:
Patch Set #9, Line 43: OPTS="--help
Missing \ at the end of each line.
It makes no difference for this task. Also, those `\` at the end of lines look ugly :(
To view, visit change 68247. To unsubscribe, or for help writing mail filters, visit settings.