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.

Sign up for GitHub

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

lpoptions -d as root #454

Closed
debiantriage opened this issue Aug 14, 2022 · 1 comment · Fixed by #456
Closed

lpoptions -d as root #454

debiantriage opened this issue Aug 14, 2022 · 1 comment · Fixed by #456
Assignees
@zdohnal
Labels
bug Something isn't working priority-medium
Milestone
v2.4.x

Comments

@debiantriage
Copy link

Yair Yarom submitted Debian bug #1008053 and observed that running lpoptions as root does not update /etc/cups/lpoptions but /root/.cups/lpoptions. He was using 2.3.3op2-3+deb11u1 and included a patch in the report.

I observe the same behaviour with 2.4.2 but not with 2.2.10-6+deb10u4.
zdohnal added a commit to zdohnal/cups that referenced this issue Aug 15, 2022
@zdohnal
cups/dest.c: Write data into /etc/cups/lpoptions if we're root
8ae6eb1 
Fixes OpenPrinting#454 , the patch is created by Yair Yarom
(https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1008053).
@zdohnal zdohnal mentioned this issue Aug 15, 2022
Merged
@zdohnal
Copy link
Member

zdohnal commented Aug 15, 2022

Hi Brian,

thank you for the report and the patch! I've created PR with it and once it is reviewed, I'll merge it.

@zdohnal zdohnal self-assigned this Aug 15, 2022
@zdohnal zdohnal added bug Something isn't working priority-medium labels Aug 15, 2022
@zdohnal zdohnal modified the milestones: v2.4.3, v2.4.x Aug 15, 2022
@zdohnal zdohnal mentioned this issue Jun 5, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zdohnal zdohnal

Labels
bug Something isn't working priority-medium
Projects
None yet
Milestone
v2.4.x
Development

Successfully merging a pull request may close this issue.

cups/dest.c: Write data into /etc/cups/lpoptions if we're root zdohnal/cups
2 participants
@zdohnal @debiantriage