search

cloudfoundry / buildpack-packager

Buildpack Packager
Apache License 2.0
23 stars 31 forks source link

Too many slashes in file URI file://// #10

Closed opiethehokie closed 8 years ago

opiethehokie commented 8 years ago

I think the sub("file:", "file://") in this change may not be needed. Unless I'm doing something wrong in my manifest.yml where I have a file URI like file://filename, it's changing it to file:////filename.

The cached file ends up with extra underscores in the name and then when the the buildpack is executing the compile-extensions download_dependency script can't find it in the cache.

cf-gitbot commented 8 years ago

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/130127105

The labels on this github issue will be updated when the story is started.

sesmith177 commented 8 years ago

Hello @opiethehokie,

The format for a file URI is file://localhost/path or, as a shortcut, file:///path. Note the triple slash.

It's confusing because the file URI format is trying to fit the general URI format, including hostnames.

We can't change the URI format, so unfortunately closing this issue.