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

common/scripts: import xbps-cycles. #31631

Merged
merged 6 commits into from Jun 30, 2021
Merged

Conversation

ericonr
Copy link
Member

@ericonr ericonr commented Jun 23, 2021

From https://github.com/ahesford/xbps-cycles, license is compatible with
void-packages. Will be run in CI, so it should live in the same
repository.

General

Have the results of the proposed changes been tested?

  • I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
  • I generally don't use the affected packages but briefly tested this PR

@ericonr
Copy link
Member Author

ericonr commented Jun 23, 2021

Probably won't work as is, should run inside a void container.

ericonr and others added 3 commits June 24, 2021 21:02
From https://github.com/ahesford/xbps-cycles, that is based on
https://gist.github.com/Chocimier/de76441493ec7775c201dac0bb03ced5 .
License is compatible with void-packages. Will be run in CI, so it
should live in the same repository.
Should help in catching cyclic dependencies early.

Rename lockthreads.yml to include all scheduled CI tasks.
@Chocimier
Copy link
Member

Top patch is for sending mail on cycles detected.

@Chocimier
Copy link
Member

Changed to open informative issue rather than sending vague email.
Same cycle is reported only once, but if there are branches, it can open more than one issue.

https://github.com/Chocimier/void-packages-org/issues

@ericonr
Copy link
Member Author

ericonr commented Jun 30, 2021

Ok, and it already implements a check to not open repeat issues ! ANything else we need?

@Chocimier
Copy link
Member

I think it's ready. @ahesford ?

@ahesford
Copy link
Member

Looks good to me

@Chocimier Chocimier merged commit 18d9a5d into void-linux:master Jun 30, 2021
@ericonr ericonr deleted the cycles branch June 30, 2021 19:20
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 29, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants