public inbox for [email protected]  

help / color / mirror / Atom feed

From: Pierre-Alain TORET <[email protected]>
To: Devrim Gündüz <[email protected]>
Cc: [email protected] <[email protected]>
Subject: Re: patch postgres user .bash_profile
Date: Mon, 11 Dec 2017 05:00:35 -0500
Message-ID: <jDU13z4IsjAXv9RCRlNvd-dBJnVI2cukWYVRVBvBS19GTftPbg8pPdOfmQHgKlMZQNPXY_ZlVW4MHTDTzkED4z47c1NIlJM3aiyoS4xwq0Y=@protonmail.com> (raw)
In-Reply-To: <[email protected]>
References: <fLXGhRZhBFT18F4i-ds5OEhseUBpoqdf5zqvaszpQyR-4Ce2-g46xZhH_5C3xcFCyAZT-R0VAiJBNBekE9bfuVgJGs7wkbapefzjgGIEE3A=@protonmail.com>
	<[email protected]>

Devrim,
​
>-------- Original Message --------
>Subject: Re: patch postgres user .bash_profile
>Local Time: 10 December 2017 8:21 PM
>UTC Time: 10 December 2017 19:21
>From: [email protected]
>To: Pierre-Alain TORET <[email protected]>, [email protected] <[email protected]>
>
>
> On Fri, 2017-12-08 at 06:17 -0500, Pierre-Alain TORET wrote:
>>I'm sending you a little patch to improve a the usability of the user
>> postgres by adding the binaries to the PATH variable in its .bash_profile.
>>Don't hesitate to comment or criticize but I think it will make the life of
>> user easier, especially those installing the package for the first time.
>>
>> Thanks for the patch.
>>
>> The policy is adding the binaries which are cross-version compatible under
>> $PATH (/usr/bin in this case), and keep others in their own directories. Your
>> patch will probably break the latter, because we want users to specify full
>> path of the "other" binaries, so that things won't be broken.
>>
>> So, unfortunately I will reject this.
>>

Thanks for your explanations, this makes perfect sense in rejecting this patch.
Sorry for the noise.

Regards,

view thread (3+ messages)

reply

Reply instructions:

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

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected]
  Subject: Re: patch postgres user .bash_profile
  In-Reply-To: <jDU13z4IsjAXv9RCRlNvd-dBJnVI2cukWYVRVBvBS19GTftPbg8pPdOfmQHgKlMZQNPXY_ZlVW4MHTDTzkED4z47c1NIlJM3aiyoS4xwq0Y=@protonmail.com>

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

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox