Skip to content

KSP-ModularManagement/MM-FORNEEDS-ProofOfConcept

Folders and files

NameName
Last commit message
Last commit date

Latest commit

 

History

19 Commits
 
 
 
 
 
 
 
 

Repository files navigation

Module Manager :FOR[]:NEEDS[] (failed) Proof of Concept

This is a FAILED Proof of Concept for using :FOR[]:NEEDS[] while patching third-parties add'ons for features introduced by one Add'On (and that not rarely depends on a third one).

Unfortunately, it failed due an excessive trust on the MM Documentation that not only prevented me to identify the code on MM where the (in my view) misbehaviour would be happening, but also leaded me to cook a Test Case where the problem didn't triggered, fooling me into believing I had a valid theory.

The subjacent idea, however, is still valid - but it can't be implemented using stock Module Manager as I was intending. That really sucks.

The aftermath is an update on the Forum's Module Manager documentation, in the hopes no one falls into this booby trap again.

Current Work in Progress Proposal

To split the :FOR behaviour into two new MM patches (and not only one as previously):

  • :THIS, a patch for creating "tags" (_modname_s in the documentation) and nothing more.
    • It will be, still, always "executed" to preserve that part of the :FOR semantics.
  • :AT, a new patch that will be used for ordering the parching.
    • Implementing this part of the :FOR semantics.
  • :FOR would be left as is forever, and it would be probably be a nice idea to issue a warning (only once) about it, as IMHO this thing should be considered deprecated once :THIS and AT: is implemented - if it will ever happen, of course...

References

About

A Proof of Concept for using :FOR[]:NEEDS[] on Module Manager.

Topics

Resources

Stars

Watchers

Forks

Releases

No releases published

Packages

No packages published