Re: patch postgres user .bash_profile

From: Pierre-Alain TORET <pierre-alain(dot)toret(at)protonmail(dot)com>
To: Devrim Gündüz <devrim(at)gunduz(dot)org>
Cc: "pgsql-pkg-yum(at)postgresql(dot)org" <pgsql-pkg-yum(at)postgresql(dot)org>
Subject: Re: patch postgres user .bash_profile
Date: 2017-12-11 10:00:35
Message-ID: jDU13z4IsjAXv9RCRlNvd-dBJnVI2cukWYVRVBvBS19GTftPbg8pPdOfmQHgKlMZQNPXY_ZlVW4MHTDTzkED4z47c1NIlJM3aiyoS4xwq0Y=@protonmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-pkg-yum

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: devrim(at)gunduz(dot)org
>To: Pierre-Alain TORET <pierre-alain(dot)toret(at)protonmail(dot)com>, pgsql-pkg-yum(at)postgresql(dot)org <pgsql-pkg-yum(at)postgresql(dot)org>
>
>
> 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,

In response to

Browse pgsql-pkg-yum by date

  From Date Subject
Next Message Ziyun Audrey Wang 2017-12-11 12:57:52 FW: [SECURITY] Missing vendor name in postgresql96 rpms
Previous Message Michael Vermaes 2017-12-11 08:06:31 9.3 RPMs not signed