Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Please don't download binary into user's home directory in build.rs #216

Open
yshui opened this issue Jul 25, 2024 · 1 comment
Open

Please don't download binary into user's home directory in build.rs #216

yshui opened this issue Jul 25, 2024 · 1 comment
Assignees

Comments

@yshui
Copy link

yshui commented Jul 25, 2024

Here: https://github.com/plotly/plotly.rs/blob/main/plotly_kaleido/build.rs

I think this is unhygienic. It is surprising to me that building something would cause files to be added to my home directory. This also breaks when building in a sandboxed environment (which is how I found out).

Plus, it is downloading something that is not strictly needed for the build process. And it is also not going to achieve what I assume is your goal. What if I build on one machine then ship the binary to run somewhere else?

@andrei-ng
Copy link
Collaborator

andrei-ng commented Jul 25, 2024

Valid point. This has come up before also in mistral-rs

Will look into it when I have more time available.

Just as a side note, this specific issues is in context of the kaleido feature. If you do not use it, my advice is to build without it. Building without it avoids all the issues you have mentioned.

I am also not a big fan of kaleido to be honest since as you highlight, it introduces an external dependency. I don't use it in my own projects for exactly that reason. That functionality is something I miss in plotly itself and still hope for the ppl at plotly to come up with a better solution for that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants