Compatibilty with cygwin

[u0078867]

Hi guys,

for job-related reasons, I have to use Windows + cygwin, and the paper creation works like a charm, apart from the conversion from .tex to .pdf from Python. This is due to the difference in symbol for piping operations in Unix (|) and Windows (&). In my case, a tiny replacement of that symbol at line 109 of file publisher/build_paper.py would do the trick. But perhaps it would be great for the next year to add a programmatic platform check in the code :)

Cheers!

Davide

[pdebuyl]

Hi Davide,

Thanks for the feedback. I don't see any pipe character in the fiel publisher/build_paper.py but it would be nice if the code worked on windows as well. What modifications did you do to have it work? If it proves to be portable we'll be happy to include it!

Groeten, Pierre

[u0078867]

Ops sutpid me! I wanted to mean, semicolon, not piping! so at that line, turning ; into & did the job.

Cheers Davide

[pdebuyl]

Ok, I see the problem. I'll have a look at it to avoid a "if/then" here, there may be a proper subprocess way to handle it.

[u0078867]

Or maybe feeding pdflatex with the full .tex file path, avoiding cd beforehand.

Davide

[pdebuyl]

If you have the opportunity, you may test the code here https://github.com/euroscipy/euroscipy_proceedings/tree/tmp_molinari

Changeset: https://github.com/euroscipy/euroscipy_proceedings/commit/c375e825ab6eb0326652a64d50e105edf3de5d29

I used the cwd argument that sets the working directory, instead of relying on cd in the command.

[u0078867]

Gald to tell you that it works :)

[pdebuyl]

Ok, then https://github.com/euroscipy/euroscipy_proceedings/commit/880a7c4f4bc083a2bbececaba3888bb84109dabe fixes it. I close the PR :-)

I sent the fix to https://github.com/scipy-conference/scipy_proceedings too.