Skip to content

make it compatible with Python3, remove -Werror#39

Open
matthiasdiener wants to merge 2 commits into
GPUOpen-Tools:masterfrom
matthiasdiener:python3
Open

make it compatible with Python3, remove -Werror#39
matthiasdiener wants to merge 2 commits into
GPUOpen-Tools:masterfrom
matthiasdiener:python3

Conversation

@matthiasdiener
Copy link
Copy Markdown

@matthiasdiener matthiasdiener commented Jun 16, 2020

Fixes #34.

@baryluk
Copy link
Copy Markdown

baryluk commented Jun 16, 2020

Why you need to use 2to3 at runtime and not change all problematic places once and commit this to repo?

@matthiasdiener
Copy link
Copy Markdown
Author

matthiasdiener commented Jun 16, 2020

Why you need to use 2to3 at runtime and not change all problematic places once and commit this to repo?

Because CXL_init.py (the only file handled by 2to3) is not actually contained within this repo (it is a symlink to ../Common/Src/Scons), but is located in a read-only, archived repo (https://github.com/GPUOpen-Archive/common-src-SCons), and I therefore can't open Pull Requests that modify this file anymore.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Build failure in scons

2 participants