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

pkg-auto: Some fixes made during weekly updates #2601

Merged
merged 4 commits into from
Jan 22, 2025
Merged

Conversation

krnowak
Copy link
Member

@krnowak krnowak commented Jan 22, 2025

  • for SDK reports, do emerge of not only coreos-devel/sdk-depends, but also dev-lang/rust directly (it's being pulled in by fsscript in stage4)
  • add diff_pkg.sh and occurences.sh scripts to help with debugging report generation failures
  • fix syncing eclass files

eclass is a file, as opposed to the package, which is a directory. So
doing `git -C eclass/foo.eclass log -- .` will fail because we can't
do "cd eclass/foo.eclass", which is what `-C` is trying to do.
The coreos-devel/sdk-depends is a metapackage that is emerged in
stage4 of the catalyst SDK build. But the fsscript that is being run
at the end of the stage4 build also pulls in dev-lang/rust package.
Thus pull both packages for the report.

This should result in updated dev-lang/rust to appear in the reports.
These tools I found useful when I had to investigate why report
generation failed. Since report generation failed, I had no reports
about ebuild diffs or package occurences. The new scripts,
`diff_pkg.sh` and `occurences.sh`, allowed me to get these reports for
the troublesome package.
@krnowak krnowak added the main label Jan 22, 2025
@krnowak krnowak requested a review from a team January 22, 2025 12:36
Copy link
Contributor

@chewi chewi left a comment

Choose a reason for hiding this comment

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

Nice!

@krnowak krnowak merged commit 1caa6dd into main Jan 22, 2025
1 check was waiting
@krnowak krnowak deleted the krnowak/pkg-auto-fixes branch January 22, 2025 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Development

Successfully merging this pull request may close these issues.

2 participants