Convert the option parser to use getopt(3). This makes it more

orthogonal with the rest of the system (you can now use either -PPS or
-P PS), and makes the parser more intelligible.  The only drawback is
that the old semantics for the -i flag in case a non-numeric argument
is following are no longer fully supported (only if -i is the very
last arg at all), since getopt(3) doesn't support the discticnction
between numeric and non-numeric arguments.

Make lpr also understand dashes as input pseudo filenames.  This
finally makes lp(1) fully comply with Posix.2.
This commit is contained in:
Joerg Wunsch 1996-05-11 19:00:55 +00:00
parent 7e703be7cd
commit afc417e07b
2 changed files with 90 additions and 88 deletions

View File

@ -29,8 +29,9 @@
.\" OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
.\" SUCH DAMAGE.
.\"
.\" @(#)lpr.1 8.1 (Berkeley) 6/6/93
.\"
.\" From @(#)lpr.1 8.1 (Berkeley) 6/6/93
.\" $Id$
.\" "
.Dd June 6, 1993
.Dt LPR 1
.Os BSD 4
@ -45,7 +46,7 @@
.Op Fl J Ar job
.Op Fl T Ar title
.Op Fl U Ar user
.Op Fl i Op Ar numcols
.Op Fl i Ar numcols
.Op Fl 1234 Ar font
.Op Fl w Ns Ar num
.Op Fl cdfghlnmprstv
@ -182,12 +183,9 @@ also for accounting purposes.
This option is only honored if the real user-id is daemon
(or that specified in the printcap file instead of daemon),
and is intended for those instances where print filters wish to requeue jobs.
.It Fl i Op numcols
The output is indented. If the next argument
is numeric
.Pq Ar numcols ,
it is used as the number of blanks to be printed before each
line; otherwise, 8 characters are printed.
.It Fl i numcols
The output is indented by
.Pq Ar numcols .
.It Fl w Ns Ar num
Uses
.Ar num

View File

@ -44,7 +44,8 @@ static char copyright[] =
#endif /* not lint */
#ifndef lint
static char sccsid[] = "@(#)lpr.c 8.4 (Berkeley) 4/28/95";
static char sccsid[] = "From: @(#)lpr.c 8.4 (Berkeley) 4/28/95"
"\n$Id$\n";
#endif /* not lint */
/*
@ -99,6 +100,7 @@ static char *width; /* width for versatec printing */
static struct stat statb;
static void card __P((int, char *));
static int checkwriteperm __P((char*, char *));
static void chkprinter __P((char *));
static void cleanup __P((int));
static void copy __P((int, char []));
@ -109,7 +111,7 @@ static char *lmktemp __P((char *, int, int));
static void mktemps __P((void));
static int nfile __P((char *));
static int test __P((char *));
static int checkwriteperm __P((char*, char *));
static void usage __P((void));
void
main(argc, argv)
@ -120,7 +122,7 @@ main(argc, argv)
struct group *gptr;
register char *arg, *cp;
char buf[BUFSIZ];
int i, f;
int c, i, f, errs;
struct stat stb;
if (signal(SIGHUP, SIG_IGN) != SIG_IGN)
@ -136,121 +138,104 @@ main(argc, argv)
gethostname(host, sizeof (host));
openlog("lpd", 0, LOG_LPR);
while (argc > 1 && argv[1][0] == '-') {
argc--;
arg = *++argv;
switch (arg[1]) {
errs = 0;
while ((c = getopt(argc, argv,
":#:1:2:3:4:C:J:P:T:U:cdfghi:lnmprstvw:")) != EOF)
switch (c) {
case '#': /* n copies */
i = atoi(optarg);
if (i > 0)
ncopies = i;
case 'P': /* specifiy printer name */
if (arg[2])
printer = &arg[2];
else if (argc > 1) {
argc--;
printer = *++argv;
}
case '1': /* troff fonts */
case '2':
case '3':
case '4':
fonts[optopt - '1'] = optarg;
break;
case 'C': /* classification spec */
hdr++;
if (arg[2])
class = &arg[2];
else if (argc > 1) {
argc--;
class = *++argv;
}
break;
case 'U': /* user name */
hdr++;
if (arg[2])
person = &arg[2];
else if (argc > 1) {
argc--;
person = *++argv;
}
class = optarg;
break;
case 'J': /* job name */
hdr++;
if (arg[2])
jobname = &arg[2];
else if (argc > 1) {
argc--;
jobname = *++argv;
}
jobname = optarg;
break;
case 'P': /* specifiy printer name */
printer = optarg;
break;
case 'T': /* pr's title line */
if (arg[2])
title = &arg[2];
else if (argc > 1) {
argc--;
title = *++argv;
}
title = optarg;
break;
case 'l': /* literal output */
case 'p': /* print using ``pr'' */
case 't': /* print troff output (cat files) */
case 'n': /* print ditroff output */
case 'U': /* user name */
hdr++;
person = optarg;
break;
case 'c': /* print cifplot output */
case 'd': /* print tex output (dvi files) */
case 'g': /* print graph(1G) output */
case 'c': /* print cifplot output */
case 'l': /* literal output */
case 'n': /* print ditroff output */
case 't': /* print troff output (cat files) */
case 'p': /* print using ``pr'' */
case 'v': /* print vplot output */
format = arg[1];
format = optopt;
break;
case 'f': /* print fortran output */
format = 'r';
break;
case '4': /* troff fonts */
case '3':
case '2':
case '1':
if (argc > 1) {
argc--;
fonts[arg[1] - '1'] = *++argv;
}
case 'h': /* toggle want of header page */
hdr = !hdr;
break;
case 'w': /* versatec page width */
width = arg+2;
break;
case 'r': /* remove file when done */
rflag++;
case 'i': /* indent output */
iflag++;
indent = atoi(optarg);
break;
case 'm': /* send mail when done */
mailflg++;
break;
case 'h': /* toggle want of header page */
hdr = !hdr;
case 'q': /* just queue job */
qflag++;
break;
case 'r': /* remove file when done */
rflag++;
break;
case 's': /* try to link files */
sflag++;
break;
case 'q': /* just q job */
qflag++;
case 'w': /* versatec page width */
width = optarg;
break;
case 'i': /* indent output */
iflag++;
indent = arg[2] ? atoi(&arg[2]) : 8;
case ':': /* catch "missing argument" error */
if (optopt == 'i') {
iflag++; /* -i without args is valid */
indent = 8;
} else
errs++;
break;
case '#': /* n copies */
if (isdigit(arg[2])) {
i = atoi(&arg[2]);
if (i > 0)
ncopies = i;
}
default:
errs++;
}
}
argc -= optind;
argv += optind;
if (errs)
usage();
if (printer == NULL && (printer = getenv("PRINTER")) == NULL)
printer = DEFLP;
chkprinter(printer);
@ -323,10 +308,16 @@ main(argc, argv)
/*
* Read the files and spool them.
*/
if (argc == 1)
if (argc == 0)
copy(0, " ");
else while (--argc) {
if ((f = test(arg = *++argv)) < 0)
else while (argc--) {
if (argv[0][0] == '-' && argv[0][1] == '\0') {
/* use stdin */
copy(0, " ");
argv++;
continue;
}
if ((f = test(arg = *argv++)) < 0)
continue; /* file unreasonable */
if (sflag && (cp = linked(arg)) != NULL) {
@ -679,6 +670,19 @@ chkprinter(s)
SC = (cgetcap(bp, "sc", ':') != NULL);
}
/*
* Tell the user what we wanna get.
*/
static void
usage()
{
fprintf(stderr,
"usage: lpr [-Pprinter] [-#num] [-C class] [-J job] [-T title] [-U user]\n"
"[-i[numcols]] [-1234 font] [-wnum] [-cdfghlnmprstv] [name ...]\n");
exit(1);
}
/*
* Make the temp files.
*/