Re: [PATCH setup] Add new option --chown-admin

2022-11-30 Thread Christian Franke
Jon Turney wrote: On 04/10/2022 13:05, Christian Franke wrote: Jon Turney wrote: Corinna had some concerns about making the owner a group, rather than a user, which I believe historically caused some difficulties in Cygwin, so I think I'll need to understand that better before making a deci

Re: [PATCH setup] Add new option --chown-admin

2022-11-29 Thread Jon Turney
On 04/10/2022 13:05, Christian Franke wrote: Jon Turney wrote: Corinna had some concerns about making the owner a group, rather than a user, which I believe historically caused some difficulties in Cygwin, so I think I'll need to understand that better before making a decision about this cha

Re: [PATCH setup] Add new option --chown-admin

2022-10-04 Thread Christian Franke
Jon Turney wrote: On 02/09/2022 16:17, Christian Franke wrote: Jon Turney wrote: On 28/08/2022 18:33, Christian Franke wrote: As the 'root_scope' issues are now fixed, here a reworked and enhanced (checkbox, setup.rc entry) version of the original patch from this thread. With the new settin

Re: [PATCH setup] Add new option --chown-admin

2022-09-15 Thread Jon Turney
On 02/09/2022 16:17, Christian Franke wrote: Jon Turney wrote: On 28/08/2022 18:33, Christian Franke wrote: As the 'root_scope' issues are now fixed, here a reworked and enhanced (checkbox, setup.rc entry) version of the original patch from this thread. With the new setting enabled, setup be

Re: [PATCH setup] Add new option --chown-admin

2022-09-02 Thread Christian Franke
Jon Turney wrote: On 28/08/2022 18:33, Christian Franke wrote: As the 'root_scope' issues are now fixed, here a reworked and enhanced (checkbox, setup.rc entry) version of the original patch from this thread. With the new setting enabled, setup behaves like other install tools when run eleva

Re: [PATCH setup] Add new option --chown-admin

2022-09-02 Thread Jon Turney
On 28/08/2022 18:33, Christian Franke wrote: As the 'root_scope' issues are now fixed, here a reworked and enhanced (checkbox, setup.rc entry) version of the original patch from this thread. With the new setting enabled, setup behaves like other install tools when run elevated: The installatio

Re: [PATCH setup] Add new option --chown-admin

2022-08-28 Thread Christian Franke
As the 'root_scope' issues are now fixed, here a reworked and enhanced (checkbox, setup.rc entry) version of the original patch from this thread. With the new setting enabled, setup behaves like other install tools when run elevated: The installation is then also protected against accidental m

Re: [PATCH setup] Add new option --chown-admin

2022-08-26 Thread Christian Franke
Jon Turney wrote: On 23/08/2022 18:27, Christian Franke wrote: Jon Turney wrote: On 12/07/2022 13:50, Jon Turney wrote: [Replying to the right list this time...] On 09/07/2022 13:21, Christian Franke wrote: [...] The UserSettings ctor has a somewhat hidden side effect which sets root_scope

Re: [PATCH setup] Add new option --chown-admin

2022-08-26 Thread Jon Turney
On 23/08/2022 18:27, Christian Franke wrote: Jon Turney wrote: On 12/07/2022 13:50, Jon Turney wrote: [Replying to the right list this time...] On 09/07/2022 13:21, Christian Franke wrote: [...] The UserSettings ctor has a somewhat hidden side effect which sets root_scope correctly:   Use

Re: [PATCH setup] Add new option --chown-admin

2022-08-23 Thread Christian Franke
Jon Turney wrote: On 12/07/2022 13:50, Jon Turney wrote: [Replying to the right list this time...] On 09/07/2022 13:21, Christian Franke wrote: [...] The UserSettings ctor has a somewhat hidden side effect which sets root_scope correctly:   UserSettings::UserSettings(...);    open_settings

Re: [PATCH setup] Add new option --chown-admin

2022-08-23 Thread Jon Turney
On 12/07/2022 13:50, Jon Turney wrote: [Replying to the right list this time...] On 09/07/2022 13:21, Christian Franke wrote: [...] The UserSettings ctor has a somewhat hidden side effect which sets root_scope correctly:   UserSettings::UserSettings(...);    open_settings("setup.rc", ...);

Re: [PATCH setup] Add new option --chown-admin

2022-07-12 Thread Jon Turney
[Replying to the right list this time...] On 09/07/2022 13:21, Christian Franke wrote: Jon Turney wrote: On 07/07/2022 15:45, Christian Franke wrote: Jon Turney wrote: On 06/07/2022 17:34, Christian Franke wrote: Jon Turney wrote: On 06/07/2022 08:14, Christian Franke wrote: [...] BTW: '

Re: [PATCH setup] Add new option --chown-admin

2022-07-07 Thread Christian Franke
Christian Franke wrote: It does fix the regression, it adds a new installation flavor which fixes it as a side effect. It does *not* fix the regression, of course

Re: [PATCH setup] Add new option --chown-admin

2022-07-07 Thread Christian Franke
Jon Turney wrote: On 06/07/2022 17:34, Christian Franke wrote: Jon Turney wrote: On 06/07/2022 08:14, Christian Franke wrote: If an installer is run elevated, the installed files will be typically owned by the local administrator (or in some cases SYSTEM or TrustedInstaller) instead of the cu

Re: [PATCH setup] Add new option --chown-admin

2022-07-07 Thread Jon Turney
On 06/07/2022 17:34, Christian Franke wrote: Jon Turney wrote: On 06/07/2022 08:14, Christian Franke wrote: If an installer is run elevated, the installed files will be typically owned by the local administrator (or in some cases SYSTEM or TrustedInstaller) instead of the current user. This is

Re: [PATCH setup] Add new option --chown-admin

2022-07-06 Thread Christian Franke
Jon Turney wrote: On 06/07/2022 08:14, Christian Franke wrote: If an installer is run elevated, the installed files will be typically owned by the local administrator (or in some cases SYSTEM or TrustedInstaller) instead of the current user. This is not the case for a Cygwin "All Users" instal

Re: [PATCH setup] Add new option --chown-admin

2022-07-06 Thread Jon Turney
On 06/07/2022 08:14, Christian Franke wrote: If an installer is run elevated, the installed files will be typically owned by the local administrator (or in some cases SYSTEM or TrustedInstaller) instead of the current user. This is not the case for a Cygwin "All Users" installation. The files a

[PATCH setup] Add new option --chown-admin

2022-07-06 Thread Christian Franke
If an installer is run elevated, the installed files will be typically owned by the local administrator (or in some cases SYSTEM or TrustedInstaller) instead of the current user. This is not the case for a Cygwin "All Users" installation. The files are then not protected from accidental changes