Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 0/3] Make exim more configurable
Date: Tue, 15 Jul 2014 21:31:25 +0200	[thread overview]
Message-ID: <20140715213125.1b848b48@free-electrons.com> (raw)
In-Reply-To: <1404489386-7523-1-git-send-email-luca@lucaceresoli.net>

Dear Luca Ceresoli,

On Fri,  4 Jul 2014 17:56:23 +0200, Luca Ceresoli wrote:
> 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. This Makefile is then
> included by the main Makefile.
> 
> These patches allow to use a custom configuration Makefile provided 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 file

Thanks, I've applied this patch.

>   exim: make EXIM_USER configurable
>   exim: generate the user with automatic uid

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_TABLES to create any additional/custom
user that may be needed.

If you don't agree with this decision, let us know and resubmit patches
with more details on why an exim specific mechanism is needed here.

Thanks a lot!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

  parent reply	other threads:[~2014-07-15 19:31 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-04 15:56 [Buildroot] [PATCH 0/3] Make exim more configurable Luca Ceresoli
2014-07-04 15:56 ` [Buildroot] [PATCH 1/3] exim: allow using a custom configuration file Luca Ceresoli
2014-07-04 15:56 ` [Buildroot] [PATCH 2/3] exim: make EXIM_USER configurable Luca Ceresoli
2014-07-04 15:56 ` [Buildroot] [PATCH 3/3] exim: generate the user with automatic uid Luca Ceresoli
2014-07-15 19:31 ` Thomas Petazzoni [this message]
2014-07-16  8:18   ` [Buildroot] [PATCH 0/3] Make exim more configurable Luca Ceresoli
2014-07-16  8:29     ` Thomas Petazzoni
2014-07-16  9:03       ` Luca Ceresoli
2014-07-16 17:14         ` Yann E. MORIN
2014-07-16  8:24   ` Thomas Petazzoni
2014-07-16  8:50     ` Luca Ceresoli
2014-07-16  9:03       ` Thomas Petazzoni
2014-07-16 16:15         ` Luca Ceresoli
2014-07-16 22:47       ` Arnout Vandecappelle

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20140715213125.1b848b48@free-electrons.com \
    --to=thomas.petazzoni@free-electrons.com \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox