Skip to content

Conversation

@pierrepebay
Copy link
Contributor

Fixes: #25

@pierrepebay pierrepebay self-assigned this Feb 5, 2025
@pierrepebay pierrepebay linked an issue Feb 5, 2025 that may be closed by this pull request
@pierrepebay pierrepebay requested review from cz4rs and nmm0 February 5, 2025 16:34
@pierrepebay pierrepebay marked this pull request as ready for review February 5, 2025 16:34
Copy link
Contributor

@cz4rs cz4rs left a comment

Choose a reason for hiding this comment

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

Note that DARMA-tasking/vt#2399 allows for using fmt 11.

Overall, maybe we could get away with NOT specifying the version for fmt? That would offer the most flexibility and reduce the maintenance for this package as well ;)

@pierrepebay
Copy link
Contributor Author

Note that DARMA-tasking/vt#2399 allows for using fmt 11.

Overall, maybe we could get away with NOT specifying the version for fmt? That would offer the most flexibility and reduce the maintenance for this package as well ;)

Yes agreed.

@cz4rs cz4rs force-pushed the 25-change-fmt-dependency-to-less-than-11 branch from f8c8968 to 4e32e27 Compare April 28, 2025 14:22
@cz4rs cz4rs changed the title #25: Specify fmt less than 11 #25: remove fmt version requirement Apr 28, 2025
@cz4rs cz4rs force-pushed the 25-change-fmt-dependency-to-less-than-11 branch from 4e32e27 to b6ae2b7 Compare July 16, 2025 10:51
@cz4rs cz4rs force-pushed the 25-change-fmt-dependency-to-less-than-11 branch 9 times, most recently from 2a27b70 to d2d81fc Compare July 16, 2025 13:00
@cz4rs cz4rs force-pushed the 25-change-fmt-dependency-to-less-than-11 branch 2 times, most recently from a3d2ca9 to bdfe256 Compare July 17, 2025 13:45
@cz4rs cz4rs force-pushed the 25-change-fmt-dependency-to-less-than-11 branch from bdfe256 to 8eb588d Compare July 17, 2025 13:46
@JacobDomagala JacobDomagala self-assigned this Jul 17, 2025
@JacobDomagala
Copy link
Member

Failure happens due to incorrect cmake setup on vt side (for vt-trace target) -> DARMA-tasking/vt#2458

Copy link
Member

@JacobDomagala JacobDomagala left a comment

Choose a reason for hiding this comment

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

🚀

Copy link
Contributor

@lifflander lifflander left a comment

Choose a reason for hiding this comment

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

Looks good

@lifflander lifflander merged commit d1300df into master Jul 21, 2025
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove fmt version requirement

5 participants