[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#865792: reportbug: Allow an arbitrary MUA to be configured (patch)



Hey David,

On Sat, Jun 24, 2017 at 4:15 PM David Steele <steele@debian.org> wrote:
> The "mua" option allowed a custom mua command to be defined, but it
> was only permitted for the list of 5 supported muas. This patch allows
> an arbitrary mua command string to be defined.
>
> The defined mua is set to default, and is added to the list of available
> MUAs.
>
> This also adds a 'mua-version' option, which specifies the mua
> command-line argument for 'no action', which is used in utils.mua_exists()
> as an 'are you there?" check. This only applies to custom muas. The option
> defaults to "--version".
>
> As a side effect, the overloaded definition of 'mua' in utils, where
> it could either be a string or a Mua object, depending on user options,
> is eliminated.
>
> This does not introduce any test failures.

I like your patch, but it doesnt apply anymore (apologies it took so
long to review) - would you please re-base it against the current
master? thanks!

-- 
Sandro "morph" Tosi
My website: http://sandrotosi.me/
Me at Debian: http://wiki.debian.org/SandroTosi
G+: https://plus.google.com/u/0/+SandroTosi


Reply to: