Skip to content

/ docs

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

Jump to bottom

Removed error in the Instructions #461

Open
wants to merge 4 commits into
base: master
from
Open

Removed error in the Instructions #461

wants to merge 4 commits into from

Conversation

@bytenaija
Copy link

@bytenaija bytenaija commented Sep 16, 2020

Removed error in the system-config-printer re-installation instructions
@bytenaija
Removed error in the Instructions
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
c3ba87c 
Removed error in the system-config-printer re-installation instructions
@jacobgkau
jacobgkau requested changes Sep 16, 2020
View changes
_articles/add-a-printer-pop.md Outdated
@@ -53,7 +53,7 @@ To look at the CUPS (Common Unix Printing System) configuration and status windo

[localhost:631](http://localhost:631)

The status window will show current print jobs, detected printers, and other information about the printing system. If you would like to share this printer with others on your local network, click on the 'Admin' link, under Server, click on the "Share printers connected to this system" and save the changes. Other computers on your network should than see that printer. When there is a prompt for your username and password, use your user name, and password used to login.
The status window will show current sudo apt install --reinstall ~n^system-config-printerprint jobs, detected printers, and other information about the printing system. If you would like to share this printer with others on your local network, click on the 'Admin' link, under Server, click on the "Share printers connected to this system" and save the changes. Other computers on your network should than see that printer. When there is a prompt for your username and password, use your user name, and password used to login.

This comment has been minimized.

Sign in to view
@jacobgkau

jacobgkau Sep 16, 2020
Member

Why is the command pasted in the middle of a sentence?
_articles/add-a-printer-pop.md
@@ -64,7 +64,7 @@ sudo apt install --reinstall cups cups-client
This will reinstall <u>CUPS</u>, the main printing software, which can help with generic issues.

```
sudo apt install --reinstall ~n^system-config-printer
sudo apt install --reinstall system-config-printer

This comment has been minimized.

Sign in to view
@jacobgkau

jacobgkau Sep 16, 2020
Member

This was not an error, ~n^system-config-printer will install any package with system-config-printer in the name. This includes:

system-config-printer
system-config-printer-gnome
system-config-printer-udev
system-config-printer-common

The revised command would only install system-config-printer. Is there a reason you would not recommend reinstalling the other three packages?

This comment has been minimized.

Sign in to view
@bytenaija

bytenaija Sep 16, 2020
Author

I actually get this error when I try that command
no matches found: ~n^system-config-printer
@bytenaija
Copy link
Author

@bytenaija bytenaija commented Sep 16, 2020

@bytenaija
Removed the erroneous command within the paragraph
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
493ffa3
@jacobgkau
Copy link
Member

@jacobgkau jacobgkau commented Sep 16, 2020

I actually get this error when I try that command no matches found: ~n^system-config-printer

I have confirmed the command still works properly on a clean Pop!_OS 20.04 machine. What version of Pop are you running? Please provide the complete output of these commands:

cat /etc/os-release
sudo apt update
apt policy apt
sudo apt install --reinstall ~n^system-config-printer
ahoneybun added 2 commits Oct 2, 2020
@ahoneybun
Merge branch 'master' into patch-1
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
25c0724
@ahoneybun
Merge branch 'master' into patch-1
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
a1b99ad
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jacobgkau jacobgkau

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
@bytenaija @jacobgkau @ahoneybun
You can’t perform that action at this time.