jupyter-xeus / xeus-cling

Jupyter kernel for the C++ programming language
BSD 3-Clause "New" or "Revised" License
3.02k stars 292 forks source link

Don't install into /usr/etc #453

Closed dimitry-ishenko closed 1 year ago

dimitry-ishenko commented 1 year ago

According to §4.9.3 of the Filesystem Hierarchy Standard:

"Note that /usr/etc is still not allowed: programs in /usr should place configuration files in /etc."

JohanMabille commented 1 year ago

CI errors are irrelevant here (looks like the removal of deprecated images in the main branch has not been taken into account, I think that rebasing the PR should get rid of them).

JohanMabille commented 1 year ago

Thanks for this fix! And sorry for the time it took to get merged.

dimitry-ishenko commented 1 year ago

@JohanMabille no worries. BTW if you are on Ubuntu 22.04 I do have native Debian packages for xeus-cling and a bunch of others hosted in my PPA:

https://launchpad.net/~ppa-verse/+archive/ubuntu/xeus-cling