Ticket #285 (closed defect: fixed)

Opened 12 years ago

Last modified 12 years ago

LPROPT isn't respected

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


opus:~ geofft$ export LPROPT=-Psipbmp3
opus:~ geofft$ lpr
meadow: nothing to print

Before deploying the fix for this you must fix #138 or people who set LPROPT=-z will be sad because they can't print anything.

We could probably fix these with wrappers around mit-lp* to set the right arguments (isn't there a -Zsomething for zephyr?). We could definitely fix these by Debathenifyng them more invasively than we do now.

Change History

comment:1 Changed 12 years ago by broder

How badly do we actually want this fixed? If we assume that Athena is eventually moving towards something that looks like CUPS, then we probably want to desupport LPROPT in favor of ~/.cups/lpoptions

comment:2 Changed 12 years ago by geofft

It's certainly a change of behavior that needs to be pretty obviously documented, well beyond what I've seen our change-of-behavior documentation practices are, and a lot of people set LPROPT. Mine personally gives me header pages that I can use (staff paper), and I'm sure hundreds of people suppress header pages entirely.

I also saw this used in a shell script the other day to force a certain printer. So I'm not sure that I want to see LPROPT go away.

Also, so long as LPRng is around (...and the default), we need something that affects it, because ~/.cups/lpoptions won't.

comment:3 Changed 12 years ago by geofft

While I'm thinking about hacking the debathena-printing-config wrapper script... it would be totally trivial to insert os.environ['LPROPT'] into the getopt call for lprng.

comment:4 Changed 12 years ago by geofft

  • Status changed from new to proposed

os.environ['LPROPT'] added to the lpr wrapper script's code to run mit-lpr in r23950. While this doesn't help people who run mit-lpr explicitly, release-team thinks we don't care about that use case.

This is just for LPRng's lpr; if you're using CUPS, you want ~/.cups/lpoptions.

comment:5 Changed 12 years ago by broder

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