Opened 10 years ago

Closed 8 years ago

#41458 closed defect (fixed)

Intermittently not able to edit svn:log property

Reported by: ryandesign (Ryan Carsten Schmidt) Owned by: ryandesign (Ryan Carsten Schmidt)
Priority: Normal Milestone:
Component: server/hosting Version:
Keywords: Cc:
Port:

Description

I used to be able to edit the svn:log property of committed revisions (using e.g. svn propedit svn:log --revprop -r 113590), but although I can still do that without error, the property does not end up getting changed, and the emails that get sent to the macports-changes mailing list show the change in the reverse order. For example, I edited this revision to add more detail, and the email said:

From: ryandesign@macports.org
Subject: propchange - r113590 svn:log
Date: November 19, 2013 at 22:30:43 CST
To: macports-changes@lists.macosforge.org
Reply-To: macports-dev@lists.macosforge.org, ryandesign@macports.org

Author: ryandesign@macports.org (original author: ryandesign@macports.org)
Revision: 113590
Property Name: svn:log

--- /var/tmp/revprops/pre-113590-ryandesign@macports.org	2013-11-19 20:30:43.564470909 -0800
+++ /var/tmp/revprops/post-113590-ryandesign@macports.org	2013-11-19 20:30:43.636357129 -0800
@@ -1 +1 @@
-bro: update to 2.2 (#41308, maintainer timeout), which fixes the build with clang 3.3+; add dependencies on libraries that do get used, like python27 (#39971)
+bro: update to 2.2 (#41308, maintainer timeout), which fixes the build with clang 3.3+; add dependencies on libraries that do get used

Later I wanted to add more detail, and the email said:

From: ryandesign@macports.org
Subject: propchange - r113590 svn:log
Date: November 19, 2013 at 22:39:38 CST
To: macports-changes@lists.macosforge.org
Reply-To: macports-dev@lists.macosforge.org, ryandesign@macports.org

Author: ryandesign@macports.org (original author: ryandesign@macports.org)
Revision: 113590
Property Name: svn:log

--- /var/tmp/revprops/pre-113590-ryandesign@macports.org	2013-11-19 20:39:38.370377437 -0800
+++ /var/tmp/revprops/post-113590-ryandesign@macports.org	2013-11-19 20:39:38.424594340 -0800
@@ -1 +1 @@
-bro: update to 2.2 (#41308, maintainer timeout), which fixes the build with clang 3.3+; add dependencies on libraries that do get used, like python27 (#39971), zlib, and libgeoip, which fixes the build on the buildbots (#39973)
+bro: update to 2.2 (#41308, maintainer timeout), which fixes the build with clang 3.3+; add dependencies on libraries that do get used

Change History (5)

comment:1 Changed 10 years ago by wsiegrist@…

Owner: changed from wsiegrist@… to ryandesign@…

Cannot reproduce. You can see that I successfully changed it to the message you wanted at https://lists.macosforge.org/pipermail/macports-changes/2013-November/098237.html

What steps are you performing to try to edit the log message? I used svn propedit ... with svn version 1.7.10.

comment:2 Changed 10 years ago by ryandesign (Ryan Carsten Schmidt)

I can't reproduce it now either.

$ /opt/local/bin/svn --version
svn, version 1.8.4 (r1534716)
   compiled Nov  1 2013, 23:07:21 on x86_64-apple-darwin13.0.0

Copyright (C) 2013 The Apache Software Foundation.
This software consists of contributions made by many people;
see the NOTICE file for more information.
Subversion is open source software, see http://subversion.apache.org/

The following repository access (RA) modules are available:

* ra_svn : Module for accessing a repository using the svn network protocol.
  - with Cyrus SASL authentication
  - handles 'svn' scheme
* ra_local : Module for accessing a repository on local disk.
  - handles 'file' scheme
* ra_serf : Module for accessing a repository via WebDAV protocol using serf.
  - using serf 1.3.2
  - handles 'http' scheme
  - handles 'https' scheme

$ /opt/local/bin/svn propedit svn:log --revprop -r 113504 https://svn.macports.org/repository/macports
Set new value for property 'svn:log' on revision 113504

(ok)

I'll keep an eye on it.

comment:3 Changed 10 years ago by ryandesign (Ryan Carsten Schmidt)

Resolution: worksforme
Status: newclosed

comment:4 Changed 10 years ago by ryandesign (Ryan Carsten Schmidt)

Resolution: worksforme
Status: closedreopened
Summary: Not able to edit svn:log propertyIntermittently not able to edit svn:log property

It happened again: https://lists.macosforge.org/pipermail/macports-changes/2013-December/099043.html

But this time, I got an error in the terminal. My transcript:

$ cat $(which svnlogedit)
#!/bin/bash
# $Id: svnlogedit 38 2008-07-15 23:05:04Z rschmidt $

svn propedit --revprop -r $1 svn:log $2
$ echo $M
https://svn.macports.org/repository/macports
$ svnlogedit 114384
svn: E175002: DAV request failed; it's possible that the repository's pre-revprop-change hook either failed or is non-existent
svn: E175002: Could not execute PROPPATCH.
$ svn log -r 114384 $M
------------------------------------------------------------------------
r114384 | devans@macports.org | 2013-12-06 17:14:44 -0600 (Fri, 06 Dec 2013) | 2 lines

VLC, VLC-devel: apply upstream patch to fix build with freetype 2.5.1.

------------------------------------------------------------------------
$

But trying a second time worked: https://lists.macosforge.org/pipermail/macports-changes/2013-December/099044.html

$ svnlogedit 114384
Set new value for property 'svn:log' on revision 114384
$ svn log -r 114384 $M
------------------------------------------------------------------------
r114384 | devans@macports.org | 2013-12-06 17:14:44 -0600 (Fri, 06 Dec 2013) | 2 lines

VLC, VLC-devel: apply upstream patch to fix build with freetype 2.5.1 (#41640)

------------------------------------------------------------------------
$

comment:5 Changed 8 years ago by ryandesign (Ryan Carsten Schmidt)

Resolution: fixed
Status: reopenedclosed

Let's assume whatever this was fixed itself somewhere along the way, since I haven't seen it again.

Note: See TracTickets for help on using tickets.