dpush revisions make unnecessary change to branch root

Bug #882705 reported by Jelmer Vernooij
22
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Bazaar Subversion Plugin
Triaged
High
Unassigned

Bug Description

Each dpush-ed revision seems to contain an additional (empty?) change to the branch root directory, e.g. /trunk, showing up in "svn log -v".

Jelmer Vernooij (jelmer)
Changed in bzr-svn:
status: New → Triaged
importance: Undecided → High
Revision history for this message
Tom Prince (tom.prince) wrote :

I encoutered this. bzr-svn seems to be generating props ven with dpush:

  bzr:revision-info
  bzr:ancestry:v3-trunk0
  bzr:file-ids
  bzr:revision-id:v3-trunk0

Revision history for this message
Jelmer Vernooij (jelmer) wrote :

Hi Tom,

That's a separate issue, please file a separate bug about it.

Revision history for this message
Tom Prince (tom.prince) wrote :

As far as I can se, it is the same issue. I get 'svn log -v' showing a change to the bracnh root, but with no --diff changes to it.
But the files do have the above listed bzr properties.

$ svn log -v -r36797:HEAD svn+ssh://svn.twistedmatrix.com/svn/Twisted/branches/http-getBody-6251
------------------------------------------------------------------------
r36803 | tomprince | 2013-01-12 19:07:45 -0700 (Sat, 12 Jan 2013) | 2 lines
Changed paths:
   M /branches/http-getBody-6251
   M /branches/http-getBody-6251/twisted/web/test/test_agent.py

Add a test to handle the case where a connection is closed uncleanly.

------------------------------------------------------------------------
r36802 | tomprince | 2013-01-12 19:07:36 -0700 (Sat, 12 Jan 2013) | 2 lines
Changed paths:
   M /branches/http-getBody-6251
   M /branches/http-getBody-6251/twisted/web/client.py
   M /branches/http-getBody-6251/twisted/web/test/test_agent.py

Handle HTTP/1.0 case of unknown length.

------------------------------------------------------------------------
r36801 | tomprince | 2013-01-12 19:07:25 -0700 (Sat, 12 Jan 2013) | 2 lines
Changed paths:
   M /branches/http-getBody-6251
   M /branches/http-getBody-6251/doc/web/howto/client.xhtml
   A /branches/http-getBody-6251/doc/web/howto/listings/client/responseBody.py
   M /branches/http-getBody-6251/twisted/web/test/test_agent.py

Add the simplest test and some documentation.

------------------------------------------------------------------------
r36800 | tomprince | 2013-01-12 19:07:14 -0700 (Sat, 12 Jan 2013) | 4 lines
Changed paths:
   M /branches/http-getBody-6251
   M /branches/http-getBody-6251/twisted/web/client.py
   M /branches/http-getBody-6251/twisted/web/test/test_agent.py

Move getBody from t.w.test.test_agent to t.w.client.

It existed but was unused. Move it to somewhere public.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Related questions

Remote bug watches

Bug watches keep track of this bug in other bug trackers.