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

Extensions Warnings should be optimized #67

Open
je4 opened this issue Mar 10, 2023 · 2 comments
Open

Extensions Warnings should be optimized #67

je4 opened this issue Mar 10, 2023 · 2 comments
Assignees

Comments

@je4
Copy link
Contributor

je4 commented Mar 10, 2023

Initial Extension

As I am using an unregistered initial extension, I feel that spec and validation codes are not entirely clear.

According to https://github.com/OCFL/extensions#optional-initial-extension the initial extension resides in the initial folder in the extensions subdirectory. This causes two problems

  1. Warning W013 or W016 is given for registered AND unregistered extensions.
  2. If the validator would accept initial as a subdirectory, there would be no warning, even if a Local Extension is used.

Extension directory with registered extension name

If an extension directory is named after a registered extension, but config.json points to a local extension, no warning is given.

Possible solution

Replace

Extension subdirectories SHOULD be named after a registered extension name in the OCFL Extensions repository.

With something like

Extension subdirectories with exception of initial SHOULD use the same name as the extension. Only registered extensions SHOULD be used.

@je4 je4 changed the title Extensions Extensions Warnings should be optimized Mar 10, 2023
@neilsjefferies
Copy link
Member

Editors agree. Spec will be updated.

@neilsjefferies
Copy link
Member

neilsjefferies commented Sep 22, 2023

Possible, and slightly more elegant, alternative from @awoods would be to simply define an extension with the special name "initial" which explains its function. transferring this ticket to the extensions repository

@rosy1280 rosy1280 transferred this issue from OCFL/spec Sep 22, 2023
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

No branches or pull requests

2 participants