-
Notifications
You must be signed in to change notification settings - Fork 11
Project doesn't build (Win10 64bit) #484
Comments
Hi @deroude , |
@jpommerening |
@alex3683: Okay, this took ridiculously long and I'm sorry for that. I think the line in question was just blindly copied from grunt-laxar. I seem to be unable to work with the Windows box I was given and I'm too much of a cheapskate to buy a Windows license and virtualization solution for myself and my personal notebook doesn't have VTx so what's the use anyway… *keeps on listing excuses* As long as the tests pass I'd say go for it |
Good that you mentioned |
Ohh, guess I thought that was just a forward-ported file from a reasonably recent node version without any patches 🙈
… On 13. Feb 2018, at 22:01, Alex ***@***.***> wrote:
Good that you mentioned grunt-laxar. @x1B added a path implementation to fix problems on windows (https://github.com/LaxarJS/grunt-laxar/blob/master/lib/path-platform/path.js). @x1B , do you think we could fix this issue by simply copying that code to laxar-loader and laxar-tooling?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
Apparently 😸 |
@jpommerening I failed to get it running with the old grunt-laxar stuff. However, it works on windows and on my virtual linux machine when using the default path module and not the posix implementation. Is there anything I'm not aware of that prevents us from using the default module instead of posix? |
@alex3683 go for it 🚢 This might actually be a case where our testing approach in laxar-tooling might be helpful :) |
@jpommerening As you can see I pushed my fixes, but for some reason I cannot publish the npm packages (wish I wanted to do before updating the yo generator). Could you try if you can publish them? |
@alex3683: Oh yeah, I think they changed how the permissions work. I just added a few more packages to the "developers" team so you should give it another try! Also, you're (co-)owner of the laxarjs org at npm, so you should already be able to assign your user to the relevant teams at https://www.npmjs.com/org/laxarjs/teams using their super-horrible UI. |
Good work btw :) |
Okay, successfully published 👍 @deroude Sorry that it took us so long, but up to now our own development environments were only Mac OS X and Linux. Seems that I am the only one having a windows machine that can also be used for development 😄 |
The following listing shows the steps and the outcome:
The text was updated successfully, but these errors were encountered: