Ticket #138 (closed defect: fixed)

Opened 13 years ago

Last modified 12 years ago

LPRng doesn't support -z

Reported by: geofft Owned by: geofft
Priority: low Milestone: Fall 2009 Release
Component: -- Keywords:
Cc: Fixed in version:
Upstream bug:

Description

The -z (zephyr) option, which is in Athena 9's "lpr" command, isn't on the LPRng in Debian/Ubuntu?. Possibly there's an equivalent -o something, because zephyr is a server-side thing. And maybe we should make the lpr wrapper turn -z into this -o, although if we can get away with documenting the option and desupporting -z, that would be cleaner.

Change History

comment:1 Changed 12 years ago by geofft

  • Component set to --

-z appears to be just -m zephyr%$USER, which is easy enough to add in the wrapper.

It requires a little more concentration than I have right now to figure out how to turn -hzZduplex into -hZduplex -mzephyr%geofft without turning -Putz into -Put -mzephyr%geofft. Basically, around line 115, if o == '-z', replace it with the -m option. Then rejoin the modified options array into your own version of the argv string, and pass it back to main() to use in fn_args in place of sys.argv.

comment:2 Changed 12 years ago by broder

Yes, that's about the right way to do this. You don't have to do worry about parsing -Putz into -Put -mzephyr%geofft if you do the find/replace after getopt has already run through the arguments. This is why we have to go to all the trouble of using getopt and tracking the different available arguments in the first place.

comment:3 Changed 12 years ago by geofft

  • Status changed from new to proposed

Implemented as outlined above in r23945 and uploaded to -proposed.

comment:4 Changed 12 years ago by mitchb

  • Owner set to geofft
  • Status changed from proposed to assigned

As discussed on zephyr, the -z option is supposed to be a toggle, but
isn't get implemented as such.

Also, we should make this the default for people who take the
debathena-printing-config package.

comment:5 Changed 12 years ago by broder

  • Status changed from assigned to proposed

-z as a toggle and on by default should be fixed in r23954, which is now in proposed

comment:6 Changed 12 years ago by jdreed

  • Milestone set to Fall Release

comment:7 Changed 12 years ago by broder

  • Status changed from proposed to closed
  • Resolution set to fixed
Note: See TracTickets for help on using tickets.