Merge lp:~jjo/juju-deployer/support-JUJU_REPOSITORY-env_var into lp:juju-deployer

[JuanJo Ciarlante]

support JUJU_REPOSITORY environment variable (to avoid having to chdir to it for juju-deployer usage), fixes lp#1229390

[JuanJo Ciarlante (jjo)]

support JUJU_REPOSITORY environment variable (to avoid having to chdir to it for juju-deployer usage), fixes lp#1229390

[Andy Doan (doanac)]

Any reason this MP hasn't gotten accepted yet? This bug is causing some problems for my team.

[Kapil Thangavelu (hazmat)]

merged a variant of this which configures this a bit more centrally in the
config stack when it constructs deployments to use the env var if defined.
The deployments already supported a repository path variable just not
defined outside of test previously.

On Wed, Jan 22, 2014 at 11:04 PM, Andy Doan <email address hidden>wrote:

> Any reason this MP hasn't gotten accepted yet? This bug is causing some
> problems for my team.
> --
>
> https://code.launchpad.net/~jjo/juju-deployer/support-JUJU_REPOSITORY-env_var/+merge/196984
> Your team juju-deployers is requested to review the proposed merge of
> lp:~jjo/juju-deployer/support-JUJU_REPOSITORY-env_var into lp:juju-deployer.
>

=== modified file 'deployer/charm.py'
--- deployer/charm.py	2013-08-21 07:29:55 +0000
+++ deployer/charm.py	2013-11-27 21:27:54 +0000
@@ -52,7 +52,8 @@
         if name.startswith('cs:'):
             store_url = name
         else:
-            charm_path = path_join(series_path, name)
+            charm_path = path_join(os.environ.get('JUJU_REPOSITORY', '.'),
+                series_path, name)
             build = d.get('build', '')
         if not store_url:
             store_url = d.get('charm_url', None)