Skip to content

sysinfo: Also replace - with _ at dns_to_rdns function - #798

Merged
ximion merged 1 commit into
ximion:mainfrom
Rirusha:main
Oct 26, 2026
Merged

ximion merged 1 commit into
ximion:mainfrom
Rirusha:main

Conversation

@Rirusha

@Rirusha Rirusha commented Oct 18, 2026 •

Copy link
Copy Markdown
Contributor

appstream validate failed if cid-rdns-contains-hyphen but as_utils_dns_to_rdns doesn't convert org.my-example to my_example so there is dead-end if HOME_URL equals to https://alt-atomic.org

@github-actions github-actions Bot added the tests label Oct 18, 2026
`appstream validate` failed if `cid-rdns-contains-hyphen` but
`as_utils_dns_to_rdns` doesn't convert `org.my-example` to `my_example`
so there is
dead-end if `HOME_URL` equals to `https://alt-atomic.org
@Rirusha Rirusha changed the title sysinfo: ALso replace - with _ at dns_to_rdns function sysinfo: Also replace - with _ at dns_to_rdns function Oct 18, 2026
@ximion

ximion commented Oct 26, 2026

Copy link
Copy Markdown
Owner

Thanks!
This was from before the times where we wanted to make the AppStream IDs match the D-Bus IDs, so I guess it's fine to change this now to make our generated IDs match the changed requirement.

@ximion
ximion merged commit ab440a3 into ximion:main Oct 26, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants