Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 17 Sep 2017 13:14:48 +0930
From:      Shane Ambler <FreeBSD@ShaneWare.Biz>
To:        blubee blubeeme <gurenchan@gmail.com>
Cc:        FreeBSD Ports <freebsd-ports@freebsd.org>
Subject:   Re: Extra Clang Tools
Message-ID:  <4d3c0359-3866-bd95-b716-4c7f78e56eeb@ShaneWare.Biz>
In-Reply-To: <CALM2mEkbR-NoqB%2B1pgo0v8F7PMhWt8ZK9-T6G=rsomN%2BFhVgjg@mail.gmail.com>
References:  <CALM2mE=zgJw_YKdUdKM1c=uD3BNuE7zX=DB3aGvpYs79JF537g@mail.gmail.com> <6ec60c86-d1c0-b556-a95a-d9d257c28c7c@ShaneWare.Biz> <CALM2mEkbR-NoqB%2B1pgo0v8F7PMhWt8ZK9-T6G=rsomN%2BFhVgjg@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
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:
<snip>
> 
> 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




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4d3c0359-3866-bd95-b716-4c7f78e56eeb>