Skip to content

Conversation

seryan-rh
Copy link
Contributor

No description provided.

@yselkowitz yselkowitz linked an issue Sep 2, 2025 that may be closed by this pull request
@tdawson
Copy link
Member

tdawson commented Sep 4, 2025

What is needed for this? Is it just testing and verifying?
If that's all it needs, I've got my testing system setup and can run it through.

@seryan-rh
Copy link
Contributor Author

Yes it is just testing and verifying please. Thanks Troy

@tdawson
Copy link
Member

tdawson commented Sep 5, 2025

OK, I can do that. It might not be done today, but I'll upload the output somewhere so you all can look at it.

@tdawson
Copy link
Member

tdawson commented Sep 8, 2025

The output and logs are here:
https://tdawson.fedorapeople.org/eln/weak-deps-output.tgz
https://tdawson.fedorapeople.org/eln/weak-deps-logs.tgz

What are a few good packages to check?

@seryan-rh
Copy link
Contributor Author

The ones I mostly used to check were the hunspell packages. In particular hunspell-ru should contain hunspell and langpacks-ru.

@tdawson
Copy link
Member

tdawson commented Sep 8, 2025

Yep, looks like it works.
hunspell-am has a "Reverse boolean Weak runtime dependency" of hunspell and lanpacks-am
hunspell-ru has a "Reverse boolean Weak runtime dependency" of hunspell and lanpacks-ru

I'm giving this a thumbs up.

Copy link
Member

@yselkowitz yselkowitz left a comment

Choose a reason for hiding this comment

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

@seryan-rh just one minor wording change.

@tdawson this didn't significantly change the processing time, did it?

@tdawson
Copy link
Member

tdawson commented Sep 8, 2025

@tdawson this didn't significantly change the processing time, did it?

My testing machine get's very different times than the production machine.
That being said, the time is fairly close to other tests I've run. So, from what I see, no, it doesn't change the processing time very much.

Co-authored-by: Yaakov Selkowitz <yselkowi@redhat.com>
@yselkowitz yselkowitz merged commit 9dd5446 into fedora-eln:master Sep 9, 2025
1 check passed
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.

Content Resolver does not show reverse boolean weak dependencies

3 participants