On Tue, 2013-12-17 at 19:41 -0800, W. Trevor King wrote:
> On Tue, Dec 17, 2013 at 05:12:09PM -0800, Brian Dolbec wrote:
> > -				self.settings["snapshot"]+"/")
> > +				self.settings["snapshot"])
> > …
> > -			"/tmp/"+self.settings["target_subpath"]+"/")
> > +			"/tmp/"+self.settings["target_subpath"])
> > …
> > -			self.settings["version_stamp"]+"/")
> > +			self.settings["version_stamp"])
> 
> Acked by me, although I think path consumers should be able to handle
> both forms.
> 
> Cheers,
> Trevor
> 

They should, but I found catalyst's code to be fragile, fixing these
issues got things working again.  I also found that once fixed, things
seemed to be more stable when making changes.

Plus I like it better when looking through output to see it not have //
or /// in the middle of a path.