Re: svn commit: r1396609 - /subversion/trunk/tools/hook-scripts/persist-ephemeral-txnprops.py

[Daniel Shahaf <d....@daniel.shahaf.name>]

cmpilato@apache.org wrote on Wed, Oct 10, 2012 at 14:06:33 -0000:
> Author: cmpilato
> Date: Wed Oct 10 14:06:32 2012
> New Revision: 1396609
> 
> URL: http://svn.apache.org/viewvc?rev=1396609&view=rev
> Log:
> * tools/hook-scripts/persist-ephemeral-txnprops.py
>   New pre-commit hook script which copies ephemeral transaction
>   properties to new names (so they outlive the transaction).
> 
> Added:
>     subversion/trunk/tools/hook-scripts/persist-ephemeral-txnprops.py   (with props)
> 
> Added: subversion/trunk/tools/hook-scripts/persist-ephemeral-txnprops.py
> URL: http://svn.apache.org/viewvc/subversion/trunk/tools/hook-scripts/persist-ephemeral-txnprops.py?rev=1396609&view=auto
> ==============================================================================
> --- subversion/trunk/tools/hook-scripts/persist-ephemeral-txnprops.py (added)
> +++ subversion/trunk/tools/hook-scripts/persist-ephemeral-txnprops.py Wed Oct 10 14:06:32 2012
> @@ -0,0 +1,81 @@
> +#!/usr/bin/env python
> +#
> +#
> +# Licensed to the Apache Software Foundation (ASF) under one
> +# or more contributor license agreements.  See the NOTICE file
> +# distributed with this work for additional information
> +# regarding copyright ownership.  The ASF licenses this file
> +# to you under the Apache License, Version 2.0 (the
> +# "License"); you may not use this file except in compliance
> +# with the License.  You may obtain a copy of the License at
> +#
> +#   http://www.apache.org/licenses/LICENSE-2.0
> +#
> +# Unless required by applicable law or agreed to in writing,
> +# software distributed under the License is distributed on an
> +# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
> +# KIND, either express or implied.  See the License for the
> +# specific language governing permissions and limitations
> +# under the License.
> +#
> +#
> +
> +"""\
> +Usage:
> +
> +   persist-ephemeral-txnprops.py REPOS_PATH TXN_NAME [PREFIX]
> +
> +Duplicate ephemeral transaction properties so that the information
> +they carry may persist as properties of the revision created once the
> +transaction is committed.  This is intended to be used as a Subversion
> +pre-commit hook script.
> +
> +REPOS_PATH is the on-disk path of the repository whose transaction
> +properties are being examined/modified.  TXN_NAME is the name of the
> +transaction.
> +
> +By default, ephemeral transaction properties will be copied to new
> +properties whose names lack the "svn:" prefix.  "svn:txn-user-agent"
> +will, then, persist as "txn-user-agent".  If, however, the optional
> +PREFIX string argument is provided, it will be prepended to the base
> +property name in place of the "svn:" namespace.  For example, a prefix
> +of "acme" will cause the "svn:txn-user-agent" property to be copied
> +to "acme:txn-user-agent".

>From the peanut gallery, but... maybe just copy it to
svn:revision-user-agent, instead of forcing everyone to invent a prefix
--- and thus requiring every tool working with these to have another
argument (the local installation's value of "acme")?

Re: svn commit: r1396609 - /subversion/trunk/tools/hook-scripts/persist-ephemeral-txnprops.py

["C. Michael Pilato" <cm...@collab.net>]

On 10/10/2012 03:31 PM, Daniel Shahaf wrote:
> From the peanut gallery, but... maybe just copy it to
> svn:revision-user-agent, instead of forcing everyone to invent a prefix
> --- and thus requiring every tool working with these to have another
> argument (the local installation's value of "acme")?

Yep.  Makes sense.  Watch commits@.

-- 
C. Michael Pilato <cm...@collab.net>
CollabNet   <>   www.collab.net   <>   Enterprise Cloud Development