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

Show git diff before build #9

Open
bastelfreak opened this issue Mar 29, 2016 · 2 comments
Open

Show git diff before build #9

bastelfreak opened this issue Mar 29, 2016 · 2 comments
Labels

Comments

@bastelfreak
Copy link
Member

@noctux requested something like this. What do we want to show here. diff between the last run and now? the last successful build?

@noctux
Copy link
Contributor

noctux commented Mar 29, 2016

I don't know how advanced pacmans hooks are. ideally, each packages stores the pkgbuild it was built with in some directore (maybe under /var) and we have an preinstall-hook inplace that diffs the new pkgbuild against whatever is stored there, showing the diff (and asking for confirmation?). This is the only way there can be any kind of trust into the repository, as currently, we are building PKGBUILDs that are maintained by heaven knows whom and this person could cause any kind of damage in its PKGBUILD/install-file.

The diff should also include all files/patches/artifacts that are stored in the packages git-repository...
An easy implementation would be to have the git-repo cloned under /var/ and pull it in the preinstall hook, then using git-diff between the revisions.

@bastelfreak
Copy link
Member Author

Jenkins takes care about pulling from the AUR repos. So it should be easy to display a diff here to the last build. A diff in that sense would be a git diff between the old job run and the new one. I don't think that any pacman hooks are required here.

I don't like the idea of asking for confirmation, that would break the CD process. The package should always be build without manual interaction.

A sort of white/blacklist that scans the diff and stops the deployment would be okay (e.g. grep for rm -rf)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants