Page MenuHomeFreeBSD

csu tests: Remove extra slash
ClosedPublic

Authored by jlduran on Feb 1 2025, 4:21 AM.
Tags
None
Referenced Files
F132628578: D48778.diff
Sat, Oct 18, 1:57 PM
Unknown Object (File)
Mon, Sep 29, 11:11 AM
Unknown Object (File)
Sep 17 2025, 5:29 AM
Unknown Object (File)
Sep 15 2025, 4:43 PM
Unknown Object (File)
Sep 15 2025, 6:19 AM
Unknown Object (File)
Sep 9 2025, 2:16 PM
Unknown Object (File)
Aug 31 2025, 12:36 AM
Unknown Object (File)
Aug 15 2025, 1:00 AM
Subscribers

Details

Summary

Remove an extra slash that ends up in the metalog. This double slash
can produce an invalid specification file if there are subdirectories
down the hierarchy when sorted.

For example, consider the following metalog excerpt:

./base/aaa type=dir
./base//aaa/bbb type=dir

If sorted, would turn out:

./base//aaa/bbb type=dir
./base/aaa type=dir

Apparently missing the ./base/aaa directory in the specification.

Luckily here are no subdirectories.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Feb 1 2025, 4:40 AM

We could add a check for double slash to tools/pkgbase/metalog_reader.lua also

We could add a check for double slash to tools/pkgbase/metalog_reader.lua also

Yes! I forgot you mentioned it in another review. I'll take a look. Thank you.

This revision was automatically updated to reflect the committed changes.