Use std::filesystem

[dschwen]

Closes #21134

[loganharbour]

If I remember right one of our min compilers doesn't fully support filesystem...

[dschwen]

If I remember right one of our min compilers doesn't fully support filesystem...

Let's see if they do with the additional -l flags added...

[dschwen]

Ugh, both min compilers error out on the include already:

fatal error: 'filesystem' file not found
#include <filesystem>
         ^~~~~~~~~~~~
1 error generated.

[dschwen]

If I remember right one of our min compilers doesn't fully support filesystem...

So it's actually "Both of our min compilers fully don't support filesystem."

[moosebuild]

Job Documentation on 1af1d72 wanted to post the following:

View the site here

This comment will be updated on new commits.

[github-actions]

This pull request has been automatically marked as stale because it has not had recent activity in the last 100 days. It will be closed in 7 days if no further activity occurs. Thank you for your contributions.

[lindsayad]

@loganharbour we've done some min compiler bumping since this PR was first opened, haven't we?

[loganharbour]

@loganharbour we've done some min compiler bumping since this PR was first opened, haven't we?

I think the current min still doesn't fully support std::filesystem, @dschwen sound right?

[dschwen]

From the issue:

Using this library may require additional compiler/linker options. GNU implementation prior to 9.1 requires linking with -lstdc++fs and LLVM implementation prior to LLVM 9.0 requires linking with -lc++fs.

We haven't bumped this much, have we?

[lindsayad]

@YaqiWang sounds like we're still a little ways off from supporting std::filesystem

[YaqiWang]

@YaqiWang sounds like we're still a little ways off from supporting std::filesystem

Thanks for letting me know. I think I can still wait.

[lindsayad]

sigh, yea still failing gcc. 😢

[YaqiWang]

It is not a building failure though. The tests might be fixable. Oh, gcc test is a building failure.

[dschwen]

I'll refold the uexternaldb test. Whether the output directory is '.' or '' should not matter.

[GiudGiud]

Can we do something like they do here? For gcc 7.5
nlohmann/json#3090

[moosebuild]

Job Coverage on 1af1d72 wanted to post the following:

Framework coverage

	ae61f4	#21137 1af1d7
	Total	Total	+/-	New
Rate	84.83%	84.83%	-0.00%	100.00%
Hits	84017	84005	-12	10
Misses	15026	15024	-2	0

Diff coverage report

Full coverage report

Modules coverage

Tensor mechanics

	ae61f4	#21137 1af1d7
	Total	Total	+/-	New
Rate	89.23%	89.23%	+0.00%	-
Hits	26560	26562	+2	0
Misses	3207	3207	-	0

Diff coverage report

Full coverage report

Full coverage reports

Reports

• framework
• chemical_reactions
• combined
• contact
• electromagnetics
• external_petsc_solver
• fluid_properties
• fsi
• functional_expansion_tools
• geochemistry
• heat_conduction
• level_set
• misc
• navier_stokes
• optimization
• peridynamics
• phase_field
• porous_flow
• ray_tracing
• rdg
• reactor
• richards
• scalar_transport
• solid_properties
• stochastic_tools
• tensor_mechanics
• thermal_hydraulics
• xfem

This comment will be updated on new commits.

[github-actions]

This pull request has been automatically marked as stale because it has not had recent activity in the last 100 days. It will be closed in 7 days if no further activity occurs. Thank you for your contributions.