Re: [GIT PULL v5] Trial of labeling lines in code snippets

["Paul E. McKenney" <paulmck@linux.vnet.ibm.com>]

On Sun, Aug 05, 2018 at 09:06:27AM +0900, Akira Yokosawa wrote:
> Hi Paul,
> 
> This is hopefully the final version of this trial branch.
> 
> Changes since v4:
> 
>     o Instead of generating CodeSamples/snippets.mk and calling it from
>       main Makefile, generate CodeSamples/snippets.d which contains
>       only dependencies and include it in the Makefile.
> 
> This change can fix the error you encountered in the previous version.
> I'm just hoping...

Hmmmm...

	$ make
	sh ./utilities/gen_snippet_d.sh
	CodeSamples/toolsoftrade/pcreate.c --> CodeSamples/toolsoftrade/pcreate@mythread.fcv

But "make 2c" works.  This might be because I don't have the
PERFBOOK_DEFAULT environment variable set?  Except that the Makefile code
looks like this would have the default "make all" have the same effect as
"make 2c".  Color me confused.

Anyway, yes, when I do "make 2c" it correctly reacts to moving labels
around in CodeSamples/toolsoftrade/forkjoinvar.c, so that is good!

Here is the output of "make -v", in case that matters:

	GNU Make 3.81
	Copyright (C) 2006  Free Software Foundation, Inc.
	This is free software; see the source for copying conditions.
	There is NO warranty; not even for MERCHANTABILITY or FITNESS FOR A
	PARTICULAR PURPOSE.

	This program built for x86_64-pc-linux-gnu

							Thanx, Paul

>         Thanks, Akira
> 
> --- 
> The following changes since commit 3b62f67a76e15f8d8f8190a4252362d4ac0cdf23:
> 
>   Regenerating the atomic counter graph on a more modern CPU (2018-07-23 13:47:51 -0700)
> 
> are available in the git repository at:
> 
>   https://github.com/akiyks/perfbook.git trial-fancyvrb-20180804b
> 
> for you to fetch changes up to d9a36224084b2e4ba55797eff0c37de528b59d6b:
> 
>   Add dependencies of perfbook_flat.tex to CodeSamples/.../*.fcv files (2018-08-05 08:44:02 +0900)
> 
> ----------------------------------------------------------------
> Akira Yokosawa (12):
>       toolsoftrade: Add labels in code samples as comments
>       Add script to extract code snippet from code sample
>       toolsoftrade: Reference line in code snippets by label
>       Update hyphen2endash
>       Add *.fcv to .gitignore
>       Automate sub makefile update
>       Specify lmtt font for VerbatimM env
>       toolsoftrade: Use \lnlbl in inline code snippet
>       Add 'linelabel' and 'lineref' environment
>       Catch failure in genaration of CodeSamples/snippets.mk
>       Add dependencis to scripts for extracting code snippets
>       Add dependencies of perfbook_flat.tex to CodeSamples/.../*.fcv files
> 
>  .gitignore                              |   2 +
>  CodeSamples/api-pthreads/api-pthreads.h |  18 +--
>  CodeSamples/toolsoftrade/forkjoinvar.c  |  12 +-
>  CodeSamples/toolsoftrade/pcreate.c      |   7 +-
>  Makefile                                |  25 +++-
>  perfbook.tex                            |  21 ++++
>  toolsoftrade/toolsoftrade.tex           | 211 +++++++++++---------------------
>  utilities/fcvextract.pl                 | 168 +++++++++++++++++++++++++
>  utilities/gen_snippet_d.pl              |  65 ++++++++++
>  utilities/gen_snippet_d.sh              |   8 ++
>  utilities/hyphen2endash.sh              |   3 +
>  11 files changed, 382 insertions(+), 158 deletions(-)
>  create mode 100755 utilities/fcvextract.pl
>  create mode 100755 utilities/gen_snippet_d.pl
>  create mode 100644 utilities/gen_snippet_d.sh
>