Remove sqlite dependency#30208
Merged
Merged
Conversation
justincormack
force-pushed
the
no-sqlite
branch
2 times, most recently
from
January 17, 2017 13:13
32aba63 to
a0806aa
Compare
Signed-off-by: Alexander Morozov <[email protected]>
justincormack
force-pushed
the
no-sqlite
branch
from
January 17, 2017 13:53
a0806aa to
95ae788
Compare
Member
|
SGTM. |
Contributor
|
Maybe we should add a warning to logs if old links DB exists. It doesn't require SQLite and small enough I think. |
Member
|
Well, we kept the db around in case ppl wanted to rollback. |
tophj-ibm
reviewed
Jan 17, 2017
Contributor
There was a problem hiding this comment.
don't forget to remove it from all the other builders we have now too.
Contributor
Author
There was a problem hiding this comment.
ah, I thought I had but you are right it didn't actually generate them as I was on amd64...
justincormack
force-pushed
the
no-sqlite
branch
2 times, most recently
from
January 17, 2017 17:23
efb4f88 to
fd540de
Compare
justincormack
force-pushed
the
no-sqlite
branch
from
January 17, 2017 17:39
fd540de to
1a65d21
Compare
This drops support for migrations from pre-1.10 Docker versions, which should be done via an external tool or an intermediate upgrade. Signed-off-by: Justin Cormack <[email protected]>
justincormack
force-pushed
the
no-sqlite
branch
from
January 18, 2017 12:27
1a65d21 to
f8119bb
Compare
Contributor
Author
|
re-pushed to unstick CI |
Contributor
|
Ok, design looks good to me |
paralin
pushed a commit
to skiffos/buildroot
that referenced
this pull request
Feb 21, 2022
Dependency on sqlite has been removed since v17.04.0-ce. See: moby/moby#30208 Signed-off-by: TIAN Yuanhao <[email protected]> Reviewed-by: Christian Stewart <[email protected]>
paralin
pushed a commit
to skiffos/buildroot
that referenced
this pull request
Feb 21, 2022
Dependency on sqlite has been removed since v17.04.0-ce. See: moby/moby#30208 Signed-off-by: TIAN Yuanhao <[email protected]> Reviewed-by: Christian Stewart <[email protected]>
paralin
pushed a commit
to skiffos/buildroot
that referenced
this pull request
Feb 21, 2022
Dependency on sqlite has been removed since v17.04.0-ce. See: moby/moby#30208 Signed-off-by: TIAN Yuanhao <[email protected]> Reviewed-by: Christian Stewart <[email protected]>
paralin
pushed a commit
to skiffos/buildroot
that referenced
this pull request
Feb 22, 2022
Dependency on sqlite has been removed since v17.04.0-ce. See: moby/moby#30208 Signed-off-by: TIAN Yuanhao <[email protected]> Reviewed-by: Christian Stewart <[email protected]>
paralin
pushed a commit
to skiffos/buildroot
that referenced
this pull request
Feb 22, 2022
Dependency on sqlite has been removed since v17.04.0-ce. See: moby/moby#30208 Signed-off-by: TIAN Yuanhao <[email protected]> Reviewed-by: Christian Stewart <[email protected]>
paralin
pushed a commit
to skiffos/buildroot
that referenced
this pull request
Mar 1, 2022
Dependency on sqlite has been removed since v17.04.0-ce. See: moby/moby#30208 Signed-off-by: TIAN Yuanhao <[email protected]> Reviewed-by: Christian Stewart <[email protected]>
paralin
pushed a commit
to skiffos/buildroot
that referenced
this pull request
Mar 1, 2022
Dependency on sqlite has been removed since v17.04.0-ce. See: moby/moby#30208 Signed-off-by: TIAN Yuanhao <[email protected]> Reviewed-by: Christian Stewart <[email protected]>
paralin
pushed a commit
to skiffos/buildroot
that referenced
this pull request
Mar 4, 2022
Dependency on sqlite has been removed since v17.04.0-ce. See: moby/moby#30208 Signed-off-by: TIAN Yuanhao <[email protected]> Reviewed-by: Christian Stewart <[email protected]>
paralin
pushed a commit
to skiffos/buildroot
that referenced
this pull request
Mar 7, 2022
Dependency on sqlite has been removed since v17.04.0-ce. See: moby/moby#30208 Signed-off-by: TIAN Yuanhao <[email protected]> Reviewed-by: Christian Stewart <[email protected]>
woodsts
pushed a commit
to woodsts/buildroot
that referenced
this pull request
Mar 7, 2022
Dependency on sqlite has been removed since v17.04.0-ce. See: moby/moby#30208 Signed-off-by: TIAN Yuanhao <[email protected]> Reviewed-by: Christian Stewart <[email protected]> Tested-by: Marcus Hoffmann <[email protected]> Signed-off-by: Peter Korsgaard <[email protected]>
buildroot-auto-update
pushed a commit
to buildroot/buildroot
that referenced
this pull request
Mar 17, 2022
Dependency on sqlite has been removed since v17.04.0-ce. See: moby/moby#30208 Signed-off-by: TIAN Yuanhao <[email protected]> Reviewed-by: Christian Stewart <[email protected]> Tested-by: Marcus Hoffmann <[email protected]> Signed-off-by: Peter Korsgaard <[email protected]> (cherry picked from commit 6105ad3) Signed-off-by: Peter Korsgaard <[email protected]>
buildroot-auto-update
pushed a commit
to buildroot/buildroot
that referenced
this pull request
Mar 17, 2022
Dependency on sqlite has been removed since v17.04.0-ce. See: moby/moby#30208 Signed-off-by: TIAN Yuanhao <[email protected]> Reviewed-by: Christian Stewart <[email protected]> Tested-by: Marcus Hoffmann <[email protected]> Signed-off-by: Peter Korsgaard <[email protected]> (cherry picked from commit 6105ad3) Signed-off-by: Peter Korsgaard <[email protected]>
ccrisan
pushed a commit
to ccrisan/thingos
that referenced
this pull request
Jun 26, 2025
Dependency on sqlite has been removed since v17.04.0-ce. See: moby/moby#30208 Signed-off-by: TIAN Yuanhao <[email protected]> Reviewed-by: Christian Stewart <[email protected]> Tested-by: Marcus Hoffmann <[email protected]> Signed-off-by: Peter Korsgaard <[email protected]>
ccrisan
pushed a commit
to qtoggle/qtoggleos
that referenced
this pull request
Oct 21, 2025
Dependency on sqlite has been removed since v17.04.0-ce. See: moby/moby#30208 Signed-off-by: TIAN Yuanhao <[email protected]> Reviewed-by: Christian Stewart <[email protected]> Tested-by: Marcus Hoffmann <[email protected]> Signed-off-by: Peter Korsgaard <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
fixes #22146
relates to #16032
Rebase of #25782 cc @LK4D4
Removed sqlite dependency.
Removed every usage of graphdb.
Compile daemon
Dependency from sqlite removed, so links upgrade from 1.9 does not work. External migration tool should be used if necessary.