Radeonhd repo not migrated to gitlab.

[Daniel Stone]

Hi Kevin,

On Wed, 13 Mar 2019 at 16:00, Kevin Brace <kevinbrace at gmx.com> wrote:
> I am not here to side with either one of you (i.e., Luc or you), but I have been wondering why some of the older, neglected (I use the word "underserved" to describe it) DDXs in general have weird git and ssh clone addresses.
> For example, Number Nine Imagine 128 DDX.
>
> [...]
>
> Why do I see this xorg/xserver-test for xorg/driver/xf86-video-i128?
>     Another example will be S3 ViRGE DDX.
>
> [...]
>
> This time I see xorg/driver/xf86-video-p690.
>     I probably should not get into the conspiracy theory territory, but for some of the underserved DDXs some months ago, the git and ssh cloning addresses were definitely pointing to xorg/driver/xf86-video-radeonhd.
> It was about roughly when you were working on GitLab migration.
> Considering that the history between you and Luc, is there a good explanation for why the cloning addresses for git and ssh were pointing to xorg/driver/xf86-video-radeonhd back then?
> I have been wondering if this was some kind of a prank considering the history of xf86-video-radeonhd.
> I do not have a personal stake in what happened between you and Luc some years ago, but if you can try to correct the wrong git and ssh cloning address issue, that will be appreciated.

Sure, I've fixed it now, though it might take a few minutes to finish updating.

In the shell script we use to generate cgitrc, we were looping over
all the repositories and either printing a GitLab clone URL if the
tree was mirrored from GitLab to cgit, or printing anongit clone URLs
if not. In the former case, we were forgetting to clear the variables
with the anongit URLs, so GitLab repos would show the anongit clone
URLs of the last non-GitLab repo found (if any).

Hence xf86-video-radeonhd appearing in some drivers (as it was
inadvertently missed until this thread), xserver-test appearing in
others (as it was deliberately not migrated), and xf86-video-p690 (not
migrated as it's empty).

I've attached the relevant shell snippet in order to hopefully calm
the conspiracy theories.

In future, if you notice anything like this, please file an issue on
https://gitlab.freedesktop.org/freedesktop/freedesktop/issues/new/ and
we'll try to fix it.

Cheers,
Daniel


                gitlab_mirror=$(test -d /git/${sh_wgit} && pushd
/git/${sh_wgit} >/dev/null && git config --local fdo.gitlab-mirror ||
true)
                if [ -n "$gitlab_mirror" ]; then
                    httpsurl="https://gitlab.freedesktop.org/$gitlab_mirror"
                    sshurl="" # XXX this was missing
                    giturl="" # XXX this was missing
                    httpurl="" # XXX this was missing
                    echo "repo.desc=$desc (mirrored from $httpsurl)"
                else
                    httpsurl=""
                    echo repo.desc=$desc
                    if [ "${url#\~}" != "$url" ]; then
                        sshurl="ssh://people.freedesktop.org/$url"
                        giturl="git://people.freedesktop.org/$url"
                    else
                        sshurl="ssh://git.freedesktop.org/git/$url"
                        giturl="git://anongit.freedesktop.org/$url"
                    fi
                fi

                echo "repo.clone-url=$httpsurl $giturl $sshurl $httpurl"