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

Fix not-working cupsSNMPSupplies and cupsIPPSupplies settings via lpa… #5610

Closed
wants to merge 2 commits into from
Closed

Fix not-working cupsSNMPSupplies and cupsIPPSupplies settings via lpa… #5610

wants to merge 2 commits into from

Conversation

zdohnal
Copy link
Contributor

@zdohnal zdohnal commented Jul 9, 2019

…dmin

Connected to issue #5609

@michaelrsweet
Copy link
Collaborator

Looks about right, will need to merge by hand however...

@zdohnal
Copy link
Contributor Author

zdohnal commented Jul 16, 2019

Oh, I missed the indentation - I hope I corrected it now, please correct me if I'm wrong.

@michaelrsweet
Copy link
Collaborator

Indentation looks fine, I just can't merge to multiple Git branches via pull requests. No biggy, this is a small change...

@michaelrsweet
Copy link
Collaborator

Oh, and there were two other places where we needed to set ppdchanged to 1...

[master 9539c53] Fix lpadmin with cupsIPPSupplies and cupsSNMPSupplies (Issue #5610)

[branch-2.2 c507739] Fix lpadmin with cupsIPPSupplies and cupsSNMPSupplies (Issue #5610)

@zdohnal
Copy link
Contributor Author

zdohnal commented Jul 16, 2019

Indentation looks fine, I just can't merge to multiple Git branches via pull requests. No biggy, this is a small change...

Can I do anything about it in the future?

Oh, and there were two other places where we needed to set ppdchanged to 1...

D'oh, I missed those two, thank you for checking!

[master 9539c53] Fix lpadmin with cupsIPPSupplies and cupsSNMPSupplies (Issue #5610)

[branch-2.2 c507739] Fix lpadmin with cupsIPPSupplies and cupsSNMPSupplies (Issue #5610)

Thank you for the fix!

@michaelrsweet
Copy link
Collaborator

@zdohnal Unfortunately, Github is all about merging to master. Until they add support for pull requests against alternate branches I'll just keep doing manual merges...

Most of these fixes are pretty small anythings, so it is no problem - the bulk of the work has already been done! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants