Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Avoid std::filesystem #3602

Merged
merged 1 commit into from
Oct 19, 2023
Merged

Conversation

WeiqunZhang
Copy link
Member

It has caused a lot of issues on systems with a default gcc <= 8. PR #3523 was trying to the issue. But unfortunately it does not always work. So we are going to simply avoid std::filesystem.

It has caused a lot of issues on systems with a default gcc <= 8.  PR AMReX-Codes#3523
was trying to the issue. But unfortunately it does not always work. So we
are going to simply avoid std::filesystem.
Copy link
Member

@ax3l ax3l left a comment

Choose a reason for hiding this comment

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

LGTM. Want to remove the build logic around it as well with the PR?

Copy link
Member

@ax3l ax3l left a comment

Choose a reason for hiding this comment

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

Ah, I see #3601 now

@WeiqunZhang WeiqunZhang merged commit 83bc468 into AMReX-Codes:development Oct 19, 2023
69 checks passed
@WeiqunZhang WeiqunZhang deleted the rm_fs_fgrad branch October 19, 2023 19:16
guj pushed a commit to guj/amrex that referenced this pull request Dec 13, 2023
It has caused a lot of issues on systems with a default gcc <= 8. PR
AMReX-Codes#3523 was trying to the issue. But unfortunately it does not always
work. So we are going to simply avoid std::filesystem.
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.

2 participants