All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Michael S. Tsirkin" <mst@redhat.com>
To: Parav Pandit <parav@nvidia.com>
Cc: Cornelia Huck <cohuck@redhat.com>,
	"virtio-dev@lists.oasis-open.org"
	<virtio-dev@lists.oasis-open.org>,
	"virtio-comment@lists.oasis-open.org"
	<virtio-comment@lists.oasis-open.org>,
	Shahaf Shuler <shahafs@nvidia.com>
Subject: Re: [PATCH] content.tex Fix Driver notifications label
Date: Wed, 15 Feb 2023 19:20:10 -0500	[thread overview]
Message-ID: <20230215191830-mutt-send-email-mst@kernel.org> (raw)
In-Reply-To: <PH0PR12MB5481A47EF49115D98E310521DCA39@PH0PR12MB5481.namprd12.prod.outlook.com>

On Wed, Feb 15, 2023 at 09:53:55PM +0000, Parav Pandit wrote:
> 
> > From: Michael S. Tsirkin <mst@redhat.com>
> > Sent: Wednesday, February 15, 2023 8:00 AM
> > 
> > btw did you try to generate a diff pdf? when changing labels it is sometimes
> > necessary to keep old ones around too as diff needs old and new links to work.
> 
> I generated the new pdf before pushing the patch. 
> I couldn't generate the diff pdf using ./makediffpdf.sh
> 
> I get error on running makediff.sh virtio.tex
> 
> Warning: command \@chapapp containing @ found in
> Warning: virtio.tex.
> Warning: consider using --makeatletter if the result is not compilable.
> Warning: command \@chapapp containing @ found in
> Warning: ./main.tex.
> Warning: consider using --makeatletter if the result is not compilable.
> Submodule path 'latexdiff' not initialized
> Maybe you want to use 'update --init'?
> make: Entering directory '/images/parav/upstream/virtio-spec/latexdiff'
> make: *** No rule to make target 'mkdirs'.  Stop.
> make: Leaving directory '/images/parav/upstream/virtio-spec/latexdiff'
> ./makediff.sh: line 51: ./latexdiff-fast: No such file or directory


Hmm. does this help:


diff --git a/makediff.sh b/makediff.sh
index c925c5f..f65a2b4 100755
--- a/makediff.sh
+++ b/makediff.sh
@@ -44,7 +44,7 @@ latexpand $MAIN -o flat.tex
 cd "${cur}"
 #wget http://mirror.math.ku.edu/tex-archive/support/latexdiff/latexdiff-fast
 #chmod +x latexdiff-fast
-git submodule update latexdiff
+git submodule update --init latexdiff
 make -C latexdiff mkdirs dist/latexdiff-fast
 ln -fs ./latexdiff/dist/latexdiff-fast ./latexdiff-fast
 ./latexdiff-fast \


  reply	other threads:[~2023-02-16  0:20 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-02-03 18:35 [PATCH] content.tex Fix Driver notifications label Parav Pandit
2023-02-10 11:23 ` [virtio-dev] " Cornelia Huck
2023-02-15  5:31   ` Parav Pandit
2023-02-15 11:14   ` Michael S. Tsirkin
2023-02-15 12:19     ` [virtio-comment] " Cornelia Huck
2023-02-15 12:59       ` Michael S. Tsirkin
2023-02-15 15:08         ` [virtio-comment] " Cornelia Huck
2023-02-15 21:53         ` Parav Pandit
2023-02-16  0:20           ` Michael S. Tsirkin [this message]
2023-02-16  5:36             ` Parav Pandit

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=20230215191830-mutt-send-email-mst@kernel.org \
    --to=mst@redhat.com \
    --cc=cohuck@redhat.com \
    --cc=parav@nvidia.com \
    --cc=shahafs@nvidia.com \
    --cc=virtio-comment@lists.oasis-open.org \
    --cc=virtio-dev@lists.oasis-open.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.