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

Add Unix ls discussion to implementation-notes #304

Closed
zimeon opened this issue Feb 14, 2019 · 2 comments
Closed

Add Unix ls discussion to implementation-notes #304

zimeon opened this issue Feb 14, 2019 · 2 comments
Assignees
Milestone

Comments

@zimeon
Copy link
Contributor

zimeon commented Feb 14, 2019

Split from #281: we have agreed to add a discussion of zero padding and lexical sorting in Unix ls to the implementation-notes. See also #2. I propose something like the following as a new paragraph to 3.2.1 Version Numbering:

Systems with version directories have often used zero-padding in order to show version order with common lexical sorting tools (such as Unix ls). Zero-padding is not recommended in order to avoid having to make arbitrary choices of padding length or to place limits of the number of versions supported. In archival storage use cases, objects with more than 9 versions are rare, and while it is important that manual OCFL object be possible, it is not expected to be the norm. OCFL-aware tools can, of course, easily sort unpadded version directories in version order.

@birkland
Copy link
Contributor

birkland commented Feb 14, 2019 via email

@ahankinson
Copy link
Contributor

I would agree with @birkland. We only have limited data about what constitutes the 'norm' in versioning, since we've been largely basing this assumption on experiences with Moab.

GNU ls does support lexographical ordering with ls -v; With BSD ls the output can be piped through sort, ls | sort -n. So I think we can still achieve lexographical ordering with built-in OS tools without going into too much detail or recommending a problematic (and permanent) change to version numbering.

@zimeon zimeon added this to the Beta milestone Feb 14, 2019
awoods pushed a commit that referenced this issue Feb 21, 2019
* Add note about zero padding not being recommended
* Fix typo

Fixes #304
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants