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

flatpak: re-use configuration from users home #52

Merged
merged 7 commits into from
Feb 27, 2024

Conversation

stefanb2
Copy link
Contributor

Add access rights to ${XDG_CONFIG_HOME}/qMasterPassword to the flatpak application. This allows users to easily switch from a system package to the flatpak application.

This partially revertsi commit 39227b8

This makes sure that editors format file content already in the format
that won't be rejected by clang-format later.
Fedora 40 has been branched off from Rawhide a short while ago.
It is annoying that this build was always triggered when you were trying
to test another build locally. Now it can be disabled with

   $ act ... --matrix pre-commit:no ...
This makes sure that changes are also verified by the Flatpak build.

- added Flatpak build manifest
  * copied .yml from

   https://github.com/flathub/io.github.bkueng.qMasterPassword

  * change source from "tar" to "dir" to build the current source tree
- add Flatpak bundle copied out from act-cli container to .gitignore
- add matrix workflow for Flatpak build
  * based on the official Flatpak Builder GitHub action with artifact
    upload disabled
  * add Flatpak bundle as GitHub build artifact
  * build can be disabled with

   $ act ... --matrix flatpak:no ...
The workflow is not an official release build, hence nothing should be
cached. That was the reason why it was so slow, because it uploaded
stuff to the cache after the artifacts had been generated.
Add access rights to ${XDG_CONFIG_HOME}/qMasterPassword to the flatpak
application. This allows users to easily switch from a system package to
the flatpak application.
Copy link
Owner

@bkueng bkueng left a comment

Choose a reason for hiding this comment

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

Make sense. I'm not sure how strict flathub is with sharing config files.

@bkueng bkueng merged commit 69b0342 into bkueng:main Feb 27, 2024
21 checks passed
@stefanb2 stefanb2 deleted the topic-allow-xdg-config-in-flatpak branch February 28, 2024 06:41
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.

2 participants