[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

[PATCH] externals_tests.py should handle missing local externals item

From: Daniel Näslund <daniel_at_longitudo.com>
Date: Mon, 7 Dec 2009 08:45:06 +0100

Hi!

[[[
When updating to loose an external the code should handle a locally
missing externals item.

* subversion/tests/cmdline/externals_tests.py
  (update_loose_external): Remove one externals wc dir.
]]]

I'm writing a patch to remove some adm_access batons. That patch made
the externals_tests even though it didn't handle missing local
externals which trunk does.

Why should we handle a locally missing externals here? The user might
have thought that the externals items should be removed manually after
beeing removed from the property. Users do strange things. Handle it
gracefully!

I thought the change was so small that it could be included
in an existing test case.

Daniel

Received on 2009-12-07 08:45:43 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.