From mboxrd@z Thu Jan 1 00:00:00 1970 From: john at keeping.me.uk (John Keeping) Date: Thu, 16 Jan 2014 21:34:24 +0000 Subject: Policy on global variables In-Reply-To: References: <20140116005912.GA20306@dcvr.yhbt.net> <20140116104751.GA26678@dcvr.yhbt.net> <20140116130844.GF7608@serenity.lan> <20140116212152.GH7608@serenity.lan> Message-ID: <20140116213424.GI7608@serenity.lan> On Thu, Jan 16, 2014 at 10:26:08PM +0100, Jason A. Donenfeld wrote: > On Thu, Jan 16, 2014 at 10:21 PM, John Keeping wrote: > > The first step in this direction may actually be useful even if we stick > > with embedding libgit.a. > > So what do you think ought to be done with the global-ctx patch? Merge > it, and then refactor afterward (whenever we "step in this > direction"), to get rid of the global? Or use what we have now > (without the patch) as a starting point for gradually getting rid of > the global? I'm not sure it makes much difference either way. Even if we use libgit2, providing we're not processing more than one request at once we can still use a global cgit_context.