public inbox for [email protected]  

help / color / mirror / Atom feed

From: Devrim Gündüz <[email protected]>
To: Pierre-Alain TORET <[email protected]>
To: [email protected] <[email protected]>
Subject: Re: patch postgres user .bash_profile
Date: Sun, 10 Dec 2017 22:21:45 +0300
Message-ID: <[email protected]> (raw)
In-Reply-To: <fLXGhRZhBFT18F4i-ds5OEhseUBpoqdf5zqvaszpQyR-4Ce2-g46xZhH_5C3xcFCyAZT-R0VAiJBNBekE9bfuVgJGs7wkbapefzjgGIEE3A=@protonmail.com>
References: <fLXGhRZhBFT18F4i-ds5OEhseUBpoqdf5zqvaszpQyR-4Ce2-g46xZhH_5C3xcFCyAZT-R0VAiJBNBekE9bfuVgJGs7wkbapefzjgGIEE3A=@protonmail.com>


Hi,

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.

Regards,
-- 
Devrim Gündüz
EnterpriseDB: https://www.enterprisedb.com
PostgreSQL Consultant, Red Hat Certified Engineer
Twitter: @DevrimGunduz , @DevrimGunduzTR

Attachments:

  [application/pgp-signature] signature.asc (833B, 2-signature.asc)
  download

view thread (3+ messages)  latest in thread

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: <[email protected]>

* 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