Created attachment 457668 [details, diff] Change "atom" to "package dependency specification" throughout Currently the term "atom" is used quite inconsistently. In the section about blockers it includes the block operator (i.e., ! or !!). When describing the best_version function, it is used in the sense of category/package-version, and when describing profiles/updates is is used in the actual sense of ebuild(5). I think that we should avoid the term altogether because it also is a misnomer. These "atoms" are everything else but atomic. PMS uses "package dependency specification" instead, see bug 174322. Patch is attached, please review.
LGTM for the most part. My only confusion is about best_version() where the word "atom" implied a specific syntax (i.e. ${PN}-{PV} etc.) whereas simply saying "Echoes category, name and version" is a bit vague about the format of the output. Maybe we should say something like "Echoes category, name and version ... in the format of a package dependency specification."
(In reply to Göktürk Yüksek from comment #1) > LGTM for the most part. My only confusion is about best_version() where the > word "atom" implied a specific syntax (i.e. ${PN}-{PV} etc.) whereas simply > saying "Echoes category, name and version" is a bit vague about the format > of the output. Maybe we should say something like "Echoes category, name and > version ... in the format of a package dependency specification." See? Because that's not what the function does. It actually takes a package dependency specification as its _input_, but its output isn't in that format. For example, "best_version app-editors/emacs:24" will output "app-editors/emacs-24.5-r3". (Maybe we should simply add that example?)
Portage has the following: https://gitweb.gentoo.org/proj/portage.git/tree/bin/phase-helpers.sh#n890 """ # Returns highest installed matching category/package-version (without .ebuild). """ I think an example would make it clear enough.
Created attachment 457850 [details, diff] Change "atom" to "package dependency specification" throughout Updated patch with example added.
Ack
Pushed. Closing.
