Skip to content

makeHardcodeGsettingsPatch: Add support for schemaExistsFunction#405229

Merged
jtojnar merged 2 commits intoNixOS:masterfrom
jtojnar:hardcode-gsettings-schema-exists
May 8, 2025
Merged

makeHardcodeGsettingsPatch: Add support for schemaExistsFunction#405229
jtojnar merged 2 commits intoNixOS:masterfrom
jtojnar:hardcode-gsettings-schema-exists

Conversation

@jtojnar
Copy link
Member

@jtojnar jtojnar commented May 8, 2025

evolution-ews introduced a check for optional schemas, which would skip our hardcoded code paths.
https://gitlab.gnome.org/GNOME/evolution-ews/-/commit/a0c514bd345c67fe0fd2ed6ef48a25649b1f4d8e

Let’s update the semantic patch to also replace the invocations of specified schemaExistsFunction for known schemas with true.

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
    • Makes evolution-ews update work properly GNOME 48 followup #405225
    • All in-tree uses produce the same patch.
    • Ran tests.makeHardcodeGsettingsPatch.
  • Fits CONTRIBUTING.md.

Add a 👍 reaction to pull requests you find important.

@jtojnar jtojnar added this to GNOME 48 May 8, 2025
@github-actions github-actions bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. labels May 8, 2025
@jtojnar jtojnar force-pushed the hardcode-gsettings-schema-exists branch from 8f19877 to fb47527 Compare May 8, 2025 13:39
@jtojnar jtojnar moved this to Ready in GNOME 48 May 8, 2025
jtojnar added 2 commits May 8, 2025 16:09
evolution-ews introduced a check for optional schemas, which would skip our hardcoded code paths.
https://gitlab.gnome.org/GNOME/evolution-ews/-/commit/a0c514bd345c67fe0fd2ed6ef48a25649b1f4d8e

Let’s update the semantic patch to also replace the invocations of specified `schemaExistsFunction` for known schemas with `true`.
It has been deprecated for over two years.
@jtojnar jtojnar force-pushed the hardcode-gsettings-schema-exists branch from fb47527 to 34fa007 Compare May 8, 2025 14:10
@jtojnar jtojnar merged commit 0c24dec into NixOS:master May 8, 2025
26 of 28 checks passed
@jtojnar jtojnar deleted the hardcode-gsettings-schema-exists branch May 8, 2025 16:29
@github-project-automation github-project-automation bot moved this from Ready to Done in GNOME 48 May 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant