Hi Nis, Nis Martensen <nis.marten...@mailbox.org> writes:
> On 12.05.2024 23.11, Xiyue Deng wrote: >> Nis Martensen <nis.marten...@mailbox.org> writes: >> >>> On 11.05.2024 08.23, Xiyue Deng wrote: >>>> I'd like to propose adding an option to skip loading configuration files >>>> (/etc/reportbug.conf and ~/.reportbugrc). The use case is for external >>>> programs that runs reportbug (e.g. debian-bug in elpa-debian-el) which >>>> provides its own command line switches and have an assumption on the >>>> output. When a user set some custom options, notably CC related ones, >>>> it may interfere with how X-Debbugs-Cc is handled and broke some of the >>>> assumptions of external tools (see https://bugs.debian.org/1032662). >>> >>> Thanks for the report. Did you intend to link to a different bug as your >>> example? The one you cite does not seem to be related to reportbug. >> >> Ah, indeed. I was trying to refer to https://bugs.debian.org/1069908. >> Thanks for the correction! > > Thanks for providing the example. To me it does not give convincing > evidence that an option to skip loading configuration files is a good > idea. Even with such an option one would still be able to get unintended > results by calling reportbug with buggy arguments. ;) > > Can you please clarify what specific assumptions of external tools you > have in mind? Would a command line option to override a "list-cc-me" > specified in the configuration file be sufficient to solve the problem? > Is this an actual problem in your case? > The point is to let other program achieve the exact behavior by the arguments supplied to reportbug. Using the use case in elpa-debian-el as an example, we handle CCing oneself in ELisp, so we assume reportbug is invoked without "list-cc-me" by not passing that argument. However, if the user has a "~/.reportbugrc" with "list-cc-me" in it, it will break this assumption, and reportbug ended up having it's own self CC and elpa-debian-el added another self CC, which would be confusing to users. If we have an option to disable loading "~/.reportbugrc" we can avoid this situation. The same applies to other arguments that user may set but affect the assumptions of other programs. As for buggy arguments, we can also fix that directly in elpa-debian-el by removing the buggy ones without having to change anything in reportbug, so this would not be an actual issue. Wdyt? -- Xiyue Deng
