-
-
Notifications
You must be signed in to change notification settings - Fork 298
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
Add support for TeX Live package manager (tlmgr
) packages
#1329
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @Okeanos, thanks for a(another) PR! Just to set expectations: we're definitely not going to accept this so long as it requires sudo
and we may not accept it even if it can be made to not because I've never heard a request for this before.
Just wanted to give you added context before you put a tonne more time into this, sorry ❤️
# TODO: See how we can get around the sudo requirement because of the way e.g. basictex installs itself | ||
# See also https://tug.org/texlive/doc/tlmgr.html#USER-MODE | ||
tlmgr_packages.each do |package| | ||
Kernel.system "sudo", "tlmgr", "remove", package |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm afraid there's no way we're going to accept this if sudo
is needed here, sorry.
Hey, thanks for the early feedback. I expected as much to be honest and mainly wanted to have a stab at it (regardless) because I like the Why? Mainly because outside of that PR there's pretty little documentation on the topic of integrating additional formats and there were a few interesting ideas for additional extension types. IntellIJ for example would likely work within the acceptable bounds. I just found TeX Live more interesting to start with. Also IntelliJ and other JetBrains tools are a little tricky if the Toolbox is used to manage IntelliJ etc. – the Toolbox doesn't have CLI and putting |
Allow tlmgr packages to be installed, dumped, checked, updated and cleaned up.
Yeh, I think it would be worth documenting what we'll consider/not if that'd be useful to folks like yourself? Could also potentially figure out how to have a plugin system here.
Yes, I could see that being a bit more acceptable 👍🏻. Could think about that in the docs? |
Knowing what you would consider acceptable for mainstream bundle extensions would be immensely helpful. A plugin system (similar to taps) that would allow providing a wrapper for e.g. Concerning the docs, there's a few things I'd be immediately interested in:
|
Added in #1333 |
Passing on this accordingly, sorry 😭 |
Hey, no worries! I did that without reaching to you first after all and when I noticed the |
Just in case anyone returns to this, tlmgr has a user mode that enables setting a user-writable directory for packages to be installed. This would require setting But I agree with Mike's additions in #1333: I don't perceive enough demand for built-in tlmgr support, even though I might use it myself. |
Just in case anyone returns to this, tlmgr has a user mode that enables setting a user-writable directory for packages to be installed. This would require setting But I agree with Mike's additions in #1333: I don't perceive enough demand presently for built-in tlmgr support, even though I might use it myself. |
Agreed 👍🏻 |
Allow tlmgr packages to be installed, dumped, checked, updated and cleaned up.
First stab at this … not entirely sure it'll work as expected. Also there are a couple of
sudo
calls in there because a standardbasictex
cask installation uses paths that are not user writable by default.There appear to be options to make this run in
user mode
, however, that has to be explicitly configured and should probably be handled viaargs
in the Brewfile resulting insudo
being skipped for those calls then?