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.

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

package: move lpr into its own package #1171

Closed
wants to merge 1 commit into from
Closed

Conversation

llfw
Copy link
Contributor

@llfw llfw commented Apr 15, 2024

several of the lp-related utilities are setuid, so this is a concrete security benefit for sites that don't need printing support.

@llfw
Copy link
Contributor Author

llfw commented Apr 15, 2024

i can squash this prior to commit, but that makes it awkward to merge/test the change locally, so i'll wait until review is finished first, if that's okay.

@bsdimp
Copy link
Member

bsdimp commented Apr 15, 2024

@evadot What do you think of this?

@evadot
Copy link
Contributor

evadot commented Apr 15, 2024

LGTM, that's something I wanted to do a while ago but just forgot about it :)

@bsdimp
Copy link
Member

bsdimp commented Apr 19, 2024

merged, forgot to tag manu on reviewed, even though he gave the lgtm. thanks for the cleanup.

@bsdimp bsdimp closed this Apr 19, 2024
freebsd-git pushed a commit that referenced this pull request Apr 19, 2024
Reviewed by: imp
Pull Request: #1171
@bsdimp bsdimp added merged and removed ready labels Apr 21, 2024
bsdjhb pushed a commit to bsdjhb/cheribsd that referenced this pull request Aug 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants