* [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).