[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH for 2.10 0/8] docs: fix broken paths
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH for 2.10 0/8] docs: fix broken paths |
Date: |
Fri, 28 Jul 2017 13:34:11 -0500 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 |
On 07/27/2017 08:25 PM, Philippe Mathieu-Daudé wrote:
> Hi, following Cleber Rosa example I cleaned more invalid references.
>
> Eric said this can wait 2.11, however these patches don't change any code
> generated, I think the 2.10 users deserve an up-to-date doc :p
I think what I said was: this doesn't change code and doc updates are
find during freeze, so this is a GOOD candidate for 2.10; but if it
misses 2.10, slipping to 2.11 doesn't hurt. (If I didn't say that, it's
what I should have done). But you caught on to my intention, even if in
the process it sounds like I was asking you to wait.
>
> I used the following command (and consider include it in some CI test job):
>
> $ git grep docs/ \
> | sed -ne "s/.* \(docs[^ :)}\"\']*\).*/\1/p" \
> | sed -e 's/\(.*\)\.$/\1/p' | sort -u | while read p;do
> ls -ld $p 1>/dev/null
Wait. What? 'ls -ld' with stdout sent to /dev/null? (I never cease to
be amazed at the random shell constructs that people come up with). I
guess you are just trying to check for file existence, by going off of
the stderr droppings from ls? 'ls -d' is less effort than 'ls -ld'; and
if all you care about is stderr messages, spawning one ls per filename
is much slower than just doing it wholesale via xargs to minimize ls calls:
git grep ... | sort -u | xargs ls -d >/dev/null
But then again, efficiency in generating the list of problems isn't a
bottleneck.
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH for 2.10 3/8] docs: fix broken paths to docs/devel/qapi-code-gen.txt, (continued)
- [Qemu-devel] [PATCH for 2.10 3/8] docs: fix broken paths to docs/devel/qapi-code-gen.txt, Philippe Mathieu-Daudé, 2017/07/27
- [Qemu-devel] [PATCH for 2.10 4/8] docs: fix broken paths to docs/devel/atomics.txt, Philippe Mathieu-Daudé, 2017/07/27
- [Qemu-devel] [PATCH for 2.10 6/8] docs: fix broken paths to docs/config/ich9-ehci-uhci.cfg, Philippe Mathieu-Daudé, 2017/07/27
- [Qemu-devel] [PATCH for 2.10 5/8] docs: fix broken paths to docs/devel/tracing.txt, Philippe Mathieu-Daudé, 2017/07/27
- [Qemu-devel] [PATCH for 2.10 7/8] docs: fix broken paths to docs/specs/ivshmem-spec.txt, Philippe Mathieu-Daudé, 2017/07/27
- [Qemu-devel] [PATCH for 2.10 8/8] docs: fix broken paths to docs/spin/, Philippe Mathieu-Daudé, 2017/07/27
- Re: [Qemu-devel] [PATCH for 2.10 0/8] docs: fix broken paths, Philippe Mathieu-Daudé, 2017/07/27
- Re: [Qemu-devel] [PATCH for 2.10 0/8] docs: fix broken paths, Michael Tokarev, 2017/07/28
- Re: [Qemu-devel] [PATCH for 2.10 0/8] docs: fix broken paths, Paolo Bonzini, 2017/07/28
- Re: [Qemu-devel] [PATCH for 2.10 0/8] docs: fix broken paths,
Eric Blake <=