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

Jump to bottom

Support for changing ctime for directories #3

Closed
RomulBY opened this issue Oct 4, 2020 · 2 comments
Closed

Support for changing ctime for directories #3

RomulBY opened this issue Oct 4, 2020 · 2 comments

Comments

@RomulBY
Copy link

RomulBY commented Oct 4, 2020

Add support for changing ctime for directories, please.
Delgan added a commit that referenced this issue Oct 4, 2020
@Delgan
Add support for changing ctime of directories (#3)
e6ac901
@Delgan
Copy link
Owner

Delgan commented Oct 4, 2020

This seems to be a welcome improvement. I updated the set_ctime() function so that it supports directories too, you can update to v1.0.3 and it should work.

This was pretty easy, according to the documentation, it only requires to specify the FILE_FLAG_BACKUP_SEMANTICS flag... Except this is wrong.
While doing some research, I came across this very interesting message which explains the problems related to this flag (basically, it's about user privileges): ocaml/ocaml#8797
Considering how complicated it would be for me to implement an alternative solution, I accept the trade-off of using an imperfect function.

@Delgan Delgan closed this as completed Oct 4, 2020
@RomulBY
Copy link
Author

RomulBY commented Oct 6, 2020

Thank you very much! You made my day :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Delgan @RomulBY