Skip to content

Improve presentation of file system mapping#3890

Open
henrikt-ma wants to merge 1 commit into
modelica:masterfrom
henrikt-ma:cleanup/filesystem-entities
Open

Improve presentation of file system mapping#3890
henrikt-ma wants to merge 1 commit into
modelica:masterfrom
henrikt-ma:cleanup/filesystem-entities

Conversation

@henrikt-ma
Copy link
Copy Markdown
Collaborator

@henrikt-ma henrikt-ma commented Apr 28, 2026

This is a follow-up to #3885 with general cleanup of the the presentation in File System Mapping of Package/Class.

Improvements include:

  • More careful introduction and use of terminology (structured entity and nonstructured entity).
  • Aligned language in subsections to make similarities easier to spot.
  • Add missing rule that package.mo should only define a single class.
  • Expand non-normative section on sub-directories without package.mo.

@henrikt-ma henrikt-ma requested a review from HansOlsson April 28, 2026 22:22
@HansOlsson
Copy link
Copy Markdown
Collaborator

I believe we need to discuss this more - as the PR made me realize that it is currently a bit of a mess.

So, the issue is that a .mo file can be:

  • A file (not called package.mo) containing a top-level package or class
  • A package.mo file for the class corresponding to the directory name
  • A .mo file inside a directory with a package.mo

To me both of the latter ones are part of "structured entity"; i.e, the entire directory from the top-down would be called a structured entity. (Note that we have package.mo both at the top and further down; with similar rules - although only the top-directory may include version.)

@HansOlsson HansOlsson added this to the 2026-June milestone Jun 3, 2026
@HansOlsson
Copy link
Copy Markdown
Collaborator

Language group: should ideally to re-work to avoid the terms structured and non-structured entity.

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