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

app-portage/eix now hooks postsync.d #254

Merged
merged 1 commit into from
Jan 15, 2025

Conversation

berney
Copy link
Contributor

@berney berney commented Jan 5, 2025

  • Since app-portage/eix 0.36.7-r2 it now installs eix-postsync as a symlink in portage's postsync via /etc/portage/postsync.d/.
  • eix-postsync does what engine/docker/bob-core/etc/portage/postsync.d/eix did, so it is no longer needed.
  • Removed configure_eix() function as it is no longer needed.
  • Run eix-update executable directly rather than calling configure_eix function.

Before this change kubler build figlet would fail with:

 * Messages for package app-portage/flaggie-0.99.8:

 * This is a preview release of flaggie 1.x. It it not fully featured
 * yet and it may have significant bugs. Please back your /etc/portage
 * up before using it. Verify the results using --pretend.
>>> Auto-cleaning packages...

>>> No outdated packages were found on your system.
Reading Portage settings...
Building database (/var/cache/eix/portage.eix)...
[0] "gentoo" /var/db/repos/gentoo/ (cache: metadata-md5-or-flat)
     Reading category 172|172 (100) Finished
Applying masks...
Calculating hash tables...
Writing database file /var/cache/eix/portage.eix...
Database contains 19076 packages in 172 categories
ln: failed to create symbolic link '/etc/portage/postsync.d/50-eix-postsync': File exists

/etc/portage/postsync.d/50-eix-postsync exists, and was installed by app-portage/eix. Because upstream eix package does the hooking now, it is no longer necessary for Kubler to install hooks.

- Since app-portage/eix 0.36.7-r2 it now installs eix-postsync as a
  symlink in portage's postsync via /etc/portage/postsync.d/.
- eix-postsync does what
  engine/docker/bob-core/etc/portage/postsync.d/eix did, so it is no
  longer needed.
- Removed `configure_eix()` function as it is no longer needed.
- Run `eix-update` executable directly rather than calling
  `configure_eix` function.
@r7l
Copy link
Contributor

r7l commented Jan 6, 2025

Thanks allot for the patch! Works perfectly and fixes the issue.

Copy link
Owner

@edannenberg edannenberg left a comment

Choose a reason for hiding this comment

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

👍

@edannenberg edannenberg merged commit d8a0430 into edannenberg:master Jan 15, 2025
@berney berney deleted the b-eix-portage-postsync.d branch January 16, 2025 06:50
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.

3 participants