We need to rewrite the tests as python tests. I am okay if you just disable the test in the patch for now. Before we convert the scripts that use the common module, we should add a python module too, then we can move the scripts in and make them testable.

Review of attachment 348672 [details] [review]:

Thanks. A great start!

::: gtkdoc-rebase.in
@@ +50,3 @@
+parser.add_argument('--version', action='version', version='@VERSION@')
+parser.add_argument('--html-dir', dest='html_dir', default='')
+parser.add_argument('--other-dir', dest='other_dir', default='')

this can be given multiple times, needs to become an array/list, use: action="append"

@@ +53,3 @@
+parser.add_argument('--dest-dir', dest='dest_dir', default='')
+parser.add_argument('--online', action='store_true', default=False)
+parser.add_argument('--version', action='version', version='@VERSION@')

please use a mutex group for online/relative:
https://docs.python.org/3/library/argparse.html#mutual-exclusion

@@ +78,3 @@
+    dir = subprocess.check_call(['@PKG_CONFIG@', '--variable=prefix', 'glib-2.0'], universal_newlines=True)
+    dir = dir.strip()
+                log(options, "Prepending GNOME2_PATH directory:", dir)

nit, whitespace after ',' but maybe we just run autopep8 once we've converted those.

@@ +82,3 @@
+    other_dirs = [dir] + other_dirs
+
+

one # is enough

@@ +90,3 @@
+        RelativizeLocalMap(options.html_dir);
+
+    log(options, "Prepending GLib directory", dir)

lowercase html_dir

@@ +120,3 @@
+                have_index = True
+
+            if file.is_dir():

None. More pythonic
if onlinedir and file.name == "index.sgml":

@@ +135,3 @@
+''' % (dir, file.name))
+
+                # Prefer this location over possibly stale index.sgml

IO:Zlib -> gzip module

@@ +156,2 @@
             # Remove trailing non-directory component.
+            onlinedir = re.sub(r'''(.*/).*''', r'\1', trial)

onlinedir = re.sub(r'''(.*/).*''', r'\1', match.groups(1))

@@ +167,3 @@
+            break
+        match = re.match(r'''^<ONLINE\s+href\s*=\s*"([^"]+)"\s*>''', line)
+        if match is not None:

if match:

@@ +171,2 @@
             # Remove trailing non-directory component.
+            onlinedir = re.sub(r'''(.*/).*''', r'\1', trial)

also inline 'trial' var

@@ +177,3 @@
+    package = None
+
+    package = os.path.split(dir)[1]

not sure, but isn't this taking the last dir entry? Maybe?
package = os.path.split(dir)[-1]

@@ +181,3 @@
+        dir = dir[len(options.dest_dir)-1:]
+
+    package = None

if onlinedir:

@@ +200,3 @@
+    prefix = os.path.split(dir)
+    for package, dir in LocalMap.items():
+    else:

?

@@ +226,3 @@
+        contents = contents[match.end(0):]
+        match = re.search(regex, contents, flags=re.MULTILINE)
+    prefix = os.path.split(dir)

re.sub should be able to use a function too:
https://docs.python.org/3/library/re.html#re.sub

@@ +239,3 @@
+    origdir = 'INVALID'
+
+            log(options, "Relativizing local location of $package to " + dir)

if match:

@@ +252,3 @@
+                package = match.groups(1);
+
+

if package:

@@ +257,3 @@
+            elif package in LocalMap:
+              dir = LocalMap[package]
+    log(options, "Fixing file: " + filename)

os.path.join() ?

@@ +265,3 @@
+                tmp = Mapped[origdir]
+                tmp[1] = tmp[1] + 1
+    processed = ''

>>> foo = {}
>>> foo['X'] = ["str", 1]
>>> foo['X'][1] += 1
>>> foo['X']
['str', 2]

so you should be able to write
Mapped[origdir][1] += 1

Created attachment 348898 [details] [review]
Updated patch

Updated and Perl test disabled.

Review of attachment 348898 [details] [review]:

Thanks. Only two small comments.

::: gtkdoc-rebase.in
@@ +157,2 @@
             # Remove trailing non-directory component.
+            onlinedir = re.sub(r'(.*/).*', r'\1', trial)

inline trial like below.

::: tests/gtkdoc-rebase.t
@@ +26,3 @@
 
+# Disabled until transition to Python is complete.
+#require_ok ("gtkdoc-rebase");

just delete the file and remove from the tests.

Created attachment 348945 [details] [review]
Version 3

The following fix has been pushed:
d1198e5 Converted gtkdoc-rebase from Perl to Python.

Created attachment 348950 [details] [review]
Converted gtkdoc-rebase from Perl to Python.