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

Fix for issue #132 #135

Merged
merged 2 commits into from
Oct 1, 2021
Merged

Fix for issue #132 #135

merged 2 commits into from
Oct 1, 2021

Conversation

phprus
Copy link
Contributor

@phprus phprus commented Sep 22, 2021

Fix ghc::filesystem::remove_all behavior if argument is symlink (#132)
See https://en.cppreference.com/w/cpp/filesystem/remove:

Symlinks are not followed (symlink is removed, not its target).

Update CI: Switch from ubuntu 16.04 to 18.04

@phprus
Copy link
Contributor Author

phprus commented Sep 28, 2021

@gulrak review my PR please.

@gulrak
Copy link
Owner

gulrak commented Oct 1, 2021

Sorry, I had some very busy weeks at my job. Thank you for this fix and for the CI update to 18.04!

@phprus
Copy link
Contributor Author

phprus commented Oct 1, 2021

I apologize for distracting you from your job.

@gulrak
Copy link
Owner

gulrak commented Oct 1, 2021

No, please, I'm happy you fixed it! No apologies needed, I plan to make a new release late (in CEST) on Sunday. Thanks for your supportive work!

@phprus phprus deleted the issue-132 branch April 16, 2022 10:39
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