rewriter: Fix that default_options would not set the correct id

Currently default_options uses "" for the kwarks id, however this
is incorect and it must be "/". Additionally, this error won't be
ignored in the future with "--skip" (this is why the tests were
passing and this wasn't detected earlier).

Author: mensinda

mesonbuild/rewriter.py

@@ -447,7 +447,7 @@ def process_default_options(self, cmd):
         # First, remove the old values
         kwargs_cmd = {
             'function': 'project',
-            'id': "",
+            'id': "/",
             'operation': 'remove_regex',
             'kwargs': {
                 'default_options': ['{}=.*'.format(x) for x in cmd['options'].keys()]
@@ -502,7 +502,7 @@ def process_kwargs(self, cmd):
         if cmd['function'] == 'project':
             if cmd['id'] != '/':
                 mlog.error('The ID for the function type project must be "/"', *self.on_error())
-                self.handle_error()
+                return self.handle_error()
             node = self.interpreter.project_node
             arg_node = node.args
         elif cmd['function'] == 'target':

test cases/rewrite/3 kwargs/delete.json

@@ -2,7 +2,7 @@
   {
     "type": "kwargs",
     "function": "project",
-    "id": "",
+    "id": "/",
     "operation": "delete",
     "kwargs": {
       "version": null