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

Create missing directories when opening directory/file on {UNIX} device for output #422

Merged
merged 1 commit into from
May 24, 2022

Conversation

nbriggs
Copy link
Collaborator

@nbriggs nbriggs commented May 24, 2022

Prior to this change there was no mechanism within Medley to create a new directory
on a {UNIX} style device. This change makes the {UNIX} directory creation happen in
the same manner that {DSK} directory creation is done. Opening a file with access
OUTPUT, BOTH, or APPEND, which would create the file if it does not exist, will also
create any missing directories in the path.

…ce for output

Prior to this change there was no mechanism within Medley to create a new directory
on a {UNIX} style device.  This change makes the {UNIX} directory creation happen in
the same manner that {DSK} directory creation is done.  Opening a file with access
OUTPUT, BOTH, or APPEND, which would create the file if it does not exist, will also
create any missing directories in the path.
@nbriggs nbriggs requested review from masinter and rmkaplan May 24, 2022 18:21
@nbriggs nbriggs self-assigned this May 24, 2022
@nbriggs
Copy link
Collaborator Author

nbriggs commented May 24, 2022

Fixes Interlisp/medley#779

Copy link

@rmkaplan rmkaplan left a comment

Choose a reason for hiding this comment

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

I tested, seems to solve the problem. Thanks!

@nbriggs nbriggs merged commit a18f09d into master May 24, 2022
@nbriggs nbriggs deleted the nhb-do-unix-directory-creation-on-open branch May 24, 2022 23:08
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