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

Stop putting system.base at the front of the order #74

Merged
merged 3 commits into from
Jul 19, 2024

Conversation

ermo
Copy link
Contributor

@ermo ermo commented Jun 29, 2024

We now use OrderedSets, which keep the original topological sort, so this shouldn't actually be necessary now.

This also implies that the only function of system.base is for the packages in it to be un-removable.

NB: This change is experimental and will need loads of upgrade testing!

@ermo ermo added the enhancement New feature or request label Jun 29, 2024
@ermo ermo force-pushed the undo-system.base-reorder-operation branch 2 times, most recently from defef0e to c1d7bd0 Compare June 29, 2024 21:20
ermo added 3 commits June 30, 2024 01:44
We now use OrderedSets, which keep the original topological sort,
so this shouldn't actually be necessary now.

This also implies that the only function of system.base is for the
packages in it to be un-removable.

Signed-off-by: Rune Morling <ermo@serpentos.com>
This should make the testing of PRs before landing them more viable.

Signed-off-by: Rune Morling <ermo@serpentos.com>
In addition, list enabled repositories in ${SOSROOT}

Signed-off-by: Rune Morling <ermo@serpentos.com>
@ermo ermo force-pushed the undo-system.base-reorder-operation branch from c1d7bd0 to 515f101 Compare June 29, 2024 23:45
@ermo
Copy link
Contributor Author

ermo commented Jun 30, 2024

In the testing I have done locally and for building VMs/chroots/ISO, the above changes have yet to break anything FWIW.

@ReillyBrogan ReillyBrogan merged commit 63c5d0a into main Jul 19, 2024
@ReillyBrogan ReillyBrogan deleted the undo-system.base-reorder-operation branch July 19, 2024 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants