It used to be at the end, until it was switched in  FS#49201 . But that's been there since the first version.

Well, the current lib/support/init.d/gitlab.default.example contains just this:

gitlab_workhorse_options="-listenUmask 0 -listenNetwork unix -listenAddr $socket_path/gitlab-workhorse.socket -authBackend http://127.0.0.1:8080 -authSocket $socket_path/gitlab.socket -documentRoot $app_root/public"

Also note that I found it partly because I added "-logFile /var/lib/gitlab/log/workhorse.log" at the end of ExecStart and it didn't work, but it worked when I added it at the beginning. Maybe the command line parser got confused after all...

Uh, so do I close this or what? If we go by this line: https://gitlab.com/gitlab-org/gitlab-ce/blob/master/lib/support/init.d/gitlab#L43 then the service file is correct.

I still don't see where the /var/lib/gitlab/repositories path comes from - nothing similar is used anywhere in that init script.

If I followed the right link, upstream has these options:
-listenUmask 0 -listenNetwork unix -listenAddr $socket_path/gitlab-workhorse.socket -authBackend http://127.0.0.1:8080 -authSocket $rails_socket -documentRoot $app_root/public"

Arch service file:
-listenUmask 0 -authBackend http://localhost:8080 -authSocket /var/lib/gitlab/sockets/gitlab.socket -listenAddr /run/gitlab/gitlab-workhorse.socket -listenNetwork unix /var/lib/gitlab/repositories -documentRoot /usr/share/webapps/gitlab/public

All options except /var/lib/gitlab/repositories nicely match each other.

Ok, the line should be synced now.

Good, so feel free to close this when the new package is released.