From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753992Ab3K0LbU (ORCPT ); Wed, 27 Nov 2013 06:31:20 -0500 Received: from mail-bk0-f43.google.com ([209.85.214.43]:64289 "EHLO mail-bk0-f43.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751325Ab3K0LbS (ORCPT ); Wed, 27 Nov 2013 06:31:18 -0500 Date: Wed, 27 Nov 2013 12:31:14 +0100 From: Ingo Molnar To: Jiri Olsa Cc: linux-kernel@vger.kernel.org, Frederic Weisbecker , Peter Zijlstra , Namhyung Kim , Mike Galbraith , Stephane Eranian , David Ahern , Adrian Hunter , Arnaldo Carvalho de Melo Subject: Re: [PATCHv2] perf tools: Fix tags/TAGS targets rebuilding Message-ID: <20131127113114.GA10495@gmail.com> References: <1385464890-19876-1-git-send-email-jolsa@redhat.com> <20131126113108.GE2410@gmail.com> <20131126115043.GG1267@krava.brq.redhat.com> <20131126120055.GA6103@gmail.com> <20131126125412.GJ1267@krava.brq.redhat.com> <20131126161316.GA7775@gmail.com> <20131126231650.GA1130@krava.redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20131126231650.GA1130@krava.redhat.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org * Jiri Olsa wrote: > On Tue, Nov 26, 2013 at 05:13:16PM +0100, Ingo Molnar wrote: > > SNIP > > > > > > > # > > > # Needed if no target specified: > > > +# (Except for tags and TAGS targets. The reason is that the > > > +# Makefile does not treat tags/TAGS as targets but as files > > > +# and thus won't rebuilt them once they are in place.) > > > > s/rebuilt/rebuild > > > > Btw., could we perhaps change the targets to 'ctags' and 'etags' > > instead? (That will make it natural to add 'gtags' in the future, > > should anyone want that.) > > my fingers vote against removing 'make tags' (and cscope) > dont care much about the rest though.. ;-) Okay, fair enough, and 'tags/TAGS' targets is also what the main kernel build system uses - so I concur. For your updated v2 patch: Acked-by: Ingo Molnar Thanks, Ingo