Opened 5 years ago

Closed 5 years ago

Last modified 4 years ago

#4375 closed defect (fixed)

Need to resetup postgis-sync

Reported by: robe Owned by: robe
Priority: blocker Milestone: Website Management, Bots
Component: management Version: master
Keywords: Cc:

Description

strk,

I still don't have a clue how this is setup but I guess it must have been running on the old debbie.

https://git.osgeo.org/gitea/postgis/repository-sync-scripts

Change History (7)

comment:1 by robe, 5 years ago

I thought it wasn't working because I had shut off the old server so I turned it back on.

Looked like it stopped worked after Martin committed his changes to svn to add himself to authors etc.

I was afraid that would happen as I recall last time sync broke was because of a disconnect with the git /svn users. — how do you fix?

comment:2 by robe, 5 years ago

okay now I remember you said you were running this on an osgeo server osgeo6? So probably wasn't running on debbie and I was just seeing an attempt.

But I did try to start doing it on new debbie, and when it was running the

make init

It errored out with:

r16147 = e7f41c9c892768197545884f397bb07bb70a68f6 (refs/remotes/svn/2.3)
        M       postgis/lwgeom_transform.c
r16148 = 17452e2e0ff7e447e5eda9889be3dee7f2460652 (refs/remotes/svn/trunk)
        M       postgis/postgis.sql.in
        M       postgis/gserialized_gist_2d.c
        M       postgis/gserialized_gist_nd.c
        M       postgis/lwgeom_transform.c
        M       postgis/gserialized_estimate.c
        M       libpgcommon/gserialized_gist.c
        M       libpgcommon/gserialized_gist.h
        M       regress/estimatedextent.sql
r16149 = ef2c4b8cd8fb6b0f01fcd21fd6ddb5859740e962 (refs/remotes/svn/trunk)
        M       postgis/lwgeom_transform.c
r16150 = f9295d9b5a679ac8a9645e71ccf018498c737e95 (refs/remotes/svn/trunk)
        M       postgis/gserialized_gist_2d.c
        M       postgis/gserialized_gist_nd.c
        M       postgis/gserialized_estimate.c
        M       libpgcommon/gserialized_gist.h
        M       NEWS
        M       regress/estimatedextent_expected
        M       regress/estimatedextent.sql
r16151 = e2a4d02baa80bfd8257530e200a8793d0898366f (refs/remotes/svn/trunk)
        M       .travis.yml
Author: komzpa not defined in /home/postgis-git-synch/repository-sync-scripts/authors.git file

make: *** [Makefile:7: init] Error 1

Which is really odd as I was expecting martin's name to come up

Last edited 5 years ago by robe (previous) (diff)

comment:3 by strk, 5 years ago

The postgis sync script is running on my own machine. It currently DOES fail for lack of martin's name in authors file. I'm fixing it now

comment:4 by strk, 5 years ago

Resolution: fixed
Status: newclosed

The script is now fixed. Somewhere on the procedure to accept a new committer we should write that a commit adding his name in authors.svn and authors.git must be performed by an existing committer before anything is committed by the new contributor.

As per the repository-sync-scripts you may want to file a ticket in its own repository, but the initial setup is pretty complex as it implies fetching data in steps so that every step has new authors (I don't remember the details but I'm pretty sure it needs update every time a new contributor is granted commit access)

comment:5 by robe, 5 years ago

okay it was already written here - https://trac.osgeo.org/postgis/wiki/DevWikiComitGuidelines

but I repeated in bold and why it's so important. But pramsey never reads instructions so I suspect we'll run into this again :)/

comment:6 by strk, 5 years ago

I wonder if we can have an SVN hook that refuses a commit if it comes from someone NOT in the authors.git and authors.svn files

comment:7 by robe, 4 years ago

Milestone: Management 2.0Website Management, Bots

Milestone renamed

Note: See TracTickets for help on using tickets.