bzrlib.tests.test_import_tariff.TestImportTariffs.test_simple_local fails with bzr-git installed

Bug #589683 reported by Gary van der Merwe
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Bazaar
Fix Released
Medium
Jelmer Vernooij

Bug Description

If you have bzr-git installed, and you run

bzr selftest bzrlib.tests.test_import_tariff.TestImportTariffs.test_simple_local

You get a test failure:

AssertionError: command ['st'] loaded forbidden modules ['bzrlib.foreign']

Related branches

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

bzr-git needs to be able to register entries in foreign_vcs_registry. Either bzrlib.foreign should be allowed or that registry should be moved.

affects: bzr-git → bzr
Revision history for this message
Gary van der Merwe (garyvdm) wrote : Re: [Bug 589683] Re: bzrlib.tests.test_import_tariff.TestImportTariffs.test_simple_local fails with bzr-git installed

How come bzr-git causes the failure, but bzr-svn does not?

Revision history for this message
Jelmer Vernooij (jelmer) wrote : Re: [Bug 589683] Re: bzrlib.tests.test_import_tariff.TestImportTariffs.test_simple_local fails with bzr-git installed

On Fri, 2010-06-04 at 14:19 +0000, Gary van der Merwe wrote:
> How come bzr-git causes the failure, but bzr-svn does not?
No idea, bzr-svn has the exact same import (line 67).

Cheers,

Jelmer

Revision history for this message
Martin Pool (mbp) wrote : Re: [Bug 589683] Re: bzrlib.tests.test_import_tariff.TestImportTariffs.test_simple_local fails with bzr-git installed

On 5 June 2010 00:08, Jelmer Vernooij <email address hidden> wrote:
> bzr-git needs to be able to register entries in foreign_vcs_registry.
> Either bzrlib.foreign should be allowed or that registry should be
> moved.

I agree with that. It would be ok to just remove it to support plugins.

Along the lines we recently discussed for hooks, perhaps we should
look at moving the registries that plugins need to connect to into a
smaller number of files so that the actually subsystem they register
into need not be loaded for the plugin to load.

--
Martin

Changed in bzr:
status: New → Confirmed
importance: Undecided → Medium
Jelmer Vernooij (jelmer)
tags: added: foreign selftest
Jelmer Vernooij (jelmer)
Changed in bzr:
status: Confirmed → Fix Released
assignee: nobody → Jelmer Vernooij (jelmer)
milestone: none → 2.4b2
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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