* [ndctl PATCH] test/dm.sh: fix incorrect script name in cleanup error message
@ 2025-09-11 0:29 Yi Zhang
2025-09-11 15:44 ` Dave Jiang
2025-09-11 16:50 ` Alison Schofield
0 siblings, 2 replies; 3+ messages in thread
From: Yi Zhang @ 2025-09-11 0:29 UTC (permalink / raw)
To: nvdimm; +Cc: alison.schofield, linux-cxl, dave.jiang
The cleanup() function was incorrectly referencing "test/sub-section.sh"
instead of "test/dm.sh" in its error message.
Signed-off-by: Yi Zhang <yi.zhang@redhat.com>
---
test/dm.sh | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/test/dm.sh b/test/dm.sh
index b780a65..e68d491 100755
--- a/test/dm.sh
+++ b/test/dm.sh
@@ -36,7 +36,7 @@ cleanup() {
for i in $namespaces
do
if ! $NDCTL destroy-namespace -f $i; then
- echo "test/sub-section.sh: cleanup() failed to destroy $i"
+ echo "test/dm.sh: cleanup() failed to destroy $i"
fi
done
exit $rc
--
2.45.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [ndctl PATCH] test/dm.sh: fix incorrect script name in cleanup error message
2025-09-11 0:29 [ndctl PATCH] test/dm.sh: fix incorrect script name in cleanup error message Yi Zhang
@ 2025-09-11 15:44 ` Dave Jiang
2025-09-11 16:50 ` Alison Schofield
1 sibling, 0 replies; 3+ messages in thread
From: Dave Jiang @ 2025-09-11 15:44 UTC (permalink / raw)
To: Yi Zhang, nvdimm; +Cc: alison.schofield, linux-cxl
On 9/10/25 5:29 PM, Yi Zhang wrote:
> The cleanup() function was incorrectly referencing "test/sub-section.sh"
> instead of "test/dm.sh" in its error message.
>
> Signed-off-by: Yi Zhang <yi.zhang@redhat.com>
Reviewed-by: Dave Jiang <dave.jiang@intel.com>
> ---
> test/dm.sh | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/test/dm.sh b/test/dm.sh
> index b780a65..e68d491 100755
> --- a/test/dm.sh
> +++ b/test/dm.sh
> @@ -36,7 +36,7 @@ cleanup() {
> for i in $namespaces
> do
> if ! $NDCTL destroy-namespace -f $i; then
> - echo "test/sub-section.sh: cleanup() failed to destroy $i"
> + echo "test/dm.sh: cleanup() failed to destroy $i"
> fi
> done
> exit $rc
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [ndctl PATCH] test/dm.sh: fix incorrect script name in cleanup error message
2025-09-11 0:29 [ndctl PATCH] test/dm.sh: fix incorrect script name in cleanup error message Yi Zhang
2025-09-11 15:44 ` Dave Jiang
@ 2025-09-11 16:50 ` Alison Schofield
1 sibling, 0 replies; 3+ messages in thread
From: Alison Schofield @ 2025-09-11 16:50 UTC (permalink / raw)
To: Yi Zhang; +Cc: nvdimm, linux-cxl, dave.jiang
On Wed, Sep 10, 2025 at 08:29:06PM -0400, Yi Zhang wrote:
> The cleanup() function was incorrectly referencing "test/sub-section.sh"
> instead of "test/dm.sh" in its error message.
Thanks!
Applied https://github.com/pmem/ndctl/commits/pending/
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2025-09-11 16:50 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-09-11 0:29 [ndctl PATCH] test/dm.sh: fix incorrect script name in cleanup error message Yi Zhang
2025-09-11 15:44 ` Dave Jiang
2025-09-11 16:50 ` Alison Schofield
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox