[pykickstart][PATCH] Remove a triple-X message that is no longer needed

Martin Kolman mkolman at redhat.com
Tue Oct 8 11:14:33 UTC 2013


As the comment above the XXX line showed up to be still valid
(the current master Pykickstart might still be used
with Python 2.4), the XXX line is not needed anymore.
Also Pylint doesn't seem to like XXX showing up in comments.

Related: rhbz#957809
Signed-off-by: Martin Kolman <mkolman at redhat.com>
---
 pykickstart/commands/firewall.py | 1 -
 1 file changed, 1 deletion(-)

diff --git a/pykickstart/commands/firewall.py b/pykickstart/commands/firewall.py
index 89e2212..2e61f70 100644
--- a/pykickstart/commands/firewall.py
+++ b/pykickstart/commands/firewall.py
@@ -203,7 +203,6 @@ class F20_Firewall(F14_Firewall):
             # python2.4 does not support action="append_const" that we were
             # using for these options.  Instead, we have to fake it by
             # appending whatever the option string is to the service list.
-            # XXX: is this still relevant ?
             if not value:
                 parser.values.ensure_value(option.dest, []).append(opt_str[2:])
                 return
-- 
1.8.3.1



More information about the anaconda-patches mailing list