Re: [PATCH 0/1] compaudit: add ZSH_COMPLETION_USERS parameter
Petros Amoiridis <[email protected]> Wed, 1 Jul 2026 10:38:13 +0300
| Newsgroups | gmane.comp.shells.zsh.devel |
|---|---|
| Message-ID | <CAN4N7iO7CYuSFuvDBX8dnQk4dwBrX4Owv7-aG68kGrnRgfpYqw@mail.gmail.com> |
Thank you all. This is a better direction than my proposed approach. Mikael, the export and the zshenv example were both wrong too, and the -t route sidesteps all of it. So I'm happy to drop ZSH_COMPLETION_USERS. dana, indeed, I used an LLM. I didn't mean any disrespect though. I've been working on this on and off for months, and it also went through an internal review by my team. And I tried to polish the cover letter to be closer to my voice, but clearly I didn't do a great job. The proposed idea came from us. We even considered a second approach which I was ready to post as a new thread. However, given your proposed approach, I think this is settled in the best way. If it helps, I can test your patch against real setups once it's available. I can test Homebrew zsh with /opt/homebrew/share/zsh owned by a non-root service user on macOS, and the same on Linux with a service account. I can run the sudo case too. Petros