-
Notifications
You must be signed in to change notification settings - Fork 123
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Upgrade sbt launcher script (sbt-extras) #225
Conversation
Actually, this PR updates the sbt launcher, and tells travis to use it -- we don't have to do that, we're currently using whatever |
looks like there is some escaping bug here. Legit failure. |
what do you think about the previous comment though? this build wasn't using the ./sbt at all, it was using the pre-installed sbt |
Well, the problem with using their installed sbt is that we are at their mercy, and we are not testing how we develop locally or how our contributors might use sbt. I think it is nicer to have the only dependency be a jvm, otherwise we will have to support a lot of sbt issues for new contributors. So, I'd rather standardize on the launcher being in the state of the repo. |
yeah sgtm, i'll try to fix this one |
OK, think it's fixed. Looks like a property test found a bug though:
|
looks like it might be the * |
We should fix that test. It does fail with randomly generated unicode, which is not a good way to generate URLs. |
Upgrade sbt launcher script (sbt-extras)
I've been trying (and failing) to find a string that repros this. Copying the above example doesn't seem to work (probably something is lost in the copy) |
the odd thing is we do Try(new Url(_)), and the only other method called on this object is .toString. the error "invalid host wildcard specification" comes from here: https://github.com/openjdk-mirror/jdk/blob/jdk7/jdk7/master/src/share/classes/java/net/SocketPermission.java#L425 but I don't yet see why / how we end up there |
seems like you get there by a) having a security manager and b) calling openConnection -- I don't know why we'd be doing b, and maybe a) has to do with travis setting one up? |
turns out URL.java is pretty complicated :/ |
No description provided.