]> git.ipfire.org Git - thirdparty/grub.git/commit
Revert "tests: Remove -w param from mkfs.hfsplus command"
authorGlenn Washburn <development@efficientek.com>
Wed, 19 Nov 2025 19:11:35 +0000 (13:11 -0600)
committerDaniel Kiper <daniel.kiper@oracle.com>
Thu, 20 Nov 2025 16:41:05 +0000 (17:41 +0100)
commita8b2beedf684f0200e252efe9ba5fad2665b2a36
treef69d2722a7376defd59229d0c9edcd83d665e5b0
parent14376470520d3106e32f5c0db4d15a86e7eaf33a
Revert "tests: Remove -w param from mkfs.hfsplus command"

The original commit removes testing of GRUB's support for HFS+
wrapping and replaces it with testing that is an exact duplicate of
another test, namely HFS+ without wrapping. To start, the change is
misleading in that it suggests that the testing of HFS+ wrapping is
still taking place, when it is not. If it was desired to remove support
for testing the HFS+ wrapping, then the test should have been removed
entirely. Second, having a series of tests that are exactly the same is
just a waste of testing resources. And third, the justification for the
change is nonsensical. Just because a required program may not have
a required feature on a particular distro is not a reason that a test
should be removed. Reducing test coverage because some distros do not
have the tools GRUB needs to run certain tests goes against the testing
priority to have test coverage be as broad as possible. The fact is
that Debian, the officially supported distro for running the tests, does
have a mkfs.hfsplus that supports the -w parameter.

This reverts commit 2bc0929a2 (tests: Remove -w param from mkfs.hfsplus command).

Signed-off-by: Glenn Washburn <development@efficientek.com>
Reviewed-by: Daniel Kiper <daniel.kiper@oracle.com>
tests/util/grub-fs-tester.in