[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL v2 10/37] scripts: Add archive-source.sh
From: |
Kamil Rytarowski |
Subject: |
Re: [Qemu-devel] [PULL v2 10/37] scripts: Add archive-source.sh |
Date: |
Sun, 10 Sep 2017 12:15:34 +0200 |
User-agent: |
Mozilla/5.0 (X11; NetBSD amd64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 |
On 10.09.2017 04:05, Fam Zheng wrote:
> On Sat, 09/09 17:08, Kamil Rytarowski wrote:
>>>>> +submodules=$(git submodule foreach --recursive --quiet 'echo $name')
>>>>> +
>>>>> +if test -n "$submodules"; then
>>>>> + {
>>>>> + git ls-files
>>>>> + for sm in $submodules; do
>>>>> + (cd $sm; git ls-files) | sed "s:^:$sm/:"
>>>>> + done
>>>>> + } | grep -x -v $(for sm in $submodules; do echo "-e $sm"; done) >
>>>>> $1.list
>>>>
>>>> Supporting '-e something' in a tar -T listfile seems to
>>>> be GNU tar specific. Do we care?
>>>
>>> The "-e $sm" will go to the grep command line, so it isn't GNU specific, is
>>> it?
>>>
>>
>> BSD OSes use GNU grep(1) the latest version GPLv2. There is a work in
>> progress BSD grep(1), but not turned on by default as it's slower.
>>
>> The -e option is supported by SmartOS grep(1).
>
> Is there a more portable way to filter out multiple patterns?
>
> Fam
>
I don't think that there is needed more portability. If so, there is an
option to detect "ggrep" (GNU grep) and if it exists - use it.
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PULL v2 05/37] docker: Update ubuntu image, (continued)
- [Qemu-devel] [PULL v2 05/37] docker: Update ubuntu image, Fam Zheng, 2017/09/09
- [Qemu-devel] [PULL v2 06/37] docker: Enable features explicitly in test-full, Fam Zheng, 2017/09/09
- [Qemu-devel] [PULL v2 07/37] tests/docker: Clean up paths, Fam Zheng, 2017/09/09
- [Qemu-devel] [PULL v2 08/37] gitignore: Ignore vm test images, Fam Zheng, 2017/09/09
- [Qemu-devel] [PULL v2 09/37] qemu.py: Add "wait()" method, Fam Zheng, 2017/09/09
- [Qemu-devel] [PULL v2 10/37] scripts: Add archive-source.sh, Fam Zheng, 2017/09/09
- Re: [Qemu-devel] [PULL v2 10/37] scripts: Add archive-source.sh, Peter Maydell, 2017/09/09
[Qemu-devel] [PULL v2 11/37] tests: Add vm test lib, Fam Zheng, 2017/09/09
[Qemu-devel] [PULL v2 12/37] tests: Add ubuntu.i386 image, Fam Zheng, 2017/09/09
[Qemu-devel] [PULL v2 13/37] tests: Add FreeBSD image, Fam Zheng, 2017/09/09
[Qemu-devel] [PULL v2 14/37] tests: Add NetBSD image, Fam Zheng, 2017/09/09
[Qemu-devel] [PULL v2 16/37] Makefile: Add rules to run vm tests, Fam Zheng, 2017/09/09
[Qemu-devel] [PULL v2 15/37] tests: Add OpenBSD image, Fam Zheng, 2017/09/09
[Qemu-devel] [PULL v2 17/37] MAINTAINERS: Add tests/vm entry, Fam Zheng, 2017/09/09
[Qemu-devel] [PULL v2 19/37] docker: Use archive-source.py, Fam Zheng, 2017/09/09
[Qemu-devel] [PULL v2 18/37] tests: Add README for vm tests, Fam Zheng, 2017/09/09