Opened 15 years ago

Closed 15 years ago

Last modified 12 years ago

#10372 closed (fixed)

get_svn_revision() should be more robust

Reported by: rjaarts Owned by: Matt Boersma
Component: Core (Other) Version: 1.0
Severity: Keywords: version Google App Engine GAE
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description (last modified by Matt Boersma)

When Django is used in the Google App Engine (GAE)environment there is a problem with the django get_svn_revision() function. This occurs only in when using the GAE SDK and when a checked out version of Django is used.
The get_svn_revision() function checks for existence of a ".svn/entries" file. This file will exist if Django was checked out, but GAE will prevent reading that file, which results in an raised error. Would be nice if this error could be catched, and perhaps a warning logged, and simple "unknown" would be returned.

Attachments (1)

10372.diff (735 bytes ) - added by Matt Boersma 15 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 by Alex Gaynor, 15 years ago

milestone: 1.1 beta1.1

This isn't a feature and thus isn't for the 1.1 beta.

comment:2 by Jacob, 15 years ago

Triage Stage: UnreviewedAccepted

comment:3 by Matt Boersma, 15 years ago

Owner: changed from nobody to Matt Boersma
Status: newassigned

comment:4 by Matt Boersma, 15 years ago

Description: modified (diff)
Has patch: set
Summary: get_svn_version() should be more robustget_svn_revision() should be more robust

by Matt Boersma, 15 years ago

Attachment: 10372.diff added

comment:5 by Valera_Grishin, 15 years ago

There is another problem with get_svn_version. Tortoise SVN client (don't known if others do so too) allows for changing default name of directory .svn to _svn. Using this feature will cause get_svn_version to always fail, since .svn is hardcoded.

comment:6 by Malcolm Tredinnick, 15 years ago

This is a utility function that works for a normal subversion checkout. So if Tortoise SVN renames things to something non-standard, they just don't see the revision number. Same as people who aren't using subversion as their primary system for checking out the code (I never see the version number, for example). We shouldn't raise errors, but if the version number doesn't display, that's the way it goes. No harm done. If you must see the version number, don't rename the .svn directory and always use subversion to check out the code.

comment:7 by Jacob, 15 years ago

Resolution: fixed
Status: assignedclosed

(In [10377]) Fixed #10372: made get_svn_revision() more robust. Thanks, mboersma.

comment:8 by Jacob, 15 years ago

(In [10378]) [1.0.X] Fixed #10372: made get_svn_revision() more robust. Thanks, mboersma. Backport of r10377 from trunk.

comment:9 by Jacob, 12 years ago

milestone: 1.1

Milestone 1.1 deleted

Note: See TracTickets for help on using tickets.
Back to Top