Please revert. This is not the solution. It is also *blatantly* untrue that nobody cares. We (at least Niels, Stefan, and I--we didn't discuss this with Peter) do care, but we want a proper solution, and this is not it. We agree that there should not be a call to umask in the code, but we should also check the modes that are used in calls to creat, open, mkdir, etc.
If you think this umask thing is important for you, say so in the bug
report and increase the priority accordingly. You can also *talk* to
people if something is important to you.
Fabian wrote:
> Update of /cvsroot/monetdb/MonetDB/src/gdk
> In directory sc8-pr-cvs16.sourceforge.net:/tmp/cvs-serv7068
>
> Modified Files:
> gdk_utils.mx
> Log Message:
> Because nobody cares but me.
> Don't wipe out the user's umask if gdk_dont_unset_umask is set in the
> environment. Can't use GDKgetenv as that stuff is not yet initialised
> at this point.
>
>
> Index: gdk_utils.mx
> ===================================================================
> RCS file: /cvsroot/monetdb/MonetDB/src/gdk/gdk_utils.mx,v
> retrieving revision 1.196
> retrieving revision 1.197
> diff -u -d -r1.196 -r1.197
> --- gdk_utils.mx 14 Nov 2007 13:22:59 -0000 1.196
> +++ gdk_utils.mx 28 Nov 2007 11:59:39 -0000 1.197
> @@ -1636,7 +1636,8 @@
> if (GDKrecovery && unlink(GDKLOCK) < 0) {
> GDKfatal("GDKlockHome: unlock DB failed\n");
> }
> - umask(0000);
> + if (getenv("gdk_dont_unset_umask") == NULL)
> + umask(0000);
> if (MT_lockf(GDKLOCK, F_TLOCK, 4, 1) < 0) {
> GDKlockFile = 0;
> GDKfatal("GDKlockHome: Database lock '%s' denied\n", GDKLOCK);
>
>
> -------------------------------------------------------------------------
> SF.Net email is sponsored by: The Future of Linux Business White Paper
> from Novell. From the desktop to the data center, Linux is going
> mainstream. Let it simplify your IT future.
> http://altfarm.mediaplex.com/ad/ck/8857-50307-18918-4
> _______________________________________________
> Monetdb-checkins mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/monetdb-checkins
--
Sjoerd Mullender
signature.asc
Description: OpenPGP digital signature
------------------------------------------------------------------------- SF.Net email is sponsored by: The Future of Linux Business White Paper from Novell. From the desktop to the data center, Linux is going mainstream. Let it simplify your IT future. http://altfarm.mediaplex.com/ad/ck/8857-50307-18918-4
_______________________________________________ Monetdb-checkins mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/monetdb-checkins
