Skip to content

subpath: serializing uses segments - #458

Closed
jkowalleck wants to merge 1 commit into
package-url:mainfrom
jkowalleck:subpath-serializing-OS_FS-independent
Closed

subpath: serializing uses segments#458
jkowalleck wants to merge 1 commit into
package-url:mainfrom
jkowalleck:subpath-serializing-OS_FS-independent

Conversation

@jkowalleck

@jkowalleck jkowalleck commented Apr 16, 2025

Copy link
Copy Markdown
Member

path segments are environment-dependent, and therefore, the splitting is not really in our domain.
Anyway, we can improve our process in the correct direction, by using "path delimiter" instead of '/'.



this is the serialization-part implied as proposed via #448 (comment):

the [subpath parsing] process joins on / to produce a final result.

  • [...]
  • I think the spec should stop after the path-segments are clear. Joining segments on FileSystem/OperatingSystem dependent path separators (/, \, >, etc - see https://en.wikipedia.org/wiki/Path_(computing) for more) is out of scope of the spec.
  • [...]

is part of #448

@jkowalleck jkowalleck added this to the 1.0-draft milestone Apr 16, 2025
@jkowalleck
jkowalleck marked this pull request as ready for review April 16, 2025 09:48
@jkowalleck
jkowalleck requested a review from a team April 16, 2025 09:48
@johnmhoran johnmhoran modified the milestones: PURL v1.0, PURL v0.90 May 29, 2025
@jkowalleck
jkowalleck force-pushed the subpath-serializing-OS_FS-independent branch from 2110570 to 540bb5f Compare October 1, 2025 08:50
Signed-off-by: Jan Kowalleck <jan.kowalleck@gmail.com>
@jkowalleck
jkowalleck force-pushed the subpath-serializing-OS_FS-independent branch from 540bb5f to 79ff82d Compare October 1, 2025 08:51
Comment thread purl-specification.md
- Append ‘\#’ to the `purl`
- Strip the `subpath` from leading and trailing ‘/’
- Split this on ‘/’ as segments
- Divide the `subpath` into segments using the path delimiter of your environment (operating system, file system, etc)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PURLs are not bound to the operating system they are created on, and their interpretation must not change when used on a different operating system.

@jkowalleck jkowalleck Oct 1, 2025

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please read the ticket's description: #458 (comment)

@mjherzog

Copy link
Copy Markdown
Member

@jkowalleck We need to "move" this PR to update the file:docs/how-to-build.md because the file: purl-specification.md is generated and not suitable for PRs (as documented in the file header).
We are also applying a 78 character line limit for markdown files to make change tracking easier. (see also #612)
Do you want to make the updates? Or I can ask @johnmhoran to make them.

@jkowalleck

Copy link
Copy Markdown
Member Author

Do you want to make the updates? Or I can ask @johnmhoran to make them.

please ask @johnmhoran to take this over.
thanks in advance.

@johnmhoran

Copy link
Copy Markdown
Member

@jkowalleck @mjherzog This PR 458 has been updated and consolidated in #711.

@jkowalleck

Copy link
Copy Markdown
Member Author

superseded by #711

@jkowalleck
jkowalleck deleted the subpath-serializing-OS_FS-independent branch October 24, 2025 09:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants