Skip to content

data: Add phrog specific schema overrides#148

Merged
samcday merged 1 commit into
samcday:mainfrom
agx:automount
May 9, 2026
Merged

data: Add phrog specific schema overrides#148
samcday merged 1 commit into
samcday:mainfrom
agx:automount

Conversation

@agx
Copy link
Copy Markdown
Collaborator

@agx agx commented Jan 24, 2026

We e.g. don't want to mount any removable media and can add other overrides here.

See: https://gitlab.postmarketos.org/postmarketOS/pmaports/-/issues/4284

We had some other settings that would require slightly different defaults but I don't remember which one those were 😢 . Marking as draft until we know this is what we actually need to fix the referenced issues.

@agx agx marked this pull request as draft January 24, 2026 15:18
Comment thread data/00_mobi.phosh.Phrog.gschema.override Outdated
We e.g. don't want to mount any removable media and can add other
overrides here.

See: https://gitlab.postmarketos.org/postmarketOS/pmaports/-/issues/4284
Signed-off-by: Guido Günther <agx@sigxcpu.org>
@samcday
Copy link
Copy Markdown
Owner

samcday commented Jan 30, 2026

The last messages from the OP in pmOS' tracker makes me think that this solution alone was insufficient or something?

@agx
Copy link
Copy Markdown
Collaborator Author

agx commented Jan 30, 2026

I don't think anybody tested it yet, I was hoping the bug submitter would pick but that was too optimistic.

@samcday
Copy link
Copy Markdown
Owner

samcday commented Jan 30, 2026

No wuckas :) I'll be active in the vicinity next few days so I'll try to pick this up 👍

@agx agx marked this pull request as ready for review May 6, 2026 10:14
@agx
Copy link
Copy Markdown
Collaborator Author

agx commented May 6, 2026

I'll undraft this as it will help for e.g. first-boot as well to have a Phrog value in XDG_CURRENT_DESKTOP.

@samcday
Copy link
Copy Markdown
Owner

samcday commented May 9, 2026

LGTM, thanks!

I'll open a follow-up that plumbs the new schema overrides into the Alpine/Debian/Fedora packages.

@samcday samcday merged commit 3321878 into samcday:main May 9, 2026
14 checks passed
@agx
Copy link
Copy Markdown
Collaborator Author

agx commented May 10, 2026

Thanks, I totally forgot about the new xtask thing.

@samcday
Copy link
Copy Markdown
Owner

samcday commented May 10, 2026

Thanks, I totally forgot about the new xtask thing.

Who can keep up with my constant ADHD whimsies? I certainly can't, so I wouldn't expect anyone else to. 😅

But also, I don't think we need to pump this through the xtask yet, unless we end up needing to template it to account for different distro quirks, see #202

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.

3 participants