Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 631186 - [kde overlay] kde5-functions.eclass documentation s/remove/require/ in commit 2c1d7fc81
Summary: [kde overlay] kde5-functions.eclass documentation s/remove/require/ in commit...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Overlays (show other bugs)
Hardware: All Linux
: Normal minor
Assignee: Gentoo KDE team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-09-17 07:34 UTC by Duncan
Modified: 2017-09-17 12:03 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Duncan 2017-09-17 07:34:14 UTC
Recent commits 2c1d7fc81 and 3bcc9956a "improved documentation" for the kde5-* eclasses.  Much clearer in general, but there's a "Cupertino"[1] in 2c1d7fc81, replacing require with remove, that needs (single-word) reversed.  From the commit:

 # @ECLASS-VARIABLE: PLASMA_MINIMAL
 # @DESCRIPTION:
-# Minimal Plasma version to require for the package.
+# Minimum version of Plasma to remove. This affects add_plasma_dep.
 : ${PLASMA_MINIMAL:=5.10.4}


Obviously, require is correct and is what's used in the similar vars documentation as well as the previous version, so s/remove/require/ .

---
[1] "The Cupertino effect is the tendency of a spell checker to suggest or autocorrect with incorrect words to replace misspelled words and words not in its dictionary." https://en.wikipedia.org/wiki/Cupertino_effect
Comment 1 Andreas Sturmlechner gentoo-dev 2017-09-17 12:03:00 UTC
Thanks, fixed in git commit 600e259eef3768a34a6183211f16941050fefc26