DPDK patches and discussions
 help / color / mirror / Atom feed
* [PATCH] dts: fix DPDK git tarball cast bug
@ 2024-07-19 15:34 Alex Chapman
  2024-07-22 13:06 ` Juraj Linkeš
  0 siblings, 1 reply; 3+ messages in thread
From: Alex Chapman @ 2024-07-19 15:34 UTC (permalink / raw)
  To: dev
  Cc: Juraj Linkeš,
	Honnappa Nagarahalli, Alex Chapman, Jack Bond-Preston,
	Luca Vizzarro, Paul Szczepanek

When running DTS with a git revision specified the following error occurs:
$ ./main.py --revision HEAD
[...]
FileNotFoundError: [Errno 2] No such file or directory:
     'dts/<framework.utils.DPDKGitTarball object at 0x78f69e73a5c0>'

Fix by casting DPDKGitTarball to Path, in order to stringify it correctly.

Bugzilla ID: 1496
Fixes: a23f22457dbd ("dts: constrain DPDK source argument")

Signed-off-by: Alex Chapman <alex.chapman@arm.com>
Reviewed-by: Jack Bond-Preston <jack.bond-preston@arm.com>
Reviewed-by: Luca Vizzarro <luca.vizzarro@arm.com>
Reviewed-by: Paul Szczepanek <paul.szczepanek@arm.com>
---
Hello,
Sending this fix to a bug that i discovered.

Best Regards,
Alex
---
 .mailmap                  | 1 +
 dts/framework/settings.py | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/.mailmap b/.mailmap
index 3f3f0442e5..a326e8e553 100644
--- a/.mailmap
+++ b/.mailmap
@@ -46,6 +46,7 @@ Aleksey Baulin <aleksey.baulin@gmail.com>
 Aleksey Katargin <gureedo@gmail.com>
 Ales Musil <amusil@redhat.com>
 Alessio Igor Bogani <alessio.bogani@elettra.eu>
+Alex Chapman <alex.chapman@arm.com>
 Alexander Bechikov <asb.tyum@gmail.com>
 Alexander Belyakov <abelyako@gmail.com>
 Alexander Chernavin <achernavin@netgate.com>
diff --git a/dts/framework/settings.py b/dts/framework/settings.py
index f95876113f..f6303066d4 100644
--- a/dts/framework/settings.py
+++ b/dts/framework/settings.py
@@ -415,7 +415,7 @@ def get_settings() -> Settings:
     args = parser.parse_args()
 
     if args.dpdk_revision_id:
-        args.dpdk_tarball_path = DPDKGitTarball(args.dpdk_revision_id, args.output_dir)
+        args.dpdk_tarball_path = Path(DPDKGitTarball(args.dpdk_revision_id, args.output_dir))
 
     args.test_suites = _process_test_suites(parser, args.test_suites)
 
-- 
2.34.1


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] dts: fix DPDK git tarball cast bug
  2024-07-19 15:34 [PATCH] dts: fix DPDK git tarball cast bug Alex Chapman
@ 2024-07-22 13:06 ` Juraj Linkeš
  2024-07-23 21:43   ` Thomas Monjalon
  0 siblings, 1 reply; 3+ messages in thread
From: Juraj Linkeš @ 2024-07-22 13:06 UTC (permalink / raw)
  To: Alex Chapman, dev
  Cc: Honnappa Nagarahalli, Jack Bond-Preston, Luca Vizzarro, Paul Szczepanek

Reviewed-by: Juraj Linkeš <juraj.linkes@pantheon.tech>

Thanks for catching this.

On 19. 7. 2024 17:34, Alex Chapman wrote:
> When running DTS with a git revision specified the following error occurs:
> $ ./main.py --revision HEAD
> [...]
> FileNotFoundError: [Errno 2] No such file or directory:
>       'dts/<framework.utils.DPDKGitTarball object at 0x78f69e73a5c0>'
> 
> Fix by casting DPDKGitTarball to Path, in order to stringify it correctly.
> 
> Bugzilla ID: 1496
> Fixes: a23f22457dbd ("dts: constrain DPDK source argument")
> 
> Signed-off-by: Alex Chapman <alex.chapman@arm.com>
> Reviewed-by: Jack Bond-Preston <jack.bond-preston@arm.com>
> Reviewed-by: Luca Vizzarro <luca.vizzarro@arm.com>
> Reviewed-by: Paul Szczepanek <paul.szczepanek@arm.com>
> ---
> Hello,
> Sending this fix to a bug that i discovered.
> 
> Best Regards,
> Alex
> ---
>   .mailmap                  | 1 +
>   dts/framework/settings.py | 2 +-
>   2 files changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/.mailmap b/.mailmap
> index 3f3f0442e5..a326e8e553 100644
> --- a/.mailmap
> +++ b/.mailmap
> @@ -46,6 +46,7 @@ Aleksey Baulin <aleksey.baulin@gmail.com>
>   Aleksey Katargin <gureedo@gmail.com>
>   Ales Musil <amusil@redhat.com>
>   Alessio Igor Bogani <alessio.bogani@elettra.eu>
> +Alex Chapman <alex.chapman@arm.com>
>   Alexander Bechikov <asb.tyum@gmail.com>
>   Alexander Belyakov <abelyako@gmail.com>
>   Alexander Chernavin <achernavin@netgate.com>
> diff --git a/dts/framework/settings.py b/dts/framework/settings.py
> index f95876113f..f6303066d4 100644
> --- a/dts/framework/settings.py
> +++ b/dts/framework/settings.py
> @@ -415,7 +415,7 @@ def get_settings() -> Settings:
>       args = parser.parse_args()
>   
>       if args.dpdk_revision_id:
> -        args.dpdk_tarball_path = DPDKGitTarball(args.dpdk_revision_id, args.output_dir)
> +        args.dpdk_tarball_path = Path(DPDKGitTarball(args.dpdk_revision_id, args.output_dir))
>   
>       args.test_suites = _process_test_suites(parser, args.test_suites)
>   

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] dts: fix DPDK git tarball cast bug
  2024-07-22 13:06 ` Juraj Linkeš
@ 2024-07-23 21:43   ` Thomas Monjalon
  0 siblings, 0 replies; 3+ messages in thread
From: Thomas Monjalon @ 2024-07-23 21:43 UTC (permalink / raw)
  To: Alex Chapman
  Cc: dev, Honnappa Nagarahalli, Jack Bond-Preston, Luca Vizzarro,
	Paul Szczepanek, Juraj Linkeš

22/07/2024 15:06, Juraj Linkeš:
> 
> On 19. 7. 2024 17:34, Alex Chapman wrote:
> > When running DTS with a git revision specified the following error occurs:
> > $ ./main.py --revision HEAD
> > [...]
> > FileNotFoundError: [Errno 2] No such file or directory:
> >       'dts/<framework.utils.DPDKGitTarball object at 0x78f69e73a5c0>'
> > 
> > Fix by casting DPDKGitTarball to Path, in order to stringify it correctly.
> > 
> > Bugzilla ID: 1496
> > Fixes: a23f22457dbd ("dts: constrain DPDK source argument")
> > 
> > Signed-off-by: Alex Chapman <alex.chapman@arm.com>
> > Reviewed-by: Jack Bond-Preston <jack.bond-preston@arm.com>
> > Reviewed-by: Luca Vizzarro <luca.vizzarro@arm.com>
> > Reviewed-by: Paul Szczepanek <paul.szczepanek@arm.com>
> > ---
> > Hello,
> > Sending this fix to a bug that i discovered.
> > 
> > Best Regards,
> > Alex
> > ---
> Reviewed-by: Juraj Linkeš <juraj.linkes@pantheon.tech>
> 
> Thanks for catching this.

Applied, thanks and welcome.




^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2024-07-23 21:43 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-07-19 15:34 [PATCH] dts: fix DPDK git tarball cast bug Alex Chapman
2024-07-22 13:06 ` Juraj Linkeš
2024-07-23 21:43   ` Thomas Monjalon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).