[25630] in Source-Commits

home help back first fref pref prev next nref lref last post

/svn/athena r25136 - in trunk/debathena/config/printing-config: debathena/printing debian

daemon@ATHENA.MIT.EDU (Jonathan D Reed)
Tue Jun 14 16:07:22 2011

Date: Tue, 14 Jun 2011 16:07:16 -0400
From: Jonathan D Reed <jdreed@MIT.EDU>
Message-Id: <201106142007.p5EK7GVT019029@drugstore.mit.edu>
To: source-commits@mit.edu
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Author: jdreed
Date: 2011-06-14 16:07:16 -0400 (Tue, 14 Jun 2011)
New Revision: 25136

Modified:
   trunk/debathena/config/printing-config/debathena/printing/lpr.py
   trunk/debathena/config/printing-config/debathena/printing/test_lpr.py
   trunk/debathena/config/printing-config/debian/changelog
Log:
In printing-config:
  * Kill LPROPT with fire (Trac: #882)
  * That's not how you spell 'nonexistent'


Modified: trunk/debathena/config/printing-config/debathena/printing/lpr.py
===================================================================
--- trunk/debathena/config/printing-config/debathena/printing/lpr.py	2011-06-14 19:43:37 UTC (rev 25135)
+++ trunk/debathena/config/printing-config/debathena/printing/lpr.py	2011-06-14 20:07:16 UTC (rev 25136)
@@ -62,10 +62,6 @@
 
 def _main(args):
     args.pop(0)
-    if 'LPROPT' in os.environ:
-        args = shlex.split(os.environ['LPROPT']) + args
-        # If we leave LPROPT set, LPRng will count the options twice
-        del os.environ['LPROPT']
 
     queue = common.get_default_printer()
     argstyle = None
@@ -117,6 +113,9 @@
     if system == common.SYSTEM_CUPS and argstyle == common.SYSTEM_LPRNG:
         args = translate_lprng_args_to_cups(args)
 
+    if system == common.SYSTEM_CUPS and 'LPROPT' in os.environ:
+        sys.stderr.write("Use of the $LPROPT environment variable is deprecated and\nits contents will be ignored.\nSee http://kb.mit.edu/confluence/x/awCxAQ\n")
+
     common.dispatch_command(system, 'lpr', args)
 
 

Modified: trunk/debathena/config/printing-config/debathena/printing/test_lpr.py
===================================================================
--- trunk/debathena/config/printing-config/debathena/printing/test_lpr.py	2011-06-14 19:43:37 UTC (rev 25135)
+++ trunk/debathena/config/printing-config/debathena/printing/test_lpr.py	2011-06-14 20:07:16 UTC (rev 25136)
@@ -65,7 +65,7 @@
         self.mox.StubOutWithMock(os, 'execvp')
 
 
-class TestNonexistantPrinter(TestLpr):
+class TestNonexistentPrinter(TestLpr):
     # LPROPT, PRINTER are unset
     environ = {'ATHENA_USER': 'quentin'}
 
@@ -88,31 +88,6 @@
         lpr._main(['lpr', '-Pstark', 'puppies biting nose.jpg'])
 
 
-class TestLpropt(TestLpr):
-    environ = {'ATHENA_USER': 'jdreed', 'LPROPT': '-Zduplex'}
-    backends = ['get-print.mit.edu']
-
-    def test(self):
-        """Test printing with LPROPT set.
-
-        Taken from Trac #509, reported on Mar 12, 2010."""
-        # We now call common.find_queue twice
-        common._hesiod_lookup('ajax', 'pcap').AndReturn(['ajax:rp=ajax:rm=GET-PRINT.MIT.EDU:ka#0:mc#0:'])
-        common.get_cups_uri('ajax').AndReturn(None)
-        common.is_cups_server('GET-PRINT.MIT.EDU').AndReturn(True)
-        common.get_default_printer().AndReturn(None)
-        common._hesiod_lookup('ajax', 'pcap').AndReturn(['ajax:rp=ajax:rm=GET-PRINT.MIT.EDU:ka#0:mc#0:'])
-        common.get_cups_uri('ajax').AndReturn(None)
-        common.is_cups_server('GET-PRINT.MIT.EDU').AndReturn(True)
-
-        # Result:
-        os.execvp('cups-lpr', ['lpr', '-Ujdreed', '-Pajax', '-osides=two-sided-long-edge', '-m'])
-
-        self.mox.ReplayAll()
-
-        lpr._main(['lpr', '-P', 'ajax'])
-
-
 class TestNoLpropt(TestLpr):
     environ = {'ATHENA_USER': 'jdreed'}
     backends = ['get-print.mit.edu']

Modified: trunk/debathena/config/printing-config/debian/changelog
===================================================================
--- trunk/debathena/config/printing-config/debian/changelog	2011-06-14 19:43:37 UTC (rev 25135)
+++ trunk/debathena/config/printing-config/debian/changelog	2011-06-14 20:07:16 UTC (rev 25136)
@@ -1,8 +1,10 @@
 debathena-printing-config (1.23.3) unstable; urgency=low
 
   * Don't barf on unexpected args (Trac #719)
+  * Kill LPROPT with fire (Trac: #882)
+  * That's not how you spell 'nonexistent'
 
- -- Jonathan Reed <jdreed@mit.edu>  Tue, 14 Jun 2011 15:41:41 -0400
+ -- Jonathan Reed <jdreed@mit.edu>  Tue, 14 Jun 2011 16:07:10 -0400
 
 debathena-printing-config (1.23.2) unstable; urgency=low
 


home help back first fref pref prev next nref lref last post