Skip to content

Fix DateTime constructor default argument handling for macOS#69

Open
ALor79 wants to merge 1 commit intoFraunhofer-FIT-DIEN:mainfrom
ALor79:main
Open

Fix DateTime constructor default argument handling for macOS#69
ALor79 wants to merge 1 commit intoFraunhofer-FIT-DIEN:mainfrom
ALor79:main

Conversation

@ALor79
Copy link

@ALor79 ALor79 commented Nov 25, 2025

Although it is better to completely move the default arguments to header files as suggestted by tomas789 due to the more aggressive nature of CMake on mac. this could be an alternative that allows mac users with intel cores to already start using this library.

@mikroskeem
Copy link

This also fixes building on Apple Silicon Macs.

@m-unkel
Copy link
Collaborator

m-unkel commented Feb 20, 2026

Thanks a lot for reporting this and for taking the time to investigate 👍

You’re absolutely right — this turned out to be a bug on our side, not an Apple-related issue.

I’ve pushed a fix 3c7d7f0 to address the problem. It’s admittedly a bit odd that this only surfaced on macOS and not on Linux or Windows, especially since the class in question isn’t even used in v2 and is scheduled to be completely rewritten in v3.

That said, it definitely shouldn’t have been a compilation blocker, so thanks again for catching it.

Please pull the latest changes and let me know if you’re still seeing any issues.

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.

3 participants