Re: Revision 196634 of sh(1) breaks lp(1)

From: Jilles Tjoelker <jilles_at_stack.nl>
Date: Sat, 26 Sep 2009 23:31:30 +0200
On Sat, Sep 26, 2009 at 11:51:28AM -0700, Steve Kargl wrote:
> troutmask:sgk[208] cat .cshrc | lp -d pug
> /usr/bin/lpr: cannot access 

> troutmask:sgk[209] uname -a
> FreeBSD troutmask.apl.washington.edu 9.0-CURRENT FreeBSD 9.0-CURRENT #0 r197195M:
> Mon Sep 14 11:11:46 PDT 2009
> kargl_at_troutmask.apl.washington.edu:/usr/obj/usr/src/sys/SPEW  amd64

The problem seems not because of r196634 of sh(1), but because of
r194171 of lp(1) (see also PR standards/129554). As of that revision,
lp passes an empty filename to lpr if the new -t option is not used.
lpr cannot access that file and prints the error message.

The attached patch should fix the issue, and should also avoid invalid
arguments to lpr if an empty title is given.

I use CUPS and not FreeBSD lpr so I just checked that the lpr command
looks right in sh -x.

-- 
Jilles Tjoelker

Received on Sat Sep 26 2009 - 19:31:31 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:39:56 UTC