[SAC] [OSGeo] #272: enable pre-revprop-change hook

#272: enable pre-revprop-change hook
-------------------+--------------------------------------------------------
Reporter: jef | Owner: sac@lists.osgeo.org
    Type: task | Status: new
Priority: normal | Component: SAC
Keywords: |
-------------------+--------------------------------------------------------
I unsuccessfully tried to fix a wrong commit message in the QGIS
repository. Could that be enabled?

{{{
$ svn propset -r 8760 --revprop svn:log "fix #924"
svn: DAV request failed; it's possible that the repository's pre-revprop-
change hook either failed or is non-existent
svn: At least one property change failed; repository is unchanged
svn: Error setting property 'log':
Repository has not been enabled to accept revision propchanges;
ask the administrator to create a pre-revprop-change hook
}}}

--
Ticket URL: <http://trac.osgeo.org/osgeo/ticket/272&gt;
OSGeo <http://www.osgeo.org/&gt;
OSGeo committee and general foundation issue tracker.

#272: enable pre-revprop-change hook
-------------------+--------------------------------------------------------
Reporter: jef | Owner: sac@…
    Type: task | Status: new
Priority: normal | Component: Systems Admin
Keywords: |
-------------------+--------------------------------------------------------

Comment(by crschmidt):

In general, allowing all users to edit svn:log messages feels like a Bad
Idea to me. Some messages are bad, and that sucks, but allowing for the
revision of history in this manner feels dangerous to me.

I would advise against this as policy, but if you would still like me to
enable this behavior, I will do so.

(Obviously, this request is a bit out of date; if you are no longer
interested, just close the ticket.)

--
Ticket URL: <http://trac.osgeo.org/osgeo/ticket/272#comment:1&gt;
OSGeo <http://www.osgeo.org/&gt;
OSGeo committee and general foundation issue tracker.

#272: enable pre-revprop-change hook
-------------------+--------------------------------------------------------
Reporter: jef | Owner: sac@…
    Type: task | Status: new
Priority: normal | Component: Systems Admin
Keywords: |
-------------------+--------------------------------------------------------

Comment(by jef):

Replying to [comment:1 crschmidt]:
> In general, allowing all users to edit svn:log messages feels like a Bad
Idea to me. Some messages are bad, and that sucks, but allowing for the
revision of history in this manner feels dangerous to me.

Any better solution? If a log message is wrong, I'd like to fix it.

> (Obviously, this request is a bit out of date; if you are no longer
interested, just close the ticket.)

No, it's not out of date. http://trac.osgeo.org/qgis/changeset/8760
still says that it's fixing http://trac.osgeo.org/qgis/ticket/934 instead
of http://trac.osgeo.org/qgis/ticket/924. And there's currently no way to
fix that.

Without this ticket it would have take much more time to figure out what
changeset {{{r8760}}} is for.

But would TRAC notice log changes at all?

--
Ticket URL: <http://trac.osgeo.org/osgeo/ticket/272#comment:2&gt;
OSGeo <http://www.osgeo.org/&gt;
OSGeo committee and general foundation issue tracker.

#272: enable pre-revprop-change hook
---------------------------+---------------------
Reporter: jef | Owner: sac@…
     Type: task | Status: closed
Priority: normal | Milestone:
Component: Systems Admin | Resolution: moved
Keywords: |
---------------------------+---------------------
Changes (by jef):

* status: new => closed
* resolution: => moved

Comment:

qgis is using git for years.

--
Ticket URL: <https://trac.osgeo.org/osgeo/ticket/272#comment:3&gt;
OSGeo <http://www.osgeo.org/&gt;
OSGeo committee and general foundation issue tracker.