All of lore.kernel.org
 help / color / mirror / Atom feed
From: Akira Yokosawa <akiyks@gmail.com>
To: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
Cc: perfbook@vger.kernel.org, Akira Yokosawa <akiyks@gmail.com>
Subject: Re: [GIT PULL v4] Trial of labeling lines in code snippets
Date: Sun, 5 Aug 2018 00:15:08 +0900	[thread overview]
Message-ID: <1acdf809-564b-3557-e869-eaf0c2f64af4@gmail.com> (raw)
In-Reply-To: <e8301596-cb58-491d-89aa-ad9a35d21b66@gmail.com>

On 2018/08/04 20:57:55 +0900, Akira Yokosawa wrote:
> Hi Paul,
> 
> This is v4 of the trial of fancyvrb scheme.
> 
> Changes since v3:
> 
>     o gen_snippet_mk.pl: Add check of failure mode observed by Paul.
>     o gen_snippet.mk.sh: Remove broken CodeSamples/snippets.mk when the failure
>       is caught.
>     o Add dependencies in Makefile regarding the generation and extraction
>       scripts/sub makefile.
>     o Change "make clean" so that it does not depend on CodeSamples/snippets.mk
>       and remove the sub makefile.
> 
> I'm not sure these changes resolve the issue you encountered.
> Does "make clean; make" work for you now?

I think I found another set of missing dependencies.
At the HEAD of this branch, after the 1st make succeeds,

    (1) remove *.fcv files, and
    (2) just do "make"

does nothing.

This is because the main Makefile does not know the dependencies
of perfbook_flat.tex to *.fcv files.

Those dependencies require yet another sub-makefile to be generated.
I'll append another commit to do so.

I think "make clean; make" should still work as the current form.

        Thanks, Akira

> 
>         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-20180804a
> 
> for you to fetch changes up to dd1559612728fcf8726a5b367147cc51ca8f27a8:
> 
>   Add dependencis to scripts for extracting code snippets (2018-08-04 19:36:07 +0900)
> 
> ----------------------------------------------------------------
> Akira Yokosawa (11):
>       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
> 
>  .gitignore                              |   2 +
>  CodeSamples/api-pthreads/api-pthreads.h |  18 +--
>  CodeSamples/toolsoftrade/forkjoinvar.c  |  12 +-
>  CodeSamples/toolsoftrade/pcreate.c      |   7 +-
>  Makefile                                |  15 ++-
>  perfbook.tex                            |  21 ++++
>  toolsoftrade/toolsoftrade.tex           | 211 +++++++++++---------------------
>  utilities/fcvextract.pl                 | 168 +++++++++++++++++++++++++
>  utilities/gen_snippet_mk.pl             |  72 +++++++++++
>  utilities/gen_snippet_mk.sh             |   8 ++
>  utilities/hyphen2endash.sh              |   3 +
>  11 files changed, 379 insertions(+), 158 deletions(-)
>  create mode 100755 utilities/fcvextract.pl
>  create mode 100755 utilities/gen_snippet_mk.pl
>  create mode 100644 utilities/gen_snippet_mk.sh
> 


  reply	other threads:[~2018-08-04 15:15 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-07-21  0:19 [NOT YET PULL] Trial of labeling lines in code snippets Akira Yokosawa
2018-07-21 16:51 ` Paul E. McKenney
2018-07-22 15:43   ` Akira Yokosawa
2018-07-22 22:49     ` Paul E. McKenney
2018-07-23 15:09       ` Akira Yokosawa
2018-07-23 15:12         ` Akira Yokosawa
2018-07-23 21:03           ` Paul E. McKenney
2018-07-30 16:04 ` [NOT YET PULL v2] " Akira Yokosawa
2018-07-31 15:10   ` Paul E. McKenney
2018-08-01 15:24     ` Akira Yokosawa
2018-08-01 15:59       ` Paul E. McKenney
2018-08-01 22:14         ` Akira Yokosawa
2018-08-02  0:22           ` Paul E. McKenney
2018-08-02 22:34 ` [GIT PULL v3] " Akira Yokosawa
2018-08-03 14:35   ` Paul E. McKenney
2018-08-03 15:02     ` Akira Yokosawa
2018-08-03 15:49     ` Akira Yokosawa
2018-08-03 16:45       ` Paul E. McKenney
2018-08-03 23:12         ` Akira Yokosawa
2018-08-03 23:43           ` Paul E. McKenney
2018-08-04  0:00             ` Akira Yokosawa
2018-08-04 11:57 ` [GIT PULL v4] " Akira Yokosawa
2018-08-04 15:15   ` Akira Yokosawa [this message]
2018-08-05  0:06 ` [GIT PULL v5] " Akira Yokosawa
2018-08-05 16:35   ` Paul E. McKenney
2018-08-05 16:54     ` Paul E. McKenney
2018-08-05 21:58       ` Akira Yokosawa
2018-08-05 22:47         ` Paul E. McKenney
2018-08-05 23:26           ` Akira Yokosawa
2018-08-05 23:40             ` Paul E. McKenney

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1acdf809-564b-3557-e869-eaf0c2f64af4@gmail.com \
    --to=akiyks@gmail.com \
    --cc=paulmck@linux.vnet.ibm.com \
    --cc=perfbook@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.