Faré wrote:
On Sun, Feb 23, 2014 at 5:55 PM, Robert P. Goldman rpgoldman@sift.info wrote:
This one puts the values in SYSTEM-DEPENDS-ON, SYSTEM-DEFSYSTEM-DEPENDS-ON, and SYSTEM-WEAKLY-DEPENDS-ON in canonical form before returning them.
It also bundles fixes from Faré.
Thanks a lot.
I pushed minor tweaks to the documentation.
I notice that your commit 86e2a6f8 has the commit message "Improved documentation of feature dependencies by Fare, editing by RPG." when in fact the commit documented the (:feature <expression> <dep>) dependency-def and __removed__ the (feature <expression>) requirement.
I mostly wrote that commit, but did not want to commit myself since it removed a feature, and only the maintainer (you, Robert) could take that manageurial decision. I believe it is the correct thing to do, at least eventually, but it definitely is worth using cl-test-grid between this and release. I just want to make sure you committed that knowingly.
I think it's the right thing to do. Let's see what happens in cl-test-grid. Meantime, to the parsing code for my caching slots, I will add a check for obsolete FEATURE syntax with an error message indicating the appropriate repair (and a test case).