From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:36126) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gK34V-00058q-Uo for qemu-devel@nongnu.org; Tue, 06 Nov 2018 10:15:16 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1gK34R-0006Rz-Ls for qemu-devel@nongnu.org; Tue, 06 Nov 2018 10:15:15 -0500 Received: from mail-wm1-f65.google.com ([209.85.128.65]:38160) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1gK34R-0006QN-FF for qemu-devel@nongnu.org; Tue, 06 Nov 2018 10:15:11 -0500 Received: by mail-wm1-f65.google.com with SMTP id l2-v6so12366145wmh.3 for ; Tue, 06 Nov 2018 07:15:11 -0800 (PST) References: <20181105181353.39804-1-mst@redhat.com> <20181105181353.39804-3-mst@redhat.com> <20181106091147-mutt-send-email-mst@kernel.org> From: =?UTF-8?Q?Philippe_Mathieu-Daud=c3=a9?= Message-ID: <4b68930c-1fc3-9462-8f9e-671b414e712d@redhat.com> Date: Tue, 6 Nov 2018 16:15:03 +0100 MIME-Version: 1.0 In-Reply-To: <20181106091147-mutt-send-email-mst@kernel.org> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit Subject: Re: [Qemu-devel] [PULL 02/33] tests: Move tests/hex-loader-check-data/ to tests/data/hex-loader/ List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Michael S. Tsirkin" Cc: qemu-devel@nongnu.org, Laurent Vivier , Peter Maydell , Thomas Huth , Su Hang , Paolo Bonzini On 6/11/18 15:13, Michael S. Tsirkin wrote: > On Tue, Nov 06, 2018 at 02:27:18PM +0100, Philippe Mathieu-Daudé wrote: >> On 5/11/18 19:14, Michael S. Tsirkin wrote: >>> From: Peter Maydell >>> >>> Currently tests/hex-loader-check-data contains data files used >>> by the hexloader-test, and configure individually symlinks those >>> data files into the build directory using a wildcard. >>> >>> Using a wildcard like this is a bad idea, because if a new >>> data file is added, nothing causes configure to be rerun, >>> and so no symlink is added for the new file. This can cause >>> tests to spuriously fail when they can't find their data. >>> Instead, it's better to symlink an entire directory of >>> data files. We already have such a directory: tests/data. >>> >>> Move the data files from tests/hex-loader-check-data/ to >>> tests/data/hex-loader/, and remove the unnecessary symlinking. >>> >>> Signed-off-by: Peter Maydell >> >> I reviewed/tested this patch too. > > > Thanks a lot Philippe! > It is unfortunately too late to update this patch info in git > commit history, however your help is still greatly appreciated! No worry, I'm not mad at all, but there might be an issue in your git PR workflow, this series also missed your maintainer S-o-b. Peter: Can you add a such check in your scripts? (during next merge window, no hurry). Rather than your scripts, this should be in scripts a maintainer can run locally, such ./scripts/checkpatch.pl --maintainer or ./scripts/checkseries.xx. > > >>> --- >>> configure | 4 ---- >>> tests/hexloader-test.c | 2 +- >>> MAINTAINERS | 2 +- >>> tests/{hex-loader-check-data => data/hex-loader}/test.hex | 0 >>> 4 files changed, 2 insertions(+), 6 deletions(-) >>> rename tests/{hex-loader-check-data => data/hex-loader}/test.hex (100%) >>> >>> diff --git a/configure b/configure >>> index 895b7483b8..bfdca8b814 100755 >>> --- a/configure >>> +++ b/configure >>> @@ -7421,10 +7421,6 @@ for bios_file in \ >>> do >>> FILES="$FILES pc-bios/$(basename $bios_file)" >>> done >>> -for test_file in $(find $source_path/tests/hex-loader-check-data -type f) >>> -do >>> - FILES="$FILES tests/hex-loader-check-data$(echo $test_file | sed -e 's/.*hex-loader-check-data//')" >>> -done >>> mkdir -p $DIRS >>> for f in $FILES ; do >>> if [ -e "$source_path/$f" ] && [ "$pwd_is_source_path" != "y" ]; then >>> diff --git a/tests/hexloader-test.c b/tests/hexloader-test.c >>> index b653d44ba1..834ed52c22 100644 >>> --- a/tests/hexloader-test.c >>> +++ b/tests/hexloader-test.c >>> @@ -23,7 +23,7 @@ static void hex_loader_test(void) >>> const unsigned int base_addr = 0x00010000; >>> QTestState *s = qtest_initf( >>> - "-M vexpress-a9 -nographic -device loader,file=tests/hex-loader-check-data/test.hex"); >>> + "-M vexpress-a9 -nographic -device loader,file=tests/data/hex-loader/test.hex"); >>> for (i = 0; i < 256; ++i) { >>> uint8_t val = qtest_readb(s, base_addr + i); >>> diff --git a/MAINTAINERS b/MAINTAINERS >>> index 98a1856afc..cfabc14b59 100644 >>> --- a/MAINTAINERS >>> +++ b/MAINTAINERS >>> @@ -1370,7 +1370,7 @@ Intel Hexadecimal Object File Loader >>> M: Su Hang >>> S: Maintained >>> F: tests/hexloader-test.c >>> -F: tests/hex-loader-check-data/test.hex >>> +F: tests/data/hex-loader/test.hex >>> CHRP NVRAM >>> M: Thomas Huth >>> diff --git a/tests/hex-loader-check-data/test.hex b/tests/data/hex-loader/test.hex >>> similarity index 100% >>> rename from tests/hex-loader-check-data/test.hex >>> rename to tests/data/hex-loader/test.hex >>>