Re: [PATCH] selftest: hid: add missing run-hid-tools-tests.sh

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 9/30/24 20:43, luyun wrote:

在 2024/10/1 04:57, Shuah Khan 写道:
On 9/30/24 02:38, Benjamin Tissoires wrote:
On Sep 29 2024, Yun Lu wrote:
The HID test cases actually run tests using the run-hid-tools-tests.sh
script. However, if installed with "make install", the run-hid-tools-tests.sh
script will not be copied over, resulting in the following error message.

   make -C tools/testing/selftests/ TARGETS=hid install \
         INSTALL_PATH=$KSFT_INSTALL_PATH

   cd $KSFT_INSTALL_PATH
   ./run_kselftest.sh -c hid

selftests: hid: hid-core.sh
bash: ./run-hid-tools-tests.sh: No such file or directory

So add the run-hid-tools-tests.sh script to the TEST_FILES in the Makefile.


I assume we probably also want:

Cc: stable@xxxxxxxxxxxxxxx

Signed-off-by: Yun Lu <luyun@xxxxxxxxxx>

Not sure about the timing regarding our next PR to Linus, so in any cases:

Acked-by: Benjamin Tissoires <bentiss@xxxxxxxxxx>

Thank you. This commit appears to be right one for Fixes tag?

Is this the right commit for Fixes tag:

Fixes: commit ffb85d5c9e80 ("selftests: hid: import hid-tools hid-core tests")

Yes,  the run-hid-tools-tests.sh script has been introduced since commit ffb85d5c9e80,

but not added to TEST_FILES.


Will apply with this tag added to linux-kselftest fixes for next rc
once I get conformation.

So should I need to send a v2 patch to add the Fixes and Cc tags?



No need to send v2 - I added the Fixes tag and applied to linux-kselftest fixes
branch for next rc.

thanks,
-- Shuah





[Index of Archives]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux