Bump hacking

... and fix the errors newly detected.

```
./bashate/bashate.py:300:15: F504 '...' % ... has unused named argument(s): line, warn
        print("%(filename)s:%(filelineno)s:1: %(error)s" %
              ^
./bashate/messages.py:238:17: E741 ambiguous variable name 'l'
            for l in v.long_msg.split('\n'):
                ^
```

Change-Id: I69a7682494550513b5190480ceb79315c501c383

bashate/bashate.py

@@ -300,9 +300,7 @@ class BashateRun(object):
         print("%(filename)s:%(filelineno)s:1: %(error)s" %
               {'filename': filename,
                'filelineno': filelineno,
-               'warn': "W" if warn else "E",
+               'error': error.replace(":", "", 1)})
-               'error': error.replace(":", "", 1),
-               'line': line.rstrip('\n')})
 
     def check_files(self, files, verbose, max_line_length=79):
         logical_line = ""

bashate/messages.py

@@ -235,6 +235,6 @@ def print_messages():
             'id': v.msg_id,
             'string': v.msg_str})
         if v.long_msg:
-            for l in v.long_msg.split('\n'):
+            for msg_line in v.long_msg.split('\n'):
-                print("            %s" % l)
+                print("            %s" % msg_line)
         print("")

requirements.txt

@@ -1,5 +1 @@
-# The order of packages is significant, because pip processes them in the order
-# of appearance. Changing the order has an impact on the overall integration
-# process, which may cause wedges in the gate later.
-
 pbr!=2.1.0,>=2.0.0 # Apache-2.0

test-requirements.txt

@@ -1,8 +1,5 @@
-# The order of packages is significant, because pip processes them in the order
+hacking>=4.1.0,<4.2.0;python_version<'3.8' # Apache-2.0
-# of appearance. Changing the order has an impact on the overall integration
+hacking>=7.0.0,<7.1.0;python_version>='3.8' # Apache-2.0
-# process, which may cause wedges in the gate later.
-
-hacking>=3.0.1,<3.1.0 # Apache-2.0
 
 coverage>=3.6
 discover