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

UNPACKFILENAME.STRING deals with TOPS20 versions #2030

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

rmkaplan
Copy link
Contributor

This replicates the behavior in the Venue sysout, as best as I can determine, if OSTYPE is TOPS20.

If there is one dot in the name, it sets off the extension.

Two dots: the extension and version

More dots: the last 2 are the extension and version, the others go into the name.

If also a ;T, return includes TEMPORARY T.

Not sure what the right behavior is for other ;FOO. Venue returns a second version (which will presumably never be seen by a LISTGET), this throws it away.

@rmkaplan rmkaplan marked this pull request as draft February 12, 2025 18:15
@MattHeffron
Copy link
Contributor

MattHeffron commented Feb 12, 2025

Not sure what the right behavior is for other ;FOO.

Ignore any irrelevant or non-standard attributes; especially since we don't need/use any information in the attributes(?)

@nbriggs
Copy link
Contributor

nbriggs commented Feb 12, 2025

I agree -- ignore other attributes. It's fine to return the (TEMPORARY T) for ;T but Unix systems don't have a built-in concept of files that get flushed at logout, so it's not terribly useful.

@rmkaplan
Copy link
Contributor Author

rmkaplan commented Feb 12, 2025 via email

@masinter
Copy link
Member

I think this request might have come through the attempts to run Eurisco / AM from Doug Lenat without making changes, but i'm nto sure. Without a clear example of the problem it seems likely that we'll get it wrong.

It's not clear to me that the "tenex/tops-20" file system will ever be used for any files we try to access.

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.

4 participants