Skip to content

feat: support backup packaging function#31

Open
sjugge wants to merge 1 commit intofosskers:masterfrom
sjugge:pkg-meta-backup
Open

feat: support backup packaging function#31
sjugge wants to merge 1 commit intofosskers:masterfrom
sjugge:pkg-meta-backup

Conversation

@sjugge
Copy link

@sjugge sjugge commented Sep 3, 2024

This PR adds support for the backup packaging function.

rationale

Additional files can be included, which may be user-modified configuration files. As-is, a package build using cargo-aur will overwrite any user-made changes to files that are contained in the package.

With this feature, files can be defined which may be user-modified. If a file is user-modified, the packaged file will be added as .pacnew and not overwrite any changes made by the user.

If a package is removed, and any packaged file is user-modified, a .pacsave file will be retained.

implementation

I extended the existing [package.metadata.aur] approach as to not get in the way of Cargo's default [package] and took a similar approach as to how the package() section is built up. This does duplicate that logic somewhat, but I suspect there may be other package functions that could piggy back of this approach in the future should there be a need for them.

I added relevant information in terms of usage in the README.

@fosskers
Copy link
Owner

fosskers commented Sep 8, 2024

Thanks for this, I'll review as soon as I'm able. Note that there will likely be some merge conflicts once #30 goes in.

@sjugge
Copy link
Author

sjugge commented Sep 13, 2024

Should #30 get merged before this PR I'll revisit and adapt as needed, nw.

@ArjixWasTaken
Copy link

ArjixWasTaken commented Jun 26, 2025

since you are already editing the README in this PR, might as well fix the language for this codeblock
image

toml -> bash

Edit: just looked at the dates...this PR won't get merged anytime soon huh

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

Comments