According to the smb.conf man page, the "load printers" smb.conf global should have the following effect: A boolean variable that controls whether all printers in the printcap will be loaded for browsing by default. See the <smbconfoption name="printers"/> section for more details. lp_load_printers() is currently only checked in the load_printers() code-path prior to pcap_printer_read_fn(). It should also be considered in the pcap_cache_reload() code path.
Created attachment 14804 [details] proposed patchset, currently undergoing testing
(In reply to David Disseldorp from comment #1) > Created attachment 14804 [details] > proposed patchset, currently undergoing testing This has been merged upstream. 4.9 and 4.8 patches to follow...
Created attachment 14829 [details] cherry-pick for 4.9.next and 4.10.next
Created attachment 14830 [details] backport for 4.8.next (load_printers() takes ev and msg contexts)
Comment on attachment 14829 [details] cherry-pick for 4.9.next and 4.10.next Dave, do you want to have this patch in 4.10 too?
(In reply to Andreas Schneider from comment #5) > Comment on attachment 14829 [details] > cherry-pick for 4.9.next > > Dave, do you want to have this patch in 4.10 too? Yes, I hadn't seen that we'd already branched. Will update the summary.
@Karo please apply for 4.8, 4.9 and 4.10.
Comment on attachment 14829 [details] cherry-pick for 4.9.next and 4.10.next LGTM
Comment on attachment 14830 [details] backport for 4.8.next (load_printers() takes ev and msg contexts) LGTM too
(In reply to Guenther Deschner from comment #9) Pushed to autobuild-v4-{10,9,8}-test.
(In reply to Karolin Seeger from comment #10) Pushed to all branches. Closing out bug report. Thanks!