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: uutils-coreutils #744

Merged
merged 53 commits into from
Nov 1, 2023
Merged

add: uutils-coreutils #744

merged 53 commits into from
Nov 1, 2023

Conversation

madonuko
Copy link
Member

No description provided.

@madonuko madonuko added enhancement New feature or request new This adds a new package labels Oct 19, 2023
korewaChino
korewaChino previously approved these changes Oct 20, 2023
nothingneko
nothingneko previously approved these changes Oct 20, 2023
Copy link
Member

@nothingneko nothingneko left a comment

Choose a reason for hiding this comment

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

LGTM!!

@korewaChino korewaChino dismissed stale reviews from nothingneko and themself via c47af33 October 20, 2023 19:45
Copy link
Member

@lleyton lleyton left a comment

Choose a reason for hiding this comment

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

good and bad news

good: it installs now
bad: I'm not sure why we have the uu- prefix on the binaries (and don't symlink)

@korewaChino korewaChino requested a review from lleyton October 21, 2023 11:39
korewaChino
korewaChino previously approved these changes Oct 23, 2023
@korewaChino
Copy link
Contributor

Many packages require coreutils itself. We need to somehow make the dep solver know that uutils can replace them all.

lleyton
lleyton previously approved these changes Nov 1, 2023
@korewaChino korewaChino merged commit eeacedb into f38 Nov 1, 2023
4 checks passed
@korewaChino korewaChino deleted the w/uutils branch November 1, 2023 01:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request new This adds a new package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants