-
Notifications
You must be signed in to change notification settings - Fork 772
Add instructions for helpless Ubuntu users like myself. #1533
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
Conversation
README.rst
Outdated
@@ -26,6 +26,14 @@ then run the following command to install the other packages that the draft requ | |||
|
|||
sudo tlmgr install latexmk isodate substr relsize ulem fixme rsfs extract layouts enumitem | |||
|
|||
------------------------- | |||
Getting Started on Ubuntu |
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.
Should this say "Debian" instead? Or "Debianesque systems"?
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.
It could. I picked Ubuntu because that's what OS I experienced the issue on. I suppose it could say "Getting Started on Systems With apt-get"?
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.
@AaronBallman: Yes. We call those systems "Debian" :-) (Or Debian-based.)
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.
Happy to change it however you'd prefer. Would you like me to go with Debian-based or Debian?
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 think "Debian-based" is good.
7e08626
to
bcaf7e1
Compare
Okay, this should be set with the new title now. |
FWIW the TeX packages I install on Fedora are: |
Would you like me to add a Fedora section? If so, let me know what you'd like the title to be and what command you use to install those packages, and I'll add another section to the patch. |
@AaronBallman Yes, please add Fedora info as well |
Am I correct in understanding this would be: sudo yum install <packages> ? |
should be |
96ebc47
to
b33f411
Compare
I've updated the patch for Fedora as well. |
No, yum is deprecated, (as root) |
b33f411
to
2cc35a2
Compare
One more try, this time with dnf instead of yum. |
Hrm, sorry to be a pain, now I look at it, it would probably be better to put the (as root) after "Fedora" not before the |
2e592a6
to
9952d89
Compare
I wasn't too keen on putting (as root) after Fedora, but I didn't mind putting it in the instructions. How does this look to you? |
I wouldn't even necessarily say "as root" at all -- people should know how to use their system; if they're already in the business of installing packages, they probably know how to do it right. |
I can certainly remove it, but we do show sudo in the other two examples, and it seems somewhat strange to leave it off for Fedora. As @jwakely pointed out, sudo is not configured by default, but perhaps we could still use it for this case as well (and assume people know how to configure sudo or drop into su manually)? |
Yes, that's true, and we can keep it in, I don't mind. It was just an idea. |
Why not do sudo here too? |
README.rst
Outdated
|
||
Install the following packages (as root): | ||
|
||
dnf install texlive texlive-isodate texlive-relsize texlive-ulem texlive-fixme texlive-extract |
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.
add sudo here to be consistent with Debian-based instructions
7a34e3d
to
62639df
Compare
Added sudo and removed the (as root). |
No, please don't add By default any Ubuntu user can just add
Instead of assuming they know how to modify their I'm fine with just dropping the "(as root)" part and assuming Fedora users building the standard know how to install packages, but a command that doesn't work is not helpful. I'd rather not mention Fedora at all than have a command that won't work. |
df32768
to
959c4a2
Compare
Updated to remove the sudo command for Fedora. |
Getting started on Ubuntu turns out to be a bit more complex than just getting latexmk and running with it. Building the documentation fails without these other packages installed, so we might as well mention that.