From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754009AbbLJPL6 (ORCPT ); Thu, 10 Dec 2015 10:11:58 -0500 Received: from mx1.redhat.com ([209.132.183.28]:53418 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750906AbbLJPL4 (ORCPT ); Thu, 10 Dec 2015 10:11:56 -0500 Date: Thu, 10 Dec 2015 09:11:55 -0600 From: Josh Poimboeuf To: Namhyung Kim Cc: Peter Zijlstra , Ingo Molnar , Arnaldo Carvalho de Melo , linux-kernel@vger.kernel.org, Jiri Olsa Subject: Re: [PATCH v2 00/14] perf tools: Move perf subcommand framework into lib/tools Message-ID: <20151210151155.GE29872@treble.redhat.com> References: <20151210020012.GC13790@sejong> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20151210020012.GC13790@sejong> User-Agent: Mutt/1.5.23.1-rc1 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Dec 10, 2015 at 11:00:12AM +0900, Namhyung Kim wrote: > On Mon, Dec 07, 2015 at 10:21:38PM -0600, Josh Poimboeuf wrote: > > Ingo suggested that I factor out the perf subcommand functionality > > (originally copied from git) into tools/lib so that it can be used by > > stacktool[*] and possibly others. > > > > All the subcommand option handling files have been moved into libapi, > > including parse-options.c, run-command.c, exec_cmd.c, help.c, usage.c, > > and their dependencies. Since several of the dependencies are more > > general-purpose "utilities" which aren't directly related to > > subcommands, I put everything in 'tools/lib/api/util'. They're linked > > into perf as part of the libapi.a library. > > > > Patches 1-13 do some cleanups and splitting up of the code in > > preparation for the move. > > > > Patch 14 does the actual moving of the files into tools/lib/api/util. > > > > Based on tip/perf/core. > > > > [*] https://lkml.kernel.org/r/cover.1445443144.git.jpoimboe@redhat.com > > Is there a public repo that I can pull and test? When I post v3 (probably later today), I'll add a repo link. -- Josh