Make exim more configurable
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest0 of 3 patches 0 repliesNew
Patches in v1 No applies recorded yet
None of the 3 patches the cover letter announces are in this archive.
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.
Cover letter [Buildroot] [PATCH 0/3] Make exim more configurable
Dear Thomas,
Quoted 27 lines · Thomas Petazzoni
Dear Luca Ceresoli,
Quoted 10 lines · Luca Ceresoli
Hi,
this small patch series makes the exim mail transfer agent more configurable at build time.
Exim does not use a framework such as the autotools or CMake for its build-time configuration. Instead it requires to provide a
Makefile(./Local/Makefile) with variables set appropriately. ThisMakefileis then included by the mainMakefile.Quoted 8 lines
These patches allow to use a custom configuration
Makefileprovided by the user and to tweak (in the Buildroot config) the user that will run the exim processes.Luca
Luca Ceresoli (3):
exim: allow using a custom configuration fileThanks, I've applied this patch.
Quoted 2 lines
exim: make
EXIM_USERconfigurable
exim: generate the user with automatic uid
Quoted 6 lines
However, I've for now rejected those two patches. The reason is that I don't think we should add an option to customize the user with which each and every daemon is started. Buildroot should use a sane default option, and for additional configuration, leave it to the Buildroot user to use
BR2_ROOTFS_USERS_TABLESto create any additional/custom user that may be needed.
I generally agree with you.
The problem with exim is that it does not allow to configure its user from a runtime configuration file, because it's hard-coded in the binary. The only way to select a user is to change the build-time configuration.
So, it's true that one can easily create a new user for exim using BR2_ROOTFS_USERS_TABLES. But currently the only way to tell exim to use that user is to supply an entire config file (which is possible thanks to patch 1 that you've just applied). This would bring out of sync from changes in the config file coming from upstream (exim as well as Buildroot).
Patch 2 allows one to keep the Buildroot-provided configuration file, and Buildroot would take care of tweaking the one line needed to actually use the new username.
I hope this clarifies the use case.
Bye,
Signature · 1 line
Luca