search

holepunchto / bare-dev

Development tooling for Bare
Apache License 2.0
8 stars 2 forks source link

Fixed some minor issues but stuck at Mirror drive #12

Closed n9lsjr closed 6 months ago

n9lsjr commented 6 months ago

Very impressive stuff you guys are building. I'm testing https://github.com/holepunchto/pear-expo-hello-world and i've encountered some issues. Some pre-info: I disabled the build processes for iOS because i do not have xcode image

  1. On some windows machines it seems like you need to add { shell: true } to the sdk.manager process. image Otherwise I got EINVAL error. image

2: I fixed the issue with execute_process with a hardcoded path ( The previous issue reported did not solve it for me ) image I seemed like the build process did not identify (WIN32) for my system by some odd reason ( logged the if/else statement ), and even though I changed the else condition to match the windows setting I still got the same error. image

3. I was using the standard dependency 0.7 of bare dev ~ and it never cloned the prebuilds folder correctly and got stuck here: image I then updated to the latest 0.10.4 and now it gets stuck at here. image

Not sure if my hacking has fked something up or if my multiple re-runs of hello-pear --configure has made something break. Anyways hope this issue might help somebody else who got stuck 👍

kasperisager commented 6 months ago

Thanks for the report! I fixed the shell option, that's indeed needed when executing batch scripts on Windows. The rest seem environment specific though, I couldn't reproduce those. In particular the mirroring can be slow depending on where you are in the world as it downloads a pretty substantial amount of data from the swarm.

n9lsjr commented 6 months ago

Great 👍 I will try again and disable VPN etc. Thanks 💯

kasperisager commented 6 months ago

Had to roll back the shell option default as it breaks command arguments.

n9lsjr commented 6 months ago

Ok 👍 The mirror / downloading worked btw 🎉

n9lsjr commented 6 months ago

I think that was a problem for me too when I added shell : true to all exec, spawn etc. It was only needed in the end to start en sdk.manager process with execFile 👍

Had to roll back the shell option default as it breaks command arguments.