This is an archive of the discontinued LLVM Phabricator instance.

Fix manifestinput-error.test on W10
ClosedPublic

Authored by aganea on Apr 3 2018, 3:47 PM.

Reviewers

Commits

Summary

Hi guys,
The following test doesn't pass on my W10 machine. Is there a way to restrict it to W8? Or upgrade?

lld/test/COFF/manifestinput-error.test
10 ↗	(On Diff #140872)	It is better to prepare for both. CHECK: error: unable to find mt.exe in PATH: {{[Nn]}}o such file or directory

Thank you Rui, works now. Would you like to commit?

This revision was not accepted when it landed; it landed in state Needs Review.Apr 3 2018, 4:15 PM

This revision was automatically updated to reflect the committed changes.

Path

Size

lld/

trunk/

test/

COFF/

2 lines

	# UNSUPPORTED: manifest_tool			# UNSUPPORTED: manifest_tool
	# UNSUPPORTED: libxml2			# UNSUPPORTED: libxml2

	# RUN: yaml2obj %p/Inputs/ret42.yaml > %t.obj			# RUN: yaml2obj %p/Inputs/ret42.yaml > %t.obj
	# RUN: not lld-link /out:%t.exe /entry:main \			# RUN: not lld-link /out:%t.exe /entry:main \
	# RUN: /manifest:embed \			# RUN: /manifest:embed \
	# RUN: /manifestuac:"level='requireAdministrator'" \			# RUN: /manifestuac:"level='requireAdministrator'" \
	# RUN: /manifestinput:%p/Inputs/manifestinput.test %t.obj 2>&1 \| FileCheck %s			# RUN: /manifestinput:%p/Inputs/manifestinput.test %t.obj 2>&1 \| FileCheck %s

	# CHECK: error: unable to find mt.exe in PATH: No such file or directory			# CHECK: error: unable to find mt.exe in PATH: {{[Nn]}}o such file or directory