Skip to content
This repository was archived by the owner on Mar 28, 2023. It is now read-only.

[SYCL] Hotfix for XPTI tests #655

Merged
merged 1 commit into from
Dec 22, 2021
Merged

Conversation

alexbatashev
Copy link

@alexbatashev alexbatashev commented Dec 22, 2021

Pre-commit is failing with the following error:

lit.py: /__w/llvm/llvm/lit/lit/TestingConfig.py:103: fatal: unable to parse config file '/__w/llvm/llvm/llvm_test_suite/SYCL/lit.cfg.py', traceback: Traceback (most recent call last):
  File "/__w/llvm/llvm/lit/lit/TestingConfig.py", line 92, in load_from_path
    exec(compile(data, path, 'exec'), cfg_globals, None)
  File "/__w/llvm/llvm/llvm_test_suite/SYCL/lit.cfg.py", line 341, in <module>
    if os.path.exists(xptifw_lib) and os.path.exists(os.path.join(xptifw_includes, 'xpti', 'xpti_trace_framework.h')):
NameError: name 'xptifw_lib' is not defined

@bader
Copy link

bader commented Dec 22, 2021

@alexbatashev, how did you manage to pass pre-commit with the breaking change?

@alexbatashev
Copy link
Author

@alexbatashev, how did you manage to pass pre-commit with the breaking change?

I have no idea.

@bader bader merged commit 23a231f into intel:intel Dec 22, 2021
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
Pre-commit is failing with the following error: 
```
lit.py: /__w/llvm/llvm/lit/lit/TestingConfig.py:103: fatal: unable to parse config file '/__w/llvm/llvm/llvm_test_suite/SYCL/lit.cfg.py', traceback: Traceback (most recent call last):
  File "/__w/llvm/llvm/lit/lit/TestingConfig.py", line 92, in load_from_path
    exec(compile(data, path, 'exec'), cfg_globals, None)
  File "/__w/llvm/llvm/llvm_test_suite/SYCL/lit.cfg.py", line 341, in <module>
    if os.path.exists(xptifw_lib) and os.path.exists(os.path.join(xptifw_includes, 'xpti', 'xpti_trace_framework.h')):
NameError: name 'xptifw_lib' is not defined
```
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants