search

aboSamoor / polyglot

Multilingual text (NLP) processing toolkit
http://polyglot-nlp.com
Other
2.31k stars 337 forks source link

Package data downloaded to / when superuser #54

Closed pudo closed 8 years ago

pudo commented 8 years ago

When deploying polyglot to a docker container ubuntu environment (thus running as root inside the container), the downloader simply drops the resources into / -- looks like there might be a bug in how the package_data path is determined for a superuser.

I was also wondering if it might make sense to consider a $POLYGLOT_PACKAGE_DATA environment variable when determining the path?

p.s. thanks for this amazing library! It's giving me great results and was much needed.

aboSamoor commented 8 years ago

Feel free to send me a PR to fix the issue.

On Wed, May 11, 2016, 1:26 AM Friedrich Lindenberg notifications@github.com wrote:

When deploying polyglot to a docker container ubuntu environment (thus running as root inside the container), the downloader simply drops the resources into / -- looks like there might be a bug in how the package_data path is determined for a superuser.

I was also wondering if it might make sense to consider a $POLYGLOT_PACKAGE_DATA environment variable when determining the path?

p.s. thanks for this amazing library! It's giving me great results and was much needed.

— You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub https://github.com/aboSamoor/polyglot/issues/54

mekarpeles commented 8 years ago

+1