Page MenuHomeFreeBSD

csu tests: Remove extra slash
ClosedPublic

Authored by jlduran on Feb 1 2025, 4:21 AM.
Tags
None
Referenced Files
F138011467: D48778.id150321.diff
Fri, Nov 28, 3:24 AM
F137995849: D48778.id150465.diff
Fri, Nov 28, 12:04 AM
Unknown Object (File)
Wed, Nov 26, 7:16 PM
Unknown Object (File)
Wed, Nov 26, 4:25 PM
Unknown Object (File)
Tue, Nov 25, 10:08 AM
Unknown Object (File)
Sun, Nov 23, 12:20 AM
Unknown Object (File)
Sat, Nov 22, 4:37 PM
Unknown Object (File)
Fri, Nov 21, 2:17 PM
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.