Skip to content

Stop vendoring toml11#10972

Merged
Ericson2314 merged 3 commits intomasterfrom
no-vendor-toml11
Jun 27, 2024
Merged

Stop vendoring toml11#10972
Ericson2314 merged 3 commits intomasterfrom
no-vendor-toml11

Conversation

@Ericson2314
Copy link
Member

Motivation

We don't apply any patches to it, and vendoring it locks users into bugs (it hasn't been updated since its introduction in late 2021).

Closes https://git.lix.systems/lix-project/lix/issues/164

Change-Id: Ied071c841fc30b0dfb575151afd1e7f66970fdb9 (cherry picked from commit 80405d0)

Context

Taken from Lix, thanks @winterqt!

Priorities and Process

Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

@cole-h
Copy link
Member

cole-h commented Jun 27, 2024

Somewhat related to #10330, I guess

@Ericson2314 Ericson2314 mentioned this pull request Jun 27, 2024
@Ericson2314 Ericson2314 force-pushed the no-vendor-toml11 branch 2 times, most recently from a7c8384 to 48d6328 Compare June 27, 2024 02:26
Ericson2314 and others added 2 commits June 26, 2024 22:26
We just don't need it!
We don't apply any patches to it, and vendoring it locks users into
bugs (it hasn't been updated since its introduction in late 2021).

Closes https://git.lix.systems/lix-project/lix/issues/164

Change-Id: Ied071c841fc30b0dfb575151afd1e7f66970fdb9
(cherry picked from commit 80405d0)
@Ericson2314 Ericson2314 merged commit aff0fbc into master Jun 27, 2024
@Ericson2314 Ericson2314 deleted the no-vendor-toml11 branch June 27, 2024 04:10
sqlite
(toml11.overrideAttrs (old: {
# TODO change in Nixpkgs, Windows works fine.
meta.platforms = lib.platforms.all;
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

Thank you!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants