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

Fix --fail-graph when there is indirect cycles #1620

Conversation

Pantsworth
Copy link
Contributor

@Pantsworth Pantsworth commented Jan 17, 2024

Fixes #1568

Bugfix to support graphing resolves if they have cycles from using "!" requirements.

… indirect cycles

Signed-off-by: Michael Nowakowski <m_nowakowski@apple.com>
@Pantsworth Pantsworth requested a review from a team as a code owner January 17, 2024 23:02
@JeanChristopheMorinPerso JeanChristopheMorinPerso added this to the 3.0.0 milestone Jan 17, 2024
@JeanChristopheMorinPerso JeanChristopheMorinPerso changed the title Bugfix #1568 - Enable graphing resolves with indirect cycles Fix --fail-graph when there is indirect cycles Jan 27, 2024
Copy link
Member

@JeanChristopheMorinPerso JeanChristopheMorinPerso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks you so much @Pantsworth and thank you for taking the time to add a test!

Signed-off-by: Jean-Christophe Morin <jean_christophe_morin@hotmail.com>
@JeanChristopheMorinPerso JeanChristopheMorinPerso merged commit 4209c8b into AcademySoftwareFoundation:main Jan 27, 2024
54 of 55 checks passed
Pixel-Minions pushed a commit to Pixel-Minions/rez that referenced this pull request Feb 14, 2024
…tion#1620)

Fix --fail-graph when there is indirect cycles (AcademySoftwareFoundation#1620)

Signed-off-by: Michael Nowakowski <m_nowakowski@apple.com>
Signed-off-by: Jean-Christophe Morin <jean_christophe_morin@hotmail.com>
Co-authored-by: Jean-Christophe Morin <jean_christophe_morin@hotmail.com>
Signed-off-by: Jose Enriquez <jose.enriquez@barnstormvfx.com>
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.

Indirect Cycle Causes Conflict Graph to Hang Indefinitely
2 participants