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 buldorder recursing when there are no dependencies #106

Closed
wants to merge 2 commits into from

Conversation

JAicewizard
Copy link
Contributor

This is a major problem since the AUR returns an error when the deplist is empty.

To test, try auracle buildorder mingw-w64-environment

This is a major problem since the AUR returns an error when the deplist is empty
@falconindy
Copy link
Owner

Thanks. Agree this is a problem (though definitely not a new one). If you're up for adding a unit test for this in test_buildorder.py that'd be hugely appreciated. Otherwise, I can get to this tonight or tomorrow.

src/auracle/auracle.cc Outdated Show resolved Hide resolved
@JAicewizard
Copy link
Contributor Author

Done

@falconindy falconindy closed this in 29fa98f Sep 3, 2024
@falconindy
Copy link
Owner

Thanks. Pushed with a fix to formatting, and made the test actually pass.

@JAicewizard
Copy link
Contributor Author

Did the tests not pass for you? Thats odd, everything worked for me. But thanks for merging it :)

@JAicewizard JAicewizard deleted the fix_bulidorder branch September 4, 2024 06:10
@falconindy
Copy link
Owner

No, they didn't, and I'm not sure how they could have without the added DB entry for mingw-w64-environment.

@JAicewizard
Copy link
Contributor Author

Oh I see, I forgot to add it to git, and since its not tracked it didn't warn me. My bad :)

@falconindy
Copy link
Owner

No worries. Thanks for providing the fix!

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.

2 participants