On Windows, it is not likely that users of ISE toolchain will have the appropriate environment variables set by default. Since the API change in MiSoC, users no longer have the ability to override this behavior when building a target. The resulting batch file generated by Migen will not be able to find any of the Xilinx tools.
Therefore, we should by default insert a call the Xilinx batch file to set up the environment before running the tools. The call is required to prevent the child batch file from terminating the parent batch file when the child is done. For those who already have Xilinx tools on the path before building Migen gateware, there is no difference in environment other than the Xilinx PATHS being duplicated for the duration of the cmd.exe subprocess. William D. Jones (1): xilinx/ise: Windows should source Xilinx toolchain by default (fixes MiSoC build failures). migen/build/xilinx/ise.py | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) -- 2.6.3 _______________________________________________ M-Labs devel mailing list https://ssl.serverraum.org/lists/listinfo/devel