made requested change and pushed it to my fork.

On Jan 22, 2013, at 10:35 AM, swarbhanu <notifications@github.com> wrote:

> In ion/agents/instrument/driver_process.py:
> 
> > +        log.debug("_egg_path")
> > +        if not CACHE_DIR:
> > +            raise ServerError("CACHE_DIR is %s'%s'" % (type(CACHE_DIR), CACHE_DIR))
> > +        elif not egg_name:
> > +            raise ServerError("egg_name for path is %s'%s'" % (type(egg_name), egg_name))
> > +
> > +        return CACHE_DIR + "/" + egg_name
> > +
> > +    def _egg_remotepath(self, egg_name):
> > +        log.debug("_egg_remotepath" + str(egg_name))
> > +        if not REPO_BASE:
> > +            raise ServerError("REPO_BASE is %s'%s'" % (type(REPO_BASE), REPO_BASE))
> > +        elif not egg_name:
> > +            raise ServerError("egg_name for remotepath is %s'%s'" % (type(egg_name), egg_name))
> > +
> > +        return REPO_BASE + "/" + egg_name
> Since, REPO_BASE = 'http://sddevrepo.oceanobservatories.org/releases/' at the beginning of the file, line 369 will result in two '/' before the egg_name. Should we not have the REPO_BASE = 'http://sddevrepo.oceanobservatories.org/releases' ? Thanks.
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>