findstr and && do not play ball together
findstr and && do not play ball together
this is supposed to detect which error was posted from 7z.exe
to "%temp%error.unp"
and act accordingly. however, the &&
after findstr
does not work at all, the specific commands are never executed. what am I doing wrong?
7z.exe
"%temp%error.unp"
&&
findstr
type "%temp%error.unp" || findstr /c:"Wrong password" && set /p "password=enter password other than fanedit.org: " && goto security
type "%temp%error.unp" || findstr /c:"not enough space" && rmdir /s /q %drive%$recycle.bin && echo disk was full, recycle bin emptied out.
1 Answer
1
As mentioned in the comment to Proteks answer you'll need parentheses to enclose a block.
No need to cram all in one line then.
Reacting solely on an errorlevel means you should suppress other output with >Nul 2>&1
>Nul 2>&1
Edit: reversed redirection order due to @sst's hint
Try:
type "%temp%error.unp" | findstr /c:"Wrong password" >Nul 2>&1 &&(
set /p "password=enter password other than fanedit.org: "
goto security
)
type "%temp%error.unp" | findstr /c:"not enough space" >Nul 2>&1 &&(
rmdir /s /q %drive%$recycle.bin
echo disk was full, recycle bin emptied out.
)
I am intrigued by that
2>&1
bit. can you explain that to someone not very familiar with the terminology?– Bricktop
Sep 10 '18 at 15:01
2>&1
It merges error stream 2 with standard output 1. See redirection
– LotPings
Sep 10 '18 at 15:07
Thanks for contributing an answer to Stack Overflow!
But avoid …
To learn more, see our tips on writing great answers.
Required, but never shown
Required, but never shown
By clicking "Post Your Answer", you acknowledge that you have read our updated terms of service, privacy policy and cookie policy, and that your continued use of the website is subject to these policies.
@sst Ack changed above.
– LotPings
Sep 9 '18 at 19:53