diff mbox series

sstate-cache-management: fix regex for 'sigdata' stamp files

Message ID 20240216104551.2122943-1-andre.draszik@linaro.org
State Accepted, archived
Commit 64281de7e352aaf48aa8c60b68633c54471fe353
Headers show
Series sstate-cache-management: fix regex for 'sigdata' stamp files | expand

Commit Message

André Draszik Feb. 16, 2024, 10:45 a.m. UTC
Given file names like
    6.4.do_fetch.sigdata.821b6c62f9f2bd8b7e1378656b8319697a21f6f6e4a351f98dc325a18ef7ed0f,
I'm pretty sure we want to match the dot here, not any character.

Fixes: 2fa1b25d7485 ("sstate-cache-management: Rewrite in python") in oe-core
Fixes: b723fcaac52f ("sstate-cache-management: Rewrite in python") in poky
Signed-off-by: André Draszik <andre.draszik@linaro.org>
---
 scripts/sstate-cache-management.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Alex Kiernan Feb. 16, 2024, 1:56 p.m. UTC | #1
On Fri, Feb 16, 2024 at 10:46 AM André Draszik <andre.draszik@linaro.org> wrote:
>
> Given file names like
>     6.4.do_fetch.sigdata.821b6c62f9f2bd8b7e1378656b8319697a21f6f6e4a351f98dc325a18ef7ed0f,
> I'm pretty sure we want to match the dot here, not any character.
>
> Fixes: 2fa1b25d7485 ("sstate-cache-management: Rewrite in python") in oe-core
> Fixes: b723fcaac52f ("sstate-cache-management: Rewrite in python") in poky
> Signed-off-by: André Draszik <andre.draszik@linaro.org>
> ---

Ouch... good catch!

>  scripts/sstate-cache-management.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/scripts/sstate-cache-management.py b/scripts/sstate-cache-management.py
> index 09b7aa2aefd8..d3f600bd28f6 100755
> --- a/scripts/sstate-cache-management.py
> +++ b/scripts/sstate-cache-management.py
> @@ -147,7 +147,7 @@ def remove_by_stamps(args, paths):
>      for stamps_dir in args.stamps_dir:
>          stamps_path = Path(stamps_dir)
>          assert stamps_path.is_dir()
> -        re_sigdata = re.compile(r"do_.*.sigdata\.([^.]*)")
> +        re_sigdata = re.compile(r"do_.*\.sigdata\.([^.]*)")
>          all_sums |= set(
>              [
>                  re_sigdata.search(x.parts[-1]).group(1)
> --
> 2.44.0.rc0.258.g7320e95886-goog
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#195744): https://lists.openembedded.org/g/openembedded-core/message/195744
> Mute This Topic: https://lists.openembedded.org/mt/104390352/3618097
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [alex.kiernan@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
diff mbox series

Patch

diff --git a/scripts/sstate-cache-management.py b/scripts/sstate-cache-management.py
index 09b7aa2aefd8..d3f600bd28f6 100755
--- a/scripts/sstate-cache-management.py
+++ b/scripts/sstate-cache-management.py
@@ -147,7 +147,7 @@  def remove_by_stamps(args, paths):
     for stamps_dir in args.stamps_dir:
         stamps_path = Path(stamps_dir)
         assert stamps_path.is_dir()
-        re_sigdata = re.compile(r"do_.*.sigdata\.([^.]*)")
+        re_sigdata = re.compile(r"do_.*\.sigdata\.([^.]*)")
         all_sums |= set(
             [
                 re_sigdata.search(x.parts[-1]).group(1)