X-Recipient: archive-cygwin@delorie.com
X-SWARE-Spam-Status: No, hits=-1.7 required=5.0	tests=AWL,BAYES_00,RCVD_IN_DNSWL_NONE,T_RP_MATCHES_RCVD,UNPARSEABLE_RELAY
X-Spam-Check-By: sourceware.org
Message-ID: <4F4E9862.4060108@t-online.de>
Date: Wed, 29 Feb 2012 22:28:02 +0100
From: Christian Franke <Christian.Franke@t-online.de>
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:10.0.2) Gecko/20120216 Firefox/10.0.2 SeaMonkey/2.7.2
MIME-Version: 1.0
To: cygwin@cygwin.com
Subject: Re: [PATCH] base-files-4.0.9: Change prompt if running with admin rights
References: <4F4BB942.80906@t-online.de> <20120228194715.GB6761@jethro.local.lan>
In-Reply-To: <20120228194715.GB6761@jethro.local.lan>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
X-IsSubscribed: yes
Mailing-List: contact cygwin-help@cygwin.com; run by ezmlm
List-Id: <cygwin.cygwin.com>
List-Subscribe: <mailto:cygwin-subscribe@cygwin.com>
List-Archive: <http://sourceware.org/ml/cygwin/>
List-Post: <mailto:cygwin@cygwin.com>
List-Help: <mailto:cygwin-help@cygwin.com>, <http://sourceware.org/ml/#faqs>
Sender: cygwin-owner@cygwin.com
Mail-Followup-To: cygwin@cygwin.com
Delivered-To: mailing list cygwin@cygwin.com

David Sastre Medina wrote:
> On Mon, Feb 27, 2012 at 06:11:30PM +0100, Christian Franke wrote:
>> This is an updated version of:
>> http://cygwin.com/ml/cygwin/2011-04/msg00020.html
>> Uses the detection method suggested here:
>> http://cygwin.com/ml/cygwin/2011-04/msg00372.html
>>
>> Tested with bash, dash, mksh, posh, and zsh (with symlink
>> /etc/zprofile ->  profile) on cygwin 1.7.11-1 and Win7 x64.
> I've revisited this idea several times, with the purpose of adding
> /usr/sbin to the PATH and coloring the prompt differently (red?) for admins.

Makes sense. Please also include the '#' as the very old & very well 
known root prompt.

A Cygwin shell run as admin has more *effective* rights than a Windows 
console run admin. So there should be a "be careful" sign.


> I'll see if I can come up with some variation on your patch that does
> all this.
>

Thanks,
Christian


--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple

