Skip to content

feat: add option to copy instead of link on app level #2072

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

kassi
Copy link

@kassi kassi commented Jul 6, 2025

This pull request adds an application specific setting symlink = false|true to application config files.
If this setting is set to false, then no files will be linked, but instead copied over.

In this version, the default value, if the setting does not exist, is true to not break backwards compatibility.

May close: #1854
May obsolete: #2008

Scope
This version allows for copying all files of one application instead of linking them.
It's not meant to work for individual files only, like link filea but do not link fileb. This would add another level of complexity, but also of complication inside the config files, either by adding a tag to each file to sync or have different sections like configuration_files_to_link and configuration_files_to_copy and the same for xdg_configuration_files.
In order to keep it clean and simple, I chose the provided variant.

All submissions

Improving the Mackup codebase

  • My submission passes the tests
  • I have linted the code locally prior to submission
  • I have written new tests as applicable
  • I have added an explanation of what the changes do

@redox
Copy link

redox commented Jul 30, 2025

Thank you for working on this @kassi - looking forward to getting it merged

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.

Symlinks replaced by applications (for Library/Preferences)
2 participants