Bug #721

Off-by-one error in screensaver setup

Added by jfeise over 13 years ago. Updated over 13 years ago.

Status:Closed Start date:09/30/2010
Priority:Normal Due date:
Assignee:jams % Done:

0%

Category:Packages: Core Spent time: 1.00 hour
Target version:-

Description

There is an off-by-one error in mv_screensaver.py, resulting in a wrong screensaver being selected.
Patch is attached.

History

Updated by jfeise over 13 years ago

I am getting an error trying to update the issue. May have to do with trying to attach the patch.
Posting it in the body now:

+++ /usr/MythVantage/bin/mv_screensaver.py      2010-09-30 23:20:57.000000000 -0700
@@ -84,7 +84,7 @@
             startline = k
         if re.search(theme,line):
             foundline = k
-    themenumber=foundline-startline-2
+    themenumber=foundline-startline-1

     for line in screenconfigcontents:
         if re.match("selected.*$",line):

Updated by jfeise over 13 years ago

Missed the first line:

--- /usr/MythVantage/bin/mv_screensaver.py.orig 2010-06-04 00:22:16.000000000 -0700
+++ /usr/MythVantage/bin/mv_screensaver.py      2010-09-30 23:20:57.000000000 -0700
@@ -84,7 +84,7 @@
             startline = k
         if re.search(theme,line):
             foundline = k
-    themenumber=foundline-startline-2
+    themenumber=foundline-startline-1

     for line in screenconfigcontents:
         if re.match("selected.*$",line):

Updated by jams over 13 years ago

  • Assignee set to jams
  • Category set to Packages: Core

Is this for Xscreensaver vs gnome ?

Updated by jfeise over 13 years ago

xscreensaver

Updated by jams over 13 years ago

  • Status changed from New to Closed

fixed

Also available in: Atom PDF