Skip to content

taskwarrior2: fix build with cmake 4#449892

Closed
Daviey wants to merge 1 commit intoNixOS:masterfrom
Daviey:fix/taskwarrior2-cmake4-build-failure
Closed

taskwarrior2: fix build with cmake 4#449892
Daviey wants to merge 1 commit intoNixOS:masterfrom
Daviey:fix/taskwarrior2-cmake4-build-failure

Conversation

@Daviey
Copy link
Contributor

@Daviey Daviey commented Oct 8, 2025

Fixes #449826

This patches the CMakeLists.txt files in taskwarrior to require CMake 3.10 instead of 3.0. CMake 4 doesn't support versions older than 3.5, and versions between 3.5-3.10 are deprecated.

The patch updates all 8 CMakeLists.txt files in the project (root, doc, performance, scripts, src, src/columns, src/commands, and test).

Upstream PR: GothenburgBitFactory/taskwarrior#3969

Tested by building the package and running the test suite on Linux.

@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux. 10.rebuild-darwin: 11-100 This PR causes between 11 and 100 packages to rebuild on Darwin. 12.first-time contribution This PR is the author's first one; please be gentle! labels Oct 8, 2025
@nix-owners nix-owners bot requested review from MarcWeber and oxalica October 8, 2025 12:23
@Daviey
Copy link
Contributor Author

Daviey commented Oct 8, 2025

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 449892
Commit: a82cecbd9e0933134f2558f5198f75013a97d4f9


x86_64-linux

✅ 21 packages built:
  • calcure
  • calcure.dist
  • gnomeExtensions.taskwhisperer
  • python312Packages.bugwarrior
  • python312Packages.bugwarrior.dist
  • python312Packages.tasklib
  • python312Packages.tasklib.dist
  • python312Packages.taskw
  • python312Packages.taskw-ng
  • python312Packages.taskw-ng.dist
  • python312Packages.taskw.dist
  • python313Packages.tasklib
  • python313Packages.tasklib.dist
  • python313Packages.taskw
  • python313Packages.taskw-ng
  • python313Packages.taskw-ng.dist
  • python313Packages.taskw.dist
  • tasknc
  • taskwarrior2
  • vit
  • vit.dist

@nixpkgs-ci nixpkgs-ci bot removed the 12.first-time contribution This PR is the author's first one; please be gentle! label Oct 9, 2025
Add a proper patch that updates all CMakeLists.txt files to require
CMake 3.10 instead of 3.0.

This approach is preferred as it fixes the issue at the source and
can be carried as a patch until merged upstream.

Upstream PR: GothenburgBitFactory/taskwarrior#3969
Fixes: NixOS#449826

Signed-off-by: Dave Walker <[email protected]>
@Daviey Daviey force-pushed the fix/taskwarrior2-cmake4-build-failure branch from a82cecb to d8d17c9 Compare October 9, 2025 09:32
Copy link
Contributor

@drupol drupol left a comment

Choose a reason for hiding this comment

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

Instead of hosting the patch in nixpkgs, wouldn't it be a good idea to fetch the patches from upstream using fetchpatch instead ?

@LordGrimmauld
Copy link
Contributor

Instead of hosting the patch in nixpkgs, wouldn't it be a good idea to fetch the patches from upstream using fetchpatch instead ?

The upstream PRis not merged. You can't fetch unmerged PRs, bcause they may be garbage-collected after being force-pushed.

@drupol
Copy link
Contributor

drupol commented Oct 14, 2025

Instead of hosting the patch in nixpkgs, wouldn't it be a good idea to fetch the patches from upstream using fetchpatch instead ?

The upstream PRis not merged. You can't fetch unmerged PRs, bcause they may be garbage-collected after being force-pushed.

How about the patches themselves. Each of them.

@LordGrimmauld
Copy link
Contributor

How about the patches themselves. Each of them.

that is exactly what is being done here? I am confused about what you want.

@iedame
Copy link
Contributor

iedame commented Oct 24, 2025

I am closing this PR as it is a duplicate of #450177

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

Labels

10.rebuild-darwin: 11-100 This PR causes between 11 and 100 packages to rebuild on Darwin. 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Build failure: taskwarrior2

4 participants