Remove fmtheaders from CUDA-Q internal headers

[sacpis]

Required prerequisites

• Search the issue tracker to check if your feature has already been mentioned or rejected in other issues.

Describe the feature

From @amccaskey
The big issue here is that Logger.h is a common header for downstream libraries (so they can get cudaq::info()) and it has FmtCore.h as an include. The changes here do not change that. The real fix is to remove any fmtlib headers from our own headers and hide them in cpp implementation code. I'm not sure how one would do that and keep cudaq::info() variadic though.

Please see comment on the PR #3014.

[sacpis]

Assigning it to @Renaud-K as he has already worked on it in PR #3764. @Renaud-K Can you please confirm and close this issue?

[Renaud-K]

#3764 takes care of the biggest issue whereby FmtCore.h was included in Logger.h, but there are still a few places where FmtCore.h or fmt/....h are directly included in headers files. So, these would probably have to be cleaned up as well before #3014 can proceed.

[sacpis]

#3764 takes care of the biggest issue whereby FmtCore.h was included in Logger.h, but there are still a few places where FmtCore.h or fmt/....h are directly included in headers files. So, these would probably have to be cleaned up as well before #3014 can proceed.

Thanks @Renaud-K.

[Renaud-K]

Fixed with #3770