From mboxrd@z Thu Jan 1 00:00:00 1970 From: Oren Laadan Subject: Re: [PATCH 4/4] Conditionally force on namespaces Date: Mon, 08 Jun 2009 02:21:29 -0400 Message-ID: <4A2CADE9.6050906@cs.columbia.edu> References: <20090605191525.C3F1C2F2@kernel> <20090605191530.04562DD1@kernel> <20090605193538.GA22848@us.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20090605193538.GA22848-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: containers-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org Errors-To: containers-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org To: "Serge E. Hallyn" Cc: containers-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org, Dave Hansen List-Id: containers.vger.kernel.org Serge E. Hallyn wrote: > Quoting Dave Hansen (dave-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org): >> This should work around the compile issues that Nathan >> pointed out yesterday. It's too bad that 'select' is >> such a blunt object. We could surely use a soft select >> or something. But this hacks around it a bit. >> >> Create a separate "*_NS_DEP" Kconfig option for the >> dependencies for each namespace. Make the real config >> option and the c/r 'select' bot dependent on the new >> common one. >> >> This should at least keep compile errors from being >> introduced. Whether this is rude or not is a separate >> questions. :) > > Yeah I'm not sure I'm a fan of this... > > While I hate chasing down missing dependencies, maybe > CONFIG_CHECKPOINT should just depend on all the namespaces > instead of selecting them? > Current c/r can work without namespaces - it should not depend on CONFIG_NAMESPACE (though it can suggest). Oren.