Opened 6 years ago

Closed 3 years ago

#1911 closed task (wontfix)

Enable trac explicit sync for metacrs subprojects

Reported by: strk Owned by: strk
Priority: normal Milestone:
Component: Trac Keywords:
Cc: rouault

Description

post-commit SVN hook for the metacrs project only signals changes to the metacrs trac instance, no matter which subproject is being committed to.

This ticket is to make the post-commit hook smarter to distinguish which project has to be notified.

This is a spin-off of ticket #1910

Change History (8)

comment:1 by strk, 6 years ago

Cc: rouault added

Metacrs subprojects are:

  • geotiff
  • csmap
  • proj4j
  • proj4js

I've tweaked the post-commit hook with this code (now under a local git repo):

MOD=`svnlook changed "${REPOS}" -r "${REV}" | head -1 | sed 's@.* \([^/]*\)/.*@\1@'`
if test "${MOD}" = "csmap" -o "${MOD} = "geotiff" -o "${MOD}" = "proj4j" -o "${MOD}" = "proj4js"; then
        TRAC_ENV="${BASE_ENV}/${MOD}/"
else
        TRAC_ENV="${BASE_ENV}/metacrs/"
fi

And disabled the per-request sync again.

Even, could you try again sending a dumb commit to geotiff please (and if it works to any of the others) ?

NOTE: if things go as I expect, the MetaCRS trac instance should also get *less* revisions notified too (not sure if this is something we want though)

comment:2 by rouault, 6 years ago

There's a syntax issue in the hook :

$ svn commit -m "Dummy commit"
Envoi          README
Transmission des données .done
Transaction de propagation...
Révision 2764 propagée.

Attention : post-commit hook failed (exit code 2) with output:
/var/www/svn/repos/metacrs/hooks/post-commit: 22: Syntax error: Unterminated quoted string

comment:3 by strk, 6 years ago

Can you please try again ?

comment:4 by rouault, 6 years ago

Yep, works that time and https://trac.osgeo.org/geotiff/timeline is properly refreshed.

I'm not confortable with committing in other projects not being an approved committer of them. I've dropped a message to metacrs so that their committers do it : https://lists.osgeo.org/pipermail/metacrs/2017-April/000954.html

comment:5 by strk, 6 years ago

Thanks, since you're at it could you also ask what they want to do with the "metacrs" trac instance itself ? Do they want to see *all* commits there (as it has been so far) or is it ok to only show commits that are NOT part of any of the known submodules ?

comment:6 by strk, 6 years ago

Note how r2764 and r2765 are NOT shown in https://trac.osgeo.org/metacrs/timeline ...

Version 0, edited 6 years ago by strk (next)

comment:7 by rouault, 6 years ago

Nobody cares about metacrs itself. It is just ... too meta ;-)

comment:8 by robe, 3 years ago

Resolution: wontfix
Status: newclosed

I think this ticket is moot as I think most of the projects listed here have moved off trac and to github. So closing.

Note: See TracTickets for help on using tickets.