Carlo Marcelo Arenas Belón <carenas@xxxxxxxxx> writes: > -if type p4d >/dev/null && type p4 >/dev/null > +if type p4d >/dev/null 2>&1 && type p4 >/dev/null 2>&1 > then > echo "$(tput setaf 6)Perforce Server Version$(tput sgr0)" > p4d -V | grep Rev. > echo "$(tput setaf 6)Perforce Client Version$(tput sgr0)" > p4 -V | grep Rev. > +else > + echo "WARNING: perforce wasn't installed, see above for clues why" >2 > fi > -if type git-lfs >/dev/null > +if type git-lfs >/dev/null 2>&1 > then > echo "$(tput setaf 6)Git-LFS Version$(tput sgr0)" > git-lfs version > +else > + echo "WARNING: git-lfs wasn't installed, see above for clues why" >2 > fi NO! Why do we want to create a file whose name is "2" here? Good that I caught them before I merged them to 'next'. Will locally amend and see if it fixes CI failure in 'seen' Thanks.
