Skip to content
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

[optimize/bundleContext] do not include absolute urls or windows slashes #45318

Merged
merged 3 commits into from
Sep 11, 2019

Conversation

spalger
Copy link
Contributor

@spalger spalger commented Sep 10, 2019

Fixes an issue with the optimizer, which is currently embedding absolute paths in the bundle entry files, causing them to always rebuild when run on user machines because the absolute paths change when the install is moved to a different location.

Additionally, the absolute path includes windows slashes when Kibana is built on windows.

Now all strings that are absolute paths will be converted: absolute:${relative(REPO_ROOT, path)} and have backslashes replaced with forward slashes.

@spalger spalger added Team:Operations Team label for Operations Team v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.5.0 labels Sep 10, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-operations

Copy link
Contributor

@tylersmalley tylersmalley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, tested

@elasticmachine
Copy link
Contributor

💔 Build Failed

@spalger
Copy link
Contributor Author

spalger commented Sep 11, 2019

@elasticmachine update branch

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@spalger spalger merged commit de7da0c into elastic:master Sep 11, 2019
spalger pushed a commit to spalger/kibana that referenced this pull request Sep 11, 2019
…tic#45318)

* [optimize/bundleContext] do not include absolute urls or windows slashes

* simplify the stable clone fn, it doesn't need to be generic
@spalger
Copy link
Contributor Author

spalger commented Sep 11, 2019

7.x/7.5: 3902a46

@spalger spalger deleted the fix/bad-bundle-cache-key branch September 11, 2019 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release_note:skip Skip the PR/issue when compiling release notes Team:Operations Team label for Operations Team v7.5.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants