sujay Reporter
	Description • 23 years ago

from bug 92344

In 4.x if lp is not definted, it throws up an error panel with something
like this:  "lpr: lp: unknown printer". 

we need to do something like this for 6.x

	Roland Mainz
	Comment 1 • 23 years ago

mozilla/gfx/public/nsIDeviceContext.h has some error codes for this purpose.
NS_ERROR_GFX_PRINTER_CMD_NOT_FOUND may be the one you're looking for...

	rods (gone)
	Comment 2 • 23 years ago

taking

	rods (gone)
	Comment 3 • 23 years ago

Is this still a problem?

	sujay Reporter
	Comment 4 • 23 years ago

yes this should still be a problem. see how we handle it 
for 4.x...

	Pete Zha Assignee
	Comment 5 • 23 years ago

Who have idea on how to get error when lp/lpr not defined?
I checked these code:
pipe = popen(mPrintSetup->print_cmd, "w");
The popen will always return no-NULL even command not found. And errno is always 0.
Shall I check the file exists for lp/lpr? I don't think it's a good idea since
it make things more complex...

	Pete Zha Assignee
	Comment 6 • 23 years ago

Based on Roland(Roland.Mainz@informatik.med.uni-giessen.de)'s patch(attachment
83527 [details] [diff] [review]) in bug 143109, but use errno to determine error.

	Pete Zha Assignee
	Comment 7 • 23 years ago

*** Bug 143109 has been marked as a duplicate of this bug. ***

	Pete Zha Assignee
	Comment 8 • 23 years ago

=>

	Pete Zha Assignee
	Comment 9 • 22 years ago

Roland, dcone,
Could you give r=? Thanks

	Roland Mainz
	Comment 10 • 22 years ago

Comment on attachment 92355 [details] [diff] [review]
patch

r=Roland.Mainz@informatik.med.uni-giessen.de

	Boris Zbarsky [:bzbarsky]
	Comment 11 • 22 years ago

Comment on attachment 92355 [details] [diff] [review]
patch

sr=bzbarsky

	Pete Zha Assignee
	Comment 12 • 22 years ago

checked in trunk

	Roland Mainz
	Comment 13 • 22 years ago

Requesting a{1.0.2}= ...