Open mateon1 opened 7 years ago
@mateon1 thank you very much for reporting the issue! I'll work to pull in the latest version of Electron that you referenced. Can you confirm that that 1.4.3 fixes the issue for you?
We haven't done any work on Windows yet, so anything may come across is uncharted territory. Would you be interested in trying to fix those issues? Regardless, much appreciate the bug report.
@haadcode Yes, I tested the install updating Electron to 1.4.3, and everything seemed to install correctly (after npm link
-ing packages depending on go-ipfs-dep
with the Windows fix).
Now, my Windows machine is a Frankenstein's monster with GNU tools available on the path (like rm
, which some post-install scripts seem to use), and properly interpreting .sh scripts (which are also used as post install scripts), so it working on my machine doesn't imply it working on any Windows machine.
There is still a lot of work to do for Windows, but this gets us significantly closer.
Yes, I tested the install updating Electron to 1.4.3, and everything seemed to install correctly
Great to hear! I'll make sure to pull the new version in as soon as I get the time to do it.
Re. those shell scripts, ideally we shouldn't use any shell scripts in the development environment (due to the reason you laid out above). The scripts were created to quickly get all the different custom versions of ipfsd-ctl
, js-ipfs-api
, go-ipfs-dep
etc. to work together due to version conflicts with pretty much in all of them (eg. ipfs-api got pulled in 5(!) times, each from different package, each different version 😢). I expect that we can completely remove the scripts as soon as the work on go-ipfs-dep
and js-ipfs-api
is done to support a) custom version of go-ipfs binary and b) pubsub support in *-ipfs-api. @mateon1 what do you think, does that make sense?
cc @diasdavid @dignifiedquire see above the comment about upcoming work on the ipfs packages. I can make sure go-ipfs-dep works, but would need some help on publishing custom versions (like, I want to be able to require go-ipfs-dep@x.x.x-floodsub
)
@mateon1 pushed Electron 1.4.3 dep to master.
Adding this issue to a milestone for the next quarter. The goal is to make Orbit work on Windows too.
Even if the build issue due to
go-ipfs-dep
on Windows is fixed, a new issue appears. When runningnpm start
:There is nothing interesting in
npm-debug.log
.