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

Add a step to check if patches apply before invoking TuxSuite #800

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nathanchance
Copy link
Member

There is no point in spinning up TuxSuite for a build if any patches that we need to apply are going to fail to do so. At the beginning of a build, check if the patches apply to the current repository and reference. This is done by downloading a tarball of the source, running git init and committing the files, then finally running git quiltimport to make sure that that patches still apply. It would be simpler to use git clone --depth=1 but cloning can be expensive on the server side and these jobs may run at the same time.

Closes: #796

NOTE: I have not bumped the generated files to make this easier to review and reason through. I will do it before merging.

There is no point in spinning up TuxSuite for a build if any patches
that we need to apply are going to fail to do so. At the beginning of a
build, check if the patches apply to the current repository and
reference. This is done by downloading a tarball of the source, running
'git init' and committing the files, then finally running
'git quiltimport' to make sure that that patches still apply. It would
be simpler to use 'git clone --depth=1' but cloning can be expensive
on the server side and these jobs may run at the same time.

Closes: ClangBuiltLinux#796
Signed-off-by: Nathan Chancellor <[email protected]>
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.

Explore adding a check that patches apply at current revision before calling tuxsuite
1 participant