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

Reply via email to