Using juju-deployer with local charms

[Kapil Thangavelu]

looks reasonable, i'll add it upstream.
thanks,
kapil


On Fri, Aug 2, 2013 at 11:14 AM, Marco Ceppi <marco.ceppi at canonical.com>wrote:

> Thanks for the tips. I ended up doing the following changes to allow for
> local charm paths:
>
> === modified file 'deployer/charm.py'
> --- deployer/charm.py 2013-07-23 19:39:11 +0000
> +++ deployer/charm.py 2013-08-02 15:13:12 +0000
> @@ -33,9 +33,11 @@
>      def get_vcs(self):
>          if not self.branch:
>              return None
> -        if self.branch.startswith('git') or 'github.com' in self.branch:
> +        if self.branch.startswith('git') or 'github.com' in self.branch
> or \
> +                os.path.exists(os.path.join(self.branch, '.git')):
>              return Git(self.path, self.branch, self.log)
> -        elif self.branch.startswith("bzr") or
> self.branch.startswith('lp:'):
> +        elif self.branch.startswith("bzr") or
> self.branch.startswith('lp:') \
> +                or os.path.exists(os.path.join(self.branch, '.bzr')):
>              return Bzr(self.path, self.branch, self.log)
>
>      @classmethod
>
>
>
> On Fri, Aug 2, 2013 at 10:09 AM, Andreas Hasenack <andreas at canonical.com>wrote:
>
>> On Fri, Aug 2, 2013 at 11:08 AM, Matthew Wedgwood <
>> matthew.wedgwood at canonical.com> wrote:
>>
>>> On 07/31/2013 09:40 PM, Marco Ceppi wrote:
>>> > Is it possibly to point the branch attribute of a service to a local
>>> bzr path? I've attempted the following
>>>
>>> IIRC, you can check out the local charm to <series>/<directory> in the
>>> current (working) directory and deployer will look for it there. I don't
>>> think deployer will check out local repositories.
>>>
>>
>> That's correct, and it's the trick I have been using.
>>
>
>
> --
> Juju mailing list
> Juju at lists.ubuntu.com
> Modify settings or unsubscribe at:
> https://lists.ubuntu.com/mailman/listinfo/juju
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.ubuntu.com/archives/juju/attachments/20130802/3154dac3/attachment.html>