-
Notifications
You must be signed in to change notification settings - Fork 242
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Disable readline support? #77
Comments
There is no direct dependency to readline in nextpnr. Can you give a hint to what do you refer ? |
Hrm, conda was detecting that nextpnr was being linked against libreadline but I may be mistaken. |
This might be coming through the dependency on libpython? In that case, I don't think you could disable it without rebuilding libpython. |
No readline anywhere in the dependency tree as far as I can see. Did you ever investigate this further? |
Confirmed no readline library linked anywhere in nextpnr |
Replaced `ICEBOX_ROOT` variable with `ICESTORM_INSTALL_PREFIX` because the first one is now deprecated. Removed `ENABLE_READLINE` which had no influence. The `nextpnr` doesn't use `readline` at all: YosysHQ/nextpnr#77 (comment) Added `CMAKE_ARGS` to be passed to CMake. These arguments are set by the `gcc_linux-64` activation script with defaults recommended when building Conda packages.
Replaced `ICEBOX_ROOT` variable with `ICESTORM_INSTALL_PREFIX` because the first one is now deprecated. Removed `ENABLE_READLINE` which had no influence. The `nextpnr` doesn't use `readline` at all: YosysHQ/nextpnr#77 (comment) Added `CMAKE_ARGS` to be passed to CMake. These arguments are set by the `gcc_linux-64` activation script with defaults recommended when building Conda packages.
Is there a way to disable building with readline support? Is
-DENABLE_READLINE=No
enough?The text was updated successfully, but these errors were encountered: