Skip to content

Enable date and time settings for remote files#9

Closed
nanshiki wants to merge 4 commits intojrohel:mainfrom
nanshiki:datetime
Closed

Enable date and time settings for remote files#9
nanshiki wants to merge 4 commits intojrohel:mainfrom
nanshiki:datetime

Conversation

@nanshiki
Copy link
Copy Markdown

@nanshiki nanshiki commented Dec 8, 2025

int 21h ax=5701h updates the sft's file_time.
The file close packet adds the date/time information and sends the file_time to the server, which then sets the file's date/time information.
With this modification, the date and time information will also be copied when copying a file.

…ata.

Fix to prevent date/time settings from being applied when closing files if file writing occurs.
@nanshiki
Copy link
Copy Markdown
Author

Fixed the server to accept file close packets without date and time data.

Fix to prevent date/time settings from being applied when closing files if file writing occurs.
This addresses an issue where the date and time are not updated when opening and writing to an existing file.

@nanshiki
Copy link
Copy Markdown
Author

In the sample program for UNDOCUMENTED DOS, SFT's rel_sector/abs_sector are not initialized, and it appears that their contents are neither referenced nor overwritten in actual DOS environments. Therefore, we store the file open date/time information in rel_sector/abs_sector, compare it upon closing, and if any changes are detected, we send the data to the server for updating.

@jrohel
Copy link
Copy Markdown
Owner

jrohel commented Dec 21, 2025

Thank you for your PRs. I’ll review them later. I currently have a lot of other work.

Note:
I also noticed your fork some time ago. I have a question about it, but I’ll leave it for later when I have more time.

@jrohel jrohel closed this Jan 25, 2026
@jrohel jrohel reopened this Jan 25, 2026
@jrohel
Copy link
Copy Markdown
Owner

jrohel commented Mar 13, 2026

@nanshiki
I’m reaching out after some time. Previously, I considered adding certain features, but I wanted to maintain full compatibility between NetMount client, NetMount server, and NMManage.

The issue with your PR is that the client would no longer work with older servers.

I’ve prepared release, 1.8.0 https://github.com/jrohel/NetMount/releases/tag/client-v1.8.0%2Cserver-v1.8.0%2Cninstall-v1.0%2Cnmmanage-v1.1.0, for both the server and client. It extends the communication protocol while keeping full version compatibility - the client detects what the server supports.

Key additions in 1.8.0:

  • Support for setting the file time when closing a file. The detection of whether the file time has changed is done in the same way as in your PR. This is essentially the simplest approach possible, although there is a limitation: if the client sets the same time as the original, the client time is ignored and the server will use its own time.

  • Reporting of "large" disk sizes. This is a new function added to the network redirector and is supported, for example, by the new FreeDOS Kernel 2044 (version 2.44, 2026).

New features are enabled only if supported by both client and server.

I’m closing this PR. Thank you very much for your contribution and effort.

@jrohel jrohel closed this Mar 13, 2026
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