[HuidaeCho]

This PR fixes findstr syntax. findstr uses spaces for or and sometimes requires /r for regex (even though it's the default?).

rem doesn't work
echo "a-b" | findstr "x a.b"

rem works
echo "a-b" | findstr /r "x a.b"

rem works?
echo "a-b" | findstr "a.b x"

Without this PR, generated HTML files are broken, causing:

  C:/Users/hcho/src/grass/grass/build/output/lib/grass86/docs/html/d.barscale.ht
ml:214:0: Error (IndexError('pop from empty list')): </div>

[echoix]

Seems like a weird previous behavior. Trusting your testing, as you're probable the one that will fix it if doesn't work.

Would there be an advantage to use CMake's file(READ), string(REPLACE), and file(WRITE)? Would it end up simpler or with better documentation, even if done in three steps? Probably wouldn't be faster on Linux, but might be easier to debug if it's the same on all platforms. But it differs from the autotools build

[HuidaeCho]

This might be possible, but file() doesn't support piping as far as I know. For now, I'll merge it and focus on other Windows stuff. We can revisit this cross-platform option later.

file(READ "${IN}" txt)

string(REPLACE "</body>" "" txt "${txt}")
string(REPLACE "</html>" "" txt "${txt}")
string(REPLACE "</div> <!-- end container -->" "" txt "${txt}")

file(WRITE "${OUT}" "${txt}")

[nilason]

@HuidaeCho Shall we backport this to 8.5?