<div dir="ltr">Hello,<div><br></div><div>os.path.join is a python thing. We try to use it as much as possible, indeed.<div>Here this is an issue with the node.js build scripts.</div><div>Nobody tests those on windows, so it make sense it brakes.</div><div><br></div><div>We will accept patches to fix any windows issue. Please send a proper fix using something like</div><div><a href="https://nodejs.org/api/path.html#path_path_join_paths">https://nodejs.org/api/path.html#path_path_join_paths</a><br></div><div><br></div><div>Regards,<br clear="all"><div><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature">Pierre</div></div><br></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Le lun. 23 mars 2020 à 22:42, Trevor Hook <<a href="mailto:trevor.k.hook@googlemail.com">trevor.k.hook@googlemail.com</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">For cross platform compatibility, not specific to builbot config, I often do something along the lines of:<br>
<br>
pathway = 'path/to/file'<br>
pathway_parts = pathway.split('/')<br>
pathway_combined = str(os.path.join(pathway_parts))<br>
<br>
i.e. I'm setting paths as a str for human readable convenience, splitting it into component parts and then using some platform independent path package to recombine the components into the local systems expectation of a path string.<br>
<br>
In Python3, I tend to use pathlib Path rather than is.path, mainly because I tend to be working with Path objects anyway.<br>
<br>
Trevor<br>
<br>
On 23 March 2020 21:03:12 GMT, "Yngve N. Pettersen" <<a href="mailto:yngve@vivaldi.com" target="_blank">yngve@vivaldi.com</a>> wrote:<br>
>Hello all,<br>
><br>
>I am currently trying to update our customized buildbot instance, and<br>
>have  <br>
>been running into problems when trying to build our modified www/base  <br>
>package.<br>
><br>
>I am running "yarn build" on Windows, in a Git Bash shell, but the<br>
>build  <br>
>failed<br>
><br>
>I got this message:<br>
><br>
>-------------------<br>
>'.' is not recognized as an internal or external command,<br>
>operable program or batch file.<br>
>www\base\node_modules\webpack-shell-plugin\lib\index.js:168<br>
>         throw error;<br>
>         ^<br>
><br>
>Error: Command failed: ./node_modules/.bin/pug src/app/index.jade -o  <br>
>buildbot_www/static/<br>
>'.' is not recognized as an internal or external command,<br>
>operable program or batch file.<br>
><br>
>     at ChildProcess.exithandler (child_process.js:303:12)<br>
>     at ChildProcess.emit (events.js:311:20)<br>
>     at maybeClose (internal/child_process.js:1021:16)<br>
>     at Socket.<anonymous> (internal/child_process.js:443:11)<br>
>     at Socket.emit (events.js:311:20)<br>
>     at Pipe.<anonymous> (net.js:668:12) {<br>
>   killed: false,<br>
>   code: 1,<br>
>   signal: null,<br>
>cmd: './node_modules/.bin/pug src/app/index.jade -o<br>
>buildbot_www/static/'<br>
><br>
>-------------------<br>
><br>
>I have just confirmed that the problem is "/" vs "\" (forward vs.<br>
>backward  <br>
>slash).<br>
><br>
>Change the following lines in www/base/webpack.config.js from<br>
><br>
>   new WebpackShellPlugin({<br>
>     onBuildEnd:['./node_modules/.bin/pug src/app/index.jade -o  <br>
>buildbot_www/static/']<br>
>   }),<br>
><br>
><br>
>to<br>
><br>
>    new WebpackShellPlugin({<br>
>       onBuildEnd:['.\\node_modules\\.bin\\pug src/app/index.jade -o  <br>
>buildbot_www/static/']<br>
>    }),<br>
><br>
>and the build works.<br>
><br>
>This is probably a problem caused by webpack, but maybe you know how to<br>
> <br>
>work around it?<br>
><br>
>In the meantime I am patching the file locally.<br>
_______________________________________________<br>
users mailing list<br>
<a href="mailto:users@buildbot.net" target="_blank">users@buildbot.net</a><br>
<a href="https://lists.buildbot.net/mailman/listinfo/users" rel="noreferrer" target="_blank">https://lists.buildbot.net/mailman/listinfo/users</a><br>
</blockquote></div>