From owner-freebsd-ports@freebsd.org Sun Sep 17 03:44:58 2017 Return-Path: Delivered-To: freebsd-ports@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 0521DE1FFBF for ; Sun, 17 Sep 2017 03:44:58 +0000 (UTC) (envelope-from FreeBSD@shaneware.biz) Received: from ipmail01.adl6.internode.on.net (ipmail01.adl6.internode.on.net [150.101.137.136]) by mx1.freebsd.org (Postfix) with ESMTP id 9514370361 for ; Sun, 17 Sep 2017 03:44:56 +0000 (UTC) (envelope-from FreeBSD@shaneware.biz) Received: from ppp118-210-151-17.bras1.adl6.internode.on.net (HELO leader.local) ([118.210.151.17]) by ipmail01.adl6.internode.on.net with ESMTP; 17 Sep 2017 13:14:49 +0930 Subject: Re: Extra Clang Tools To: blubee blubeeme Cc: FreeBSD Ports References: <6ec60c86-d1c0-b556-a95a-d9d257c28c7c@ShaneWare.Biz> From: Shane Ambler Message-ID: <4d3c0359-3866-bd95-b716-4c7f78e56eeb@ShaneWare.Biz> Date: Sun, 17 Sep 2017 13:14:48 +0930 User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-AU Content-Transfer-Encoding: 7bit X-BeenThere: freebsd-ports@freebsd.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Porting software to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 17 Sep 2017 03:44:58 -0000 On 16/09/2017 23:22, blubee blubeeme wrote: > Howdy > > I made a few changes to the devel/llvm40/Makefile and added pp-trace as the > last line of EXTRA_COMMANDS > > Then I rebuilt llvm40, then I noticed that the pp-trace executable is > built, here's a output of the work directory grepping for pp-trace: > > So it now gets built but not installed; is it possible to have the port > updated to move these files to the proper after they are built? Create a new report at https://bugs.freebsd.org with a patch for the Makefile and pkg-plist. While the pp-trace binary has not been included, the docs for it are already in the existing packages, the makefile is done in a way that adding it to the command list adds it to the package. To make the following patch you would use diff -udp Makefile.orig Makefile diff -udp pkg-plist.orig pkg-plist --- Makefile.orig 2017-09-17 13:02:06.907563000 +0930 +++ Makefile 2017-09-17 13:02:16.043096000 +0930 @@ -164,7 +164,8 @@ EXTRAS_COMMANDS+= \ clang-reorder-fields \ clang-tidy \ find-all-symbols \ - modularize + modularize \ + pp-trace EXTRAS_LIBS= libclangApplyReplacements \ libclangChangeNamespace \ libclangIncludeFixer \ --- pkg-plist.orig 2017-05-26 17:46:41.237943000 +0930 +++ pkg-plist 2017-09-17 12:46:44.526703000 +0930 @@ -58,6 +58,7 @@ bin/sancov%%LLVM_SUFFIX%% %%EXTRAS%%bin/clang-tidy%%LLVM_SUFFIX%% %%EXTRAS%%bin/find-all-symbols%%LLVM_SUFFIX%% %%EXTRAS%%bin/modularize%%LLVM_SUFFIX%% +%%EXTRAS%%bin/pp-trace%%LLVM_SUFFIX%% %%LLD%%bin/lld%%LLVM_SUFFIX%% %%LLD%%bin/lld-link%%LLVM_SUFFIX%% %%LIT%%bin/lit%%LLVM_SUFFIX%% -- FreeBSD - the place to B...Software Developing Shane Ambler