*** tpb has joined #tomu | 00:00 | |
*** emeb has left #tomu | 00:55 | |
mithro | xobs: I can't tell if GitHub or Windows has mangled your message in https://github.com/enjoy-digital/litex/issues/287 | 02:04 |
---|---|---|
tpb | Title: Software only compiles on Windows when V=1 · Issue #287 · enjoy-digital/litex · GitHub (at github.com) | 02:04 |
xobs | mithro: It has not. That's what the message is. | 02:05 |
xobs | Oh, it's partially mangled. Let me fix it. | 02:05 |
mithro | I think your text is wrong, you are setting `V=1` right? | 02:06 |
xobs | I shouldn't context switch so much :( | 02:07 |
xobs | I changed it so instead of `ifeq($(V),1)` it now reads `ifneq($(V),0)` | 02:08 |
xobs | No, upon further inspection, the original comment wasn't manged. But the text was wrong, I'm setting `V=1`. | 02:11 |
mithro | xobs: I'm suspecting something changing in your environment and not LiteX.... | 02:28 |
xobs | mithro: You're right. It's Conda. | 02:29 |
mithro | xobs: Oh? | 02:32 |
xobs | I'm still trying to narrow down the exact cause, but if I build it with Conda disabled, it works just fine. | 02:32 |
xobs | Removing `m2-bash` fixes the build under Conda. | 02:35 |
xobs | Conversely, adding `m2-bash` to a conda environment breaks the build. | 02:35 |
mithro | Where did m2-bash come from? | 02:36 |
xobs | https://anaconda.org/msys2/m2-bash | 02:36 |
xobs | It was installed in an environment and is required to compile packages such as yosys. Ordinarily it's not required. | 02:37 |
mithro | xobs: Generally you want to avoid anything not in the default conda or if you must conda-forge channels... | 02:37 |
xobs | msys2 is one of the default conda channels, though | 02:38 |
mithro | Really, I didn't know that... | 02:39 |
xobs | I think only official channels are on anaconda.com, and the feed is at https://repo.anaconda.com/pkgs/msys2/ | 02:39 |
tpb | Title: msys2 (at repo.anaconda.com) | 02:39 |
xobs | In any case, I can instlal things like `m2-bash` without adding additional chanels to the config, or specifying additional channels with the `-c` option. | 02:40 |
mithro | Indexes of MSYS2 (Windows) packages (pkgs/msys2 channel) | 02:41 |
mithro | Windows only. Included in conda's defaults channel. Necessary for Anaconda, Inc.'s R conda packages and some others in pkgs/main and pkgs/free. It provides a bash shell, Autotools, revision control systems and the like for building native Windows applications using MinGW-w64 toolchains. | 02:41 |
mithro | https://repo.anaconda.com/pkgs/ | 02:41 |
tpb | Title: Anaconda repositories (at repo.anaconda.com) | 02:41 |
xobs | My guess is what happens when bash is installed is that the litex `Makefile` switches to using bash to unescape strings, whereas before it just passes them through unescaped. | 02:43 |
xobs | Coming from Unix, it's a bit spooky on Windows to write strings such as `$vcpkg_path = "D:\code\vcpkg"` | 02:44 |
mithro | xobs: Maybe installing m2-bash makes Make use bash as your shell rather than a previous sh implementation? | 02:47 |
mithro | https://www.gnu.org/software/make/manual/html_node/Choosing-the-Shell.html | 02:48 |
tpb | Title: GNU make: Choosing the Shell (at www.gnu.org) | 02:48 |
mithro | However, on MS-DOS and MS-Windows the value of SHELL in the environment is used, since on those systems most users do not set this variable, and therefore it is most likely set specifically to be used by make. On MS-DOS, if the setting of SHELL is not suitable for make, you can set the variable MAKESHELL to the shell that make should use; if set it will be used as the shell instead of the value of SHELL. | 02:49 |
mithro | There is a whole section "Choosing a Shell in DOS and Windows" | 02:49 |
xobs | You're really good at this. | 02:50 |
TomKeddie | It's kinda sad that people have to be so good at this but yes, it's appreciated. | 02:54 |
mithro | xobs: I have a lot of experience from the X many years of litex-buildenv development | 02:55 |
*** ppisati has quit IRC | 08:25 | |
*** ppisati has joined #tomu | 08:31 | |
*** moon_child is now known as elronnd | 09:29 | |
*** elronnd is now known as moon_child | 09:29 | |
*** kuldeep has quit IRC | 13:57 | |
*** krogozinski has quit IRC | 14:11 | |
*** CarlFK has joined #tomu | 15:08 | |
*** emeb has joined #tomu | 15:26 | |
*** TheJJ has quit IRC | 19:21 | |
*** TheJJ has joined #tomu | 19:22 | |
*** kuldeep has joined #tomu | 19:23 | |
*** buZz has quit IRC | 20:07 | |
*** emeb has quit IRC | 22:53 |
Generated by irclog2html.py 2.13.1 by Marius Gedminas - find it at mg.pov.lt!