[PATCH] test/crypto: fix non ASCII character

[PATCH] test/crypto: fix non ASCII character

[Anoob Joseph]

Fix non ASCII character in the comment. Revert to original text.

Bugzilla ID: 1396
Fixes: f97c63f4f445 ("test/crypto: add AES-GCM external mbuf case")

Signed-off-by: Anoob Joseph <anoobj@marvell.com>
---
 app/test/test_cryptodev.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/test/test_cryptodev.c b/app/test/test_cryptodev.c
index c3c3f587b4..754fab39c5 100644
--- a/app/test/test_cryptodev.c
+++ b/app/test/test_cryptodev.c
@@ -1615,7 +1615,7 @@ ext_mbuf_create(struct rte_mempool *mbuf_pool, int pkt_len,
 			goto fail;
 		}
 
-		/* Save shared data (like callback function) in external buffer’s end */
+		/* Save shared data (like callback function) in external buffer's end */
 		ret_shinfo = rte_pktmbuf_ext_shinfo_init_helper(ext_buf_addr, &buf_len,
 			ext_mbuf_callback_fn_free, &freed);
 		if (ret_shinfo == NULL) {
-- 
2.25.1

RE: [PATCH] test/crypto: fix non ASCII character

[Ali Alnubani]

> -----Original Message-----
> From: Anoob Joseph <anoobj@marvell.com>
> Sent: Tuesday, March 5, 2024 1:34 PM
> To: Akhil Goyal <gakhil@marvell.com>; David Marchand
> <david.marchand@redhat.com>
> Cc: Aakash Sasidharan <asasidharan@marvell.com>; dev@dpdk.org
> Subject: [PATCH] test/crypto: fix non ASCII character
> 
> Fix non ASCII character in the comment. Revert to original text.
> 
> Bugzilla ID: 1396
> Fixes: f97c63f4f445 ("test/crypto: add AES-GCM external mbuf case")
> 
> Signed-off-by: Anoob Joseph <anoobj@marvell.com>
> ---

Build passes with this patch, thanks!

Tested-by: Ali Alnubani <alialnu@nvidia.com>

Re: [PATCH] test/crypto: fix non ASCII character

[Thomas Monjalon]

05/03/2024 13:14, Ali Alnubani:
> > Subject: [PATCH] test/crypto: fix non ASCII character
> > 
> > Fix non ASCII character in the comment. Revert to original text.
> > 
> > Bugzilla ID: 1396
> > Fixes: f97c63f4f445 ("test/crypto: add AES-GCM external mbuf case")
> > 
> > Signed-off-by: Anoob Joseph <anoobj@marvell.com>
> > ---
> 
> Build passes with this patch, thanks!
> 
> Tested-by: Ali Alnubani <alialnu@nvidia.com>

Reported-by: Ali Alnubani <alialnu@nvidia.com>

Applied, thanks.


The script buildtools/get-test-suites.py is not tolerant with Unicode.
We may be more flexible probably.
Bruce, Robin, what do you think about allowing Unicode characters in this script?

Re: [PATCH] test/crypto: fix non ASCII character

[David Marchand]

On Tue, Mar 5, 2024 at 2:25 PM Thomas Monjalon <thomas@monjalon.net> wrote:
>
> 05/03/2024 13:14, Ali Alnubani:
> > > Subject: [PATCH] test/crypto: fix non ASCII character
> > >
> > > Fix non ASCII character in the comment. Revert to original text.
> > >
> > > Bugzilla ID: 1396
> > > Fixes: f97c63f4f445 ("test/crypto: add AES-GCM external mbuf case")
> > >
> > > Signed-off-by: Anoob Joseph <anoobj@marvell.com>
> > > ---
> >
> > Build passes with this patch, thanks!
> >
> > Tested-by: Ali Alnubani <alialnu@nvidia.com>
>
> Reported-by: Ali Alnubani <alialnu@nvidia.com>
>
> Applied, thanks.
>
>
> The script buildtools/get-test-suites.py is not tolerant with Unicode.
> We may be more flexible probably.
> Bruce, Robin, what do you think about allowing Unicode characters in this script?

Note: I could reproduce this issue with a Ubuntu 18.04 container, but
not with my fedora 38.
Maybe something to do with a python version.. ?


-- 
David Marchand

Re: [PATCH] test/crypto: fix non ASCII character

[Robin Jarry]

David Marchand, Mar 05, 2024 at 14:27:
> > The script buildtools/get-test-suites.py is not tolerant with Unicode.
> > We may be more flexible probably.
> > Bruce, Robin, what do you think about allowing Unicode characters in this script?
>
> Note: I could reproduce this issue with a Ubuntu 18.04 container, but
> not with my fedora 38.
> Maybe something to do with a python version.. ?

The files are opened without specifying an encoding. So it depends on 
the LC_ALL environment variable.

https://git.dpdk.org/dpdk/tree/buildtools/get-test-suites.py#n22

I will send a patch to fix this.