ceb53836b59c9464865d13de38d73b65b388c4e2 — Konstantin Ryabitsev 11 months ago 373c2b8
Don't force url to contain hostname

If we force the url= entry to contain the hostname, his makes it look
extra busy on the wwwlisting page.

Signed-off-by: Konstantin Ryabitsev <konstantin@linuxfoundation.org>
1 files changed, 9 insertions(+), 6 deletions(-)

M grokmirror/pi_indexer.py
M grokmirror/pi_indexer.py => grokmirror/pi_indexer.py +9 -6
@@ 102,8 102,11 @@ def init_pi_inbox(gdir: str, pdir: str, opts) -> bool:
    if origins:
        # Okay, let's process it
        # Generate a config entry
        local_host = opts.local_host.rstrip('/')
        local_url = f'{local_host}/{inboxname}/'
        if opts.local_toplevel:
            local_toplevel = opts.local_toplevel.rstrip('/')
            local_url = f'{local_toplevel}/{inboxname}'
            local_url = inboxname
        extraopts = list()
        description = None
        newsgroup = None

@@ 136,7 139,8 @@ def init_pi_inbox(gdir: str, pdir: str, opts) -> bool:
                    extraopts.append(('boost', str(boostval)))

                if opt not in {'infourl', 'listid'}:
                # Can add other options later
                if opt not in {'listid'}:
                extraopts.append((opt, val))
            except ValueError:

@@ 326,9 330,8 @@ def command():
    sp = ap.add_subparsers(help='sub-command help', dest='subcmd')
    sp_init = sp.add_parser('init', help='Run public-inbox-init+index on repositories passed via stdin')

    sp_init.add_argument('--local-hostname', dest='local_host',
                         help='URL of the local mirror toplevel')
    sp_init.add_argument('--local-toplevel', dest='local_toplevel', default='',
                         help='URL of the local mirror toplevel (omit if serving from /)')
    sp_init.add_argument('--origin-hostname', dest='origin_host',
                         help='URL of the origin toplevel serving config files')