Opened 8 years ago

Closed 8 years ago

Last modified 5 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: UI/UX:

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 8 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 8 years ago by Alex Gaynor

milestone: 1.1 beta1.1

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

comment:2 Changed 8 years ago by Jacob

Triage Stage: UnreviewedAccepted

comment:3 Changed 8 years ago by Matt Boersma

Owner: changed from nobody to Matt Boersma
Status: newassigned

comment:4 Changed 8 years ago by Matt Boersma

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

Changed 8 years ago by Matt Boersma

Attachment: 10372.diff added

comment:5 Changed 8 years ago by Valera_Grishin

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 Changed 8 years ago by Malcolm Tredinnick

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 Changed 8 years ago by Jacob

Resolution: fixed
Status: assignedclosed

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

comment:8 Changed 8 years ago by Jacob

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

comment:9 Changed 5 years ago by Jacob

milestone: 1.1

Milestone 1.1 deleted

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