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

Use emaint for syncing and for applying package moves #4

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Use emaint for syncing and for applying package moves #4

wants to merge 7 commits into from

Conversation

ghost
Copy link

@ghost ghost commented May 7, 2015

By using emaint for syncing, EMERGE_DEFAULT_OPTS is ignored and, thus, it
will not break if a user has --ask as a default option (as some do). Note that
emerge --sync calls through to the emaint sync system anyway.

Also, emaint provides a specific option for applying package moves so use it in place of emerge --info.

By using emaint for syncing, EMERGE_DEFAULT_OPTS is ignored and, thus, it
will not break if a user has --ask as a default option (as some do). Note that
emerge --sync calls through to the emaint sync system anyway.

Also, emaint provides a specific option for applying package moves so use it.
@GreatEmerald
Copy link

emaint also syncs layman repos now, so in addition to this the layman syncing step should be disabled.

Kerin Millar added 5 commits August 24, 2018 01:09
The previous commit merged upstream changes while also removing support
for SYNC_OVERLAYS_CMD. In this day and age, portage supports a formal
repo configuration syntax and there is really no need to specify a
distinct command. Instead, default to emaint sync -a, which
automatically syncs all repos that have "auto-sync = yes" in effect. To
account for this change, also provide a commented sample SYNC_CMD that
syncs only the gentoo repo - thus, avoiding any overlays.
@ghost
Copy link
Author

ghost commented Aug 24, 2018

Pull request updated to address the comment from @GreatEmerald, with test support added and passed. Please review.

@hartwork
Copy link
Member

Hi! Will need to come back later for a closer look. Could you get the merge commit out of the commit list? That would be cool. Thanks!

@ghost
Copy link
Author

ghost commented Aug 25, 2018

I don't know how to do that but I'll see if I can figure it out. Otherwise, I'll do it all over and overwrite my branch.

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.

2 participants