This is the mail archive of the cygwin-apps mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] setup: allow running as non-admin


On Thu, Nov 07, 2013 at 10:52:00AM +1100, Shaddy Baddah wrote:
>Hi,
>
>As discussed, please find the patch to allow setup to be run as a
>non-admin user include clear text.
>
>-- 
>Regards,
>Shaddy
>

Only a very minor comment:

>+
>+        // Note, this is necessary to avoid an infinite loop.
>+        // The understanding is that pre-Vista, the runas verb will not

This indentation looks wrong.

>+		// result in a privilege elevated process. Therefore we need to
>+		// indicate to the forked process that it should be happy with
>+		// whatever privileges it is run with.

cgf


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]