blang / latex-docker

Docker-based latex compilation
MIT License
585 stars 146 forks source link

`exec` seems unecessary #30

Closed startakovsky closed 3 years ago

startakovsky commented 4 years ago

In latexdockercmd.sh it seems exec in the line below, when removed, still works just fine. Why is it necessary?

Thanks, Steven

https://github.com/blang/latex-docker/blob/42d2c532a09a00938331512fd2b539f9944c400c/latexdockercmd.sh#L3

blang commented 4 years ago

It replaces the shell process instead of forking. This comes in handy so you don't need to manage exit codes and signals. Please see https://mywiki.wooledge.org/WrapperScript for reference

startakovsky commented 4 years ago

Thanks Benedikt! Very helpful.

On Wed, Sep 30, 2020 at 11:52 AM Benedikt Lang notifications@github.com wrote:

It replaces the shell process instead of forking. This comes in handy so you don't need to manage exit codes and signals. Please see https://mywiki.wooledge.org/WrapperScript for reference

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/blang/latex-docker/issues/30#issuecomment-701578256, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFIMO6AZV44XFGNH3A5KMLSIN477ANCNFSM4R7MW2KA .