Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

upload.py does not support replaced files #208

Closed
GoogleCodeExporter opened this issue Mar 28, 2015 · 10 comments
Closed

upload.py does not support replaced files #208

GoogleCodeExporter opened this issue Mar 28, 2015 · 10 comments

Comments

@GoogleCodeExporter
Copy link

@GoogleCodeExporter GoogleCodeExporter commented Mar 28, 2015

What steps will reproduce the problem?
1. svn mv oldfile newfile
2. touch oldfile
3. svn add oldfile
4. upload.py

What is the expected output? What do you see instead?

Got error status from ['svn', 'cat', 'oldfile']

What version of the product are you using? On what operating system?

Latest upload.py on OSX 10.6

*** If you are a Google employee please say so or mail me (Guido van
Rossum) directly. ***


Please provide any additional information below.

Original issue reported on code.google.com by alexande...@gmail.com on 14 Jul 2010 at 1:20

@GoogleCodeExporter
Copy link
Author

@GoogleCodeExporter GoogleCodeExporter commented Mar 28, 2015

http://codereview.appspot.com/1801042

The error is caused by a missing local history (base file) for oldfile. Even 
with the fix uploaded to codereview applied, there's still a strange behavior 
of SVN. In the scenario described above "svn st" reports "newfile" added with 
modification ("A +") and "oldfile" replaced ("R"), while "svn diff" only lists 
"oldfile". But that seems to be a bug in svn, IMO.

Original comment by albrecht.andi on 14 Jul 2010 at 7:30

  • Changed state: Accepted
  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

@GoogleCodeExporter GoogleCodeExporter commented Mar 28, 2015

This issue was closed by revision r538.

Original comment by albrecht.andi on 15 Jul 2010 at 5:35

  • Changed state: Fixed
  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

@GoogleCodeExporter GoogleCodeExporter commented Mar 28, 2015

I've downloaded the r538 version of upload.py and it did not fix my issue.  The 
error message changed, though.  It is now

Got error status from ['svn', 'cat', 
'http://svn.python.org/view/*checkout*/python/branches/py3k//Lib/test/test_datet
ime.py']

$ svn st Lib/test/test_datetime.py
R       Lib/test/test_datetime.py

Original comment by alexande...@gmail.com on 15 Jul 2010 at 6:12

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

@GoogleCodeExporter GoogleCodeExporter commented Mar 28, 2015

It looks like you've hit another bug :)

In detail: The base URL detected/guessed by upload.py in 
SubversionVCS._GuessBase() returns an URL suitable for accessing base files via 
HTTP requests, but not suitable as a SVN path.

I'm reoping this issue to keep track of it.

Original comment by albrecht.andi on 15 Jul 2010 at 6:29

  • Changed state: Accepted
  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

@GoogleCodeExporter GoogleCodeExporter commented Mar 28, 2015

Original comment by techtonik@gmail.com on 27 Sep 2010 at 12:15

  • Added labels: Priority-High
  • Removed labels: Priority-Medium

@GoogleCodeExporter
Copy link
Author

@GoogleCodeExporter GoogleCodeExporter commented Mar 28, 2015

I run into the same problem like alexander.belopolsky.

Original comment by leei...@gmail.com on 28 Dec 2010 at 5:47

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

@GoogleCodeExporter GoogleCodeExporter commented Mar 28, 2015

The problem described in original user story is fixed. Test now passes. I'd 
like to close this issue now.

May I ask you to add your findings as a separate bug reports with similar test 
cases, so it will be easier to pick them later.

Andi, is it an issue #82 with SVN bug you mentioned?

Original comment by techtonik@gmail.com on 29 Mar 2011 at 8:01

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

@GoogleCodeExporter GoogleCodeExporter commented Mar 28, 2015

The issue described in comment 4 looks much like issue82. But to go for sure we 
need a reproducable test.

Original comment by albrecht.andi on 30 Mar 2011 at 3:53

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

@GoogleCodeExporter GoogleCodeExporter commented Mar 28, 2015

Original comment by albrecht.andi on 6 Apr 2012 at 6:14

  • Added labels: Component-Scripts
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

@GoogleCodeExporter GoogleCodeExporter commented Mar 28, 2015

Original comment by jrobbins@google.com on 12 Jul 2014 at 5:37

  • Changed state: Obsolete
  • Added labels: ****
  • Removed labels: ****

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant