From 476f06d82b3a199f651b89b83779c723a6980c2b Mon Sep 17 00:00:00 2001 From: Zac Medico Date: Fri, 16 Oct 2009 22:47:55 +0000 Subject: Bug #266454 - Make @world an all-inclusive set once again, like it was prior to portage-2.2_rc* releases. In addition to @system, @world now includes a @selected set which represents user-selected "world" packages and sets that saved in /var/lib/portage/world{,sets}. svn path=/main/trunk/; revision=14614 --- RELEASE-NOTES | 5 +++-- cnf/sets.conf | 8 +++++++- doc/config/sets.docbook | 17 ++++++++++++----- man/emerge.1 | 17 ++++++++++------- man/portage.5 | 2 +- pym/_emerge/Scheduler.py | 2 +- pym/_emerge/actions.py | 14 +++++++------- pym/_emerge/create_world_atom.py | 2 +- pym/_emerge/depgraph.py | 23 ++++++++++++----------- pym/_emerge/help.py | 8 ++++---- pym/_emerge/main.py | 4 ++-- pym/_emerge/unmerge.py | 31 ++++++++++++++++--------------- pym/portage/sets/files.py | 8 ++++---- 13 files changed, 80 insertions(+), 61 deletions(-) diff --git a/RELEASE-NOTES b/RELEASE-NOTES index 81ac86682..b4908beb5 100644 --- a/RELEASE-NOTES +++ b/RELEASE-NOTES @@ -18,8 +18,9 @@ portage-2.2 - packages won't be unmerged if they are referenced by an installed package set (with the exception of the world set, and installed being determined by the world_sets file). -* "world" no longer includes "system" unconditionally, but you can add - "@system" to the world_sets file to restore the old state. +* "world" now includes separate "selected" and "system" sets, where the + "selected" set includes packages listed in /var/lib/portage/world and nested + sets that may be listed /var/lib/portage/world_sets. portage-2.1.7 ================================== diff --git a/cnf/sets.conf b/cnf/sets.conf index a921a2bb1..87242ce59 100644 --- a/cnf/sets.conf +++ b/cnf/sets.conf @@ -6,7 +6,13 @@ # Not much that could be changed for world, so better leave it alone [world] -class = portage.sets.files.WorldSet +class = portage.sets.base.DummyPackageSet +world-candidate = False +packages = @selected @system + +# Not much that could be changed for world, so better leave it alone +[selected] +class = portage.sets.files.WorldSelectedSet world-candidate = False # Same as for world, though later portage versions might use a different class diff --git a/doc/config/sets.docbook b/doc/config/sets.docbook index e50c85cc9..8f93b1df2 100644 --- a/doc/config/sets.docbook +++ b/doc/config/sets.docbook @@ -74,9 +74,15 @@ # The classic world set [world] - class = portage.sets.files.WorldSet + class = portage.sets.base.DummyPackageSet world-candidate = False - + packages = @selected @system + + # The selected set + [selected] + class = portage.sets.files.WorldSelectedSet + world-candidate = False + # The classic system set [system] class = portage.sets.profiles.PackagesSystemSet @@ -250,8 +256,8 @@ - - portage.sets.files.WorldSet + + portage.sets.files.WorldSelectedSet A minor variation of StaticFileSet, mainly for implementation reasons. It should never be used in user configurations as it's already configured @@ -580,8 +586,9 @@ The default sets are: + world: uses DummySet + selected: uses WorldSelectedSet system: uses PackagesSystemSet - world: uses WorldSet security: uses NewAffectedSet with default options installed: uses EverythingSet preserved-rebuild: uses PreservedLibraryConsumerSet diff --git a/man/emerge.1 b/man/emerge.1 index 4e33573de..fc0a880eb 100644 --- a/man/emerge.1 +++ b/man/emerge.1 @@ -58,10 +58,13 @@ would like to query the owners of one or more files or directories. .TP .BR set A \fIset\fR is a convenient shorthand for a large group of -packages. Two sets are currently always available: \fBsystem\fR -and \fBworld\fR. \fBsystem\fR refers to a set of packages -deemed necessary for your system to run properly. \fBworld\fR -contains all the packages listed in \fB/var/lib/portage/world\fR. [See +packages. Three sets are currently always available: \fBselected\fR, +\fBsystem\fR and \fBworld\fR. \fBselected\fR contains the user-selected +"world" packages that are listed in \fB/var/lib/portage/world\fR, +and nested sets that may be listed +in \fB/var/lib/portage/world_sets\fR. \fBsystem\fR refers to a set of +packages deemed necessary for your system to run properly. \fBworld\fR +encompasses both the \fBselected\fR and \fBsystem\fR sets. [See \fBFILES\fR below for more information.] Other sets can exist depending on the current configuration. The default set configuration is located in \fB/usr/share/portage/config/sets.conf\fR. Note that a \fIset\fR @@ -105,7 +108,7 @@ setups that the user may wish to run. .BR \-\-depclean Cleans the system by removing packages that are not associated with explicitly merged packages. Depclean works by creating the -full dependency tree from the @system and @world sets, +full dependency tree from the @world set, then comparing it to installed packages. Packages installed, but not part of the dependency tree, will be uninstalled by depclean. See \fB\-\-with\-bdeps\fR for behavior with respect to build time dependencies @@ -114,7 +117,7 @@ always be kept. They can be manually added to this set with \fIemerge \-\-noreplace \fR. As a safety measure, depclean will not remove any packages unless *all* required dependencies have been resolved. As a consequence, it is often necessary to run \fIemerge \-\-update \-\-newuse -\-\-deep @system @world\fR prior to depclean. +\-\-deep @world\fR prior to depclean. \fBWARNING:\fR Inexperienced users are advised to use \fB\-\-pretend\fR with this @@ -300,7 +303,7 @@ and old version to be displayed in an aligned format for easy cut\-n\-paste. .TP .BR "\-\-complete\-graph[=n]" This causes \fBemerge\fR to consider the deep dependencies of all -packages from the system and world sets. With this option enabled, +packages from the world set. With this option enabled, \fBemerge\fR will bail out if it determines that the given operation will break any dependencies of the packages that have been added to the graph. Like the \fB\-\-deep\fR option, the \fB\-\-complete\-graph\fR diff --git a/man/portage.5 b/man/portage.5 index bcf065825..d81232f1c 100644 --- a/man/portage.5 +++ b/man/portage.5 @@ -894,7 +894,7 @@ packages sets which always begin with the @ character. .I Example: .nf -@system +@kde .fi .RE .SH "REPORTING BUGS" diff --git a/pym/_emerge/Scheduler.py b/pym/_emerge/Scheduler.py index eb2f89497..c95003dd7 100644 --- a/pym/_emerge/Scheduler.py +++ b/pym/_emerge/Scheduler.py @@ -1629,7 +1629,7 @@ class Scheduler(PollScheduler): logger = self._logger pkg_count = self._pkg_count root_config = pkg.root_config - world_set = root_config.sets["world"] + world_set = root_config.sets["selected"] world_locked = False if hasattr(world_set, "lock"): world_set.lock() diff --git a/pym/_emerge/actions.py b/pym/_emerge/actions.py index bc0a518f1..febe3b545 100644 --- a/pym/_emerge/actions.py +++ b/pym/_emerge/actions.py @@ -566,7 +566,7 @@ def action_depclean(settings, trees, ldpath_mtimes, msg.append("unless *all* required dependencies have been resolved. As a\n") msg.append("consequence, it is often necessary to run %s\n" % \ good("`emerge --update")) - msg.append(good("--newuse --deep @system @world`") + \ + msg.append(good("--newuse --deep @world`") + \ " prior to depclean.\n") if action == "depclean" and "--quiet" not in myopts and not myfiles: @@ -614,7 +614,7 @@ def action_depclean(settings, trees, ldpath_mtimes, print("Packages installed: " + str(len(vardb.cpv_all()))) print("Packages in world: " + \ - str(len(root_config.sets["world"].getAtoms()))) + str(len(root_config.sets["selected"].getAtoms()))) print("Packages in system: " + \ str(len(root_config.sets["system"].getAtoms()))) print("Required packages: "+str(req_pkg_count)) @@ -634,7 +634,7 @@ def calc_depclean(settings, trees, ldpath_mtimes, vardb = trees[myroot]["vartree"].dbapi deselect = myopts.get('--deselect') != 'n' - required_set_names = ("system", "world") + required_set_names = ("world",) required_sets = {} set_args = [] @@ -647,7 +647,7 @@ def calc_depclean(settings, trees, ldpath_mtimes, # which excludes packages that are intended to be eligible for # removal. world_temp_set = required_sets["world"] - system_set = required_sets["system"] + system_set = root_config.sets["system"] if not system_set or not world_temp_set: @@ -787,7 +787,7 @@ def calc_depclean(settings, trees, ldpath_mtimes, msg.append(" %s" % (parent,)) msg.append("") msg.append("Have you forgotten to run " + \ - good("`emerge --update --newuse --deep @system @world`") + " prior") + good("`emerge --update --newuse --deep @world`") + " prior") msg.append(("to %s? It may be necessary to manually " + \ "uninstall packages that no longer") % action) msg.append("exist in the portage tree since " + \ @@ -1176,9 +1176,9 @@ def calc_depclean(settings, trees, ldpath_mtimes, def action_deselect(settings, trees, opts, atoms): root_config = trees[settings['ROOT']]['root_config'] - world_set = root_config.sets['world'] + world_set = root_config.sets['selected'] if not hasattr(world_set, 'update'): - writemsg_level("World set does not appear to be mutable.\n", + writemsg_level("World @selected set does not appear to be mutable.\n", level=logging.ERROR, noiselevel=-1) return 1 diff --git a/pym/_emerge/create_world_atom.py b/pym/_emerge/create_world_atom.py index 30ade0acf..7ded8af13 100644 --- a/pym/_emerge/create_world_atom.py +++ b/pym/_emerge/create_world_atom.py @@ -68,7 +68,7 @@ def create_world_atom(pkg, args_set, root_config): if len(matched_slots) == 1: new_world_atom = slot_atom - if new_world_atom == sets["world"].findAtomForPackage(pkg): + if new_world_atom == sets["selected"].findAtomForPackage(pkg): # Both atoms would be identical, so there's nothing to add. return None if not slotted: diff --git a/pym/_emerge/depgraph.py b/pym/_emerge/depgraph.py index 525a05296..de60bd801 100644 --- a/pym/_emerge/depgraph.py +++ b/pym/_emerge/depgraph.py @@ -90,7 +90,7 @@ class _frozen_depgraph_config(object): self.pkgsettings[myroot] = portage.config( clone=self.trees[myroot]["vartree"].settings) - self._required_set_names = set(["system", "world"]) + self._required_set_names = set(["world"]) class _dynamic_depgraph_config(object): @@ -1756,7 +1756,7 @@ class depgraph(object): continue if not (isinstance(arg, SetArg) and \ - arg.name in ("system", "world")): + arg.name in ("selected", "system", "world")): self._dynamic_config._unsatisfied_deps_for_display.append( ((myroot, atom), {})) return 0, myfavorites @@ -1783,7 +1783,7 @@ class depgraph(object): # out here if the atom is not from either the system or # world set. if not (isinstance(arg, SetArg) and \ - arg.name in ("system", "world")): + arg.name in ("selected", "system", "world")): return 0, myfavorites # Add the selected package to the graph as soon as possible @@ -3486,7 +3486,7 @@ class depgraph(object): skip = False try: for atom in root_config.sets[ - "world"].iterAtomsForPackage(task): + "selected"].iterAtomsForPackage(task): satisfied = False for pkg in graph_db.match_pkgs(atom): if pkg == inst_pkg: @@ -3775,7 +3775,7 @@ class depgraph(object): if not parent_atoms: atom = self._dynamic_config._blocked_world_pkgs.get(pkg) if atom is not None: - parent_atoms = set([("@world", atom)]) + parent_atoms = set([("@selected", atom)]) if parent_atoms: conflict_pkgs[pkg] = parent_atoms @@ -4428,7 +4428,7 @@ class depgraph(object): pkg_cp = xs[0] root_config = self._frozen_config.roots[myroot] system_set = root_config.sets["system"] - world_set = root_config.sets["world"] + world_set = root_config.sets["selected"] pkg_system = False pkg_world = False @@ -4647,9 +4647,9 @@ class depgraph(object): if "world" in self._dynamic_config._sets: # Filter out indirect members of world (from nested sets) # since only direct members of world are desired here. - world_set = self._frozen_config.roots[self._frozen_config.target_root].sets["world"] + world_set = self._frozen_config.roots[self._frozen_config.target_root].sets["selected"] for arg, atom in self._dynamic_config._missing_args: - if arg.name == "world" and atom in world_set: + if arg.name in ("selected", "world") and atom in world_set: world_problems = True break @@ -4698,7 +4698,7 @@ class depgraph(object): ref_string = " pulled in by " + ref_string msg.append(" %s%s\n" % (colorize("INFORM", str(arg)), ref_string)) msg.append("\n") - if "world" in problems_sets: + if "selected" in problems_sets or "world" in problems_sets: msg.append("This problem can be solved in one of the following ways:\n\n") msg.append(" A) Use emaint to clean offending packages from world (if not installed).\n") msg.append(" B) Uninstall offending packages (cleans them from world).\n") @@ -4729,7 +4729,7 @@ class depgraph(object): if x in self._frozen_config.myopts: return root_config = self._frozen_config.roots[self._frozen_config.target_root] - world_set = root_config.sets["world"] + world_set = root_config.sets["selected"] world_locked = False if hasattr(world_set, "lock"): @@ -4761,7 +4761,8 @@ class depgraph(object): del e all_added = [] for k in self._dynamic_config._sets: - if k in ("args", "world") or not root_config.sets[k].world_candidate: + if k in ("args", "selected", "world") or \ + not root_config.sets[k].world_candidate: continue s = SETPREFIX + k if s in world_set: diff --git a/pym/_emerge/help.py b/pym/_emerge/help.py index e2c7a80ea..50145dad5 100644 --- a/pym/_emerge/help.py +++ b/pym/_emerge/help.py @@ -70,8 +70,8 @@ def help(myopts, havecolor=1): paragraph = "Cleans the system by removing packages that are " + \ "not associated with explicitly merged packages. Depclean works " + \ - "by creating the full dependency tree from the @system and " + \ - "@world sets, then comparing it to installed packages. Packages " + \ + "by creating the full dependency tree from the " + \ + "@world set, then comparing it to installed packages. Packages " + \ "installed, but not part of the dependency tree, will be " + \ "uninstalled by depclean. See --with-bdeps for behavior with " + \ "respect to build time dependencies that are not strictly " + \ @@ -80,7 +80,7 @@ def help(myopts, havecolor=1): "emerge --noreplace . As a safety measure, depclean " + \ "will not remove any packages unless *all* required dependencies " + \ "have been resolved. As a consequence, it is often necessary to " + \ - "run emerge --update --newuse --deep @system @world " + \ + "run emerge --update --newuse --deep @world " + \ "prior to depclean." for line in wrap(paragraph, desc_width): @@ -312,7 +312,7 @@ def help(myopts, havecolor=1): print() print(" "+green("--complete-graph") + "[=%s]" % turquoise("n")) desc = "This causes emerge to consider the deep dependencies of all" + \ - " packages from the system and world sets. With this option enabled," + \ + " packages from the world set. With this option enabled," + \ " emerge will bail out if it determines that the given operation will" + \ " break any dependencies of the packages that have been added to the" + \ " graph. Like the --deep option, the --complete-graph" + \ diff --git a/pym/_emerge/main.py b/pym/_emerge/main.py index 667a811df..3e445b2e8 100644 --- a/pym/_emerge/main.py +++ b/pym/_emerge/main.py @@ -904,8 +904,8 @@ def expand_set_arguments(myfiles, myaction, root_config): for e in setconfig.errors: print(colorize("BAD", "Error during set creation: %s" % e)) - # emerge relies on the existance of sets with names "world" and "system" - required_sets = ("world", "system") + # emerge requires existence of "world", "selected", and "system" + required_sets = ("selected", "system", "world",) missing_sets = [] for s in required_sets: diff --git a/pym/_emerge/unmerge.py b/pym/_emerge/unmerge.py index 4d2a546c5..967d72478 100644 --- a/pym/_emerge/unmerge.py +++ b/pym/_emerge/unmerge.py @@ -283,9 +283,9 @@ def unmerge(root_config, myopts, unmerge_action, from portage.sets.base import EditablePackageSet - # generate a list of package sets that are directly or indirectly listed in "world", + # generate a list of package sets that are directly or indirectly listed in "selected", # as there is no persistent list of "installed" sets - installed_sets = ["world"] + installed_sets = ["selected"] stop = False pos = 0 while not stop: @@ -333,7 +333,8 @@ def unmerge(root_config, myopts, unmerge_action, # skip sets that the user requested to unmerge, and skip world # unless we're unmerging a package set (as the package would be # removed from "world" later on) - if s in root_config.setconfig.active or (s == "world" and not root_config.setconfig.active): + if s in root_config.setconfig.active or \ + (s == "selected" and not root_config.setconfig.active): continue if s not in sets: @@ -510,22 +511,22 @@ def unmerge(root_config, myopts, unmerge_action, raise UninstallFailure(retval) sys.exit(retval) else: - if clean_world and hasattr(sets["world"], "cleanPackage")\ - and hasattr(sets["world"], "lock"): - sets["world"].lock() - if hasattr(sets["world"], "load"): - sets["world"].load() - sets["world"].cleanPackage(vartree.dbapi, y) - sets["world"].unlock() + if clean_world and hasattr(sets["selected"], "cleanPackage")\ + and hasattr(sets["selected"], "lock"): + sets["selected"].lock() + if hasattr(sets["selected"], "load"): + sets["selected"].load() + sets["selected"].cleanPackage(vartree.dbapi, y) + sets["selected"].unlock() emergelog(xterm_titles, " >>> unmerge success: "+y) - if clean_world and hasattr(sets["world"], "remove")\ - and hasattr(sets["world"], "lock"): - sets["world"].lock() + if clean_world and hasattr(sets["selected"], "remove")\ + and hasattr(sets["selected"], "lock"): + sets["selected"].lock() # load is called inside remove() for s in root_config.setconfig.active: - sets["world"].remove(SETPREFIX+s) - sets["world"].unlock() + sets["selected"].remove(SETPREFIX + s) + sets["selected"].unlock() return 1 diff --git a/pym/portage/sets/files.py b/pym/portage/sets/files.py index e8650becd..f4b35037e 100644 --- a/pym/portage/sets/files.py +++ b/pym/portage/sets/files.py @@ -21,7 +21,7 @@ from portage.env.loaders import ItemFileLoader, KeyListFileLoader from portage.env.validators import ValidAtomValidator from portage import dep_getkey, cpv_getkey -__all__ = ["StaticFileSet", "ConfigFileSet", "WorldSet"] +__all__ = ["StaticFileSet", "ConfigFileSet", "WorldSelectedSet"] class StaticFileSet(EditablePackageSet): _operations = ["merge", "unmerge"] @@ -205,11 +205,11 @@ class ConfigFileSet(PackageSet): return rValue multiBuilder = classmethod(multiBuilder) -class WorldSet(EditablePackageSet): +class WorldSelectedSet(EditablePackageSet): description = "Set of packages that were directly installed by the user" def __init__(self, root): - super(WorldSet, self).__init__() + super(WorldSelectedSet, self).__init__() # most attributes exist twice as atoms and non-atoms are stored in # separate files self._lock = None @@ -324,5 +324,5 @@ class WorldSet(EditablePackageSet): self.replace(newworldlist) def singleBuilder(self, options, settings, trees): - return WorldSet(settings["ROOT"]) + return WorldSelectedSet(settings["ROOT"]) singleBuilder = classmethod(singleBuilder) -- cgit v1.2.3-1-g7c22