Conversation
390d449 to
cb87d85
Compare
The name `upstream` doesn't have any meaning anymore now that there is no more fastcomp. It was removed from the SDK names in #1166 and this is the last usage of that name/term.
cb87d85 to
c610879
Compare
kripken
left a comment
There was a problem hiding this comment.
This sounds good, but I think it will surprise some users that have hardcoded paths using upstream/. We never recommended that but they might have just found where things are, and depended on it.
The best I can think of is to add a changelog to this repo, either in the readme or a separate file, and mention this there. We should also mention it in the emscripten changelog (but just mentioning it there doesn't seem like enough, since this will affect users using our standard install mechanism, which uses newest emsdk of the latest released emscripten; one option might be to wait to land this until right before an emscripten tag, but that has its own risks...)
Yeah, I was thinking the same thing.. I will hold off on this change for a bit as I know some folks were hardcoding paths to |
I think maybe we can create a hard link or a symlink from |
The name
upstreamdoesn't have any meaning anymore now that there is no more fastcomp. It was removed from the SDK names in #1166 and this is the last usage of that name/term.