Skip to content
This repository has been archived by the owner on Jun 21, 2023. It is now read-only.

[offline] expose packDatabase #64

Merged
merged 1 commit into from
Nov 22, 2019
Merged

[offline] expose packDatabase #64

merged 1 commit into from
Nov 22, 2019

Conversation

tobrun
Copy link
Member

@tobrun tobrun commented Nov 22, 2019

This updates to latest of gl-native master and brings in changes for mapbox/mapbox-gl-native#15899.

@tobrun tobrun added the dependencies Pull requests that update a dependency file label Nov 22, 2019
@tobrun tobrun added this to the release-tequila milestone Nov 22, 2019
@tobrun tobrun requested a review from a team November 22, 2019 08:27
@tobrun tobrun self-assigned this Nov 22, 2019
@chloekraw
Copy link
Contributor

hmm @tobrun why is doing this important now as opposed to after we add the other API that decouples ambient cache clearing from VACUUM?

@tobrun
Copy link
Member Author

tobrun commented Nov 22, 2019

No reason @chloekraw , we need to periodically update gl-native submodule and pull in all required changes from time to time. I was already seeing 90 commits since last update. Expect such PRs to land on a weekly basis. The longer we wait, the higher chance we see breaking changes (changes that require to align the code here vs code in gl-native).

Copy link

@LukasPaczos LukasPaczos left a comment

Choose a reason for hiding this comment

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

👍 to bump. Just a note that packDatabase has already been brought in a couple of days ago in dfdbaf5.

@tobrun
Copy link
Member Author

tobrun commented Nov 22, 2019

90 commits since last update

+1 to bump. Just a note that packDatabase has already been brought in a couple of days ago in dfdbaf5.

I think, I might have had an stale master 😅

@tobrun tobrun merged commit 72c28e4 into master Nov 22, 2019
@tobrun tobrun deleted the tvn-sync-master branch November 22, 2019 11:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants