search

FriendsOfREDAXO / cache_warmup

Generiert den Cache vorab, so dass die Website bereits beim Erstaufruf performant läuft
MIT License
41 stars 2 forks source link

package.yml: Conflicts definieren #51

Closed schuer closed 7 years ago

schuer commented 7 years ago 
conflicts:
    packages:
        media_manager: ">= 3"
        structure: ">= 3"

Siehe @gharlan s Vorschlag hier: https://github.com/FriendsOfREDAXO/cache_warmup/issues/50#issuecomment-278774852

gharlan commented 7 years ago

Wenn dann irgendwann https://github.com/redaxo/redaxo/issues/899 umgesetzt ist, kann man genauso dann auch nach unten Schranken setzen, wenn man mal mindestens eine Version brauchen sollte. Geht aktuell aber noch nicht. Also nach unten und oben beschränken, ohne eine feste Abhängigkeit zu haben.

Ich meine sowas:

conflicts:
    packages:
        media_manager: "< 2.1 || >= 3"

Hieße dann: Wenn media_manager installiert ist, dann bitte >= 2.1 und < 3. Aber es muss nicht installiert sein.

schuer commented 7 years ago

Danke für den Hinweis!

schuer commented 7 years ago

Dieses Issue hat sich vorerst erledigt, oder?

Wenn ein neues Major-Release vom Media Manager kommt, würde man Cache-Warmup sowieso darauf anpassen und nicht etwa mittels conflicts-Definition auf den alten Media Manager beschränken wollen.

gharlan commented 7 years ago

Ich fände es schon korrekt/besser, das hinzuzufügen. Und zwar nicht erst, wenn es irgendwann mal einen MM 3.0 gibt, sondern eben gerade frühzeitig. Es geht ja darum, dass bei denen, die dann noch die alte cache_warmup-Version haben, das Update verhindert wird. Ansonsten läuft das Update durch, und danach kommt es dann zu Fehlern.

Ist ja auch der Grund, warum man überhaupt sowas wie ^2.0 bei den normalen Abhängigkeiten verwendet, und nicht etwa >=2.0. Also auch da beschränkt man es ja bewusst nach oben, da man noch nicht absehen kann, ob man zu zukünftigen Major Releases kompatibel sein wird.

ping @staabm oder wie siehst du das?

schuer commented 7 years ago

Stimmt, klingt sehr sinnvoll. Auch für andere AddOns, die sowas nach und nach übernehmen sollten.

staabm commented 7 years ago

Mit conflict rules habe ich bisher nie gearbeitet, kann nix dazu sagen.

generell sind nach oben offene version constraints aber ein anti-pattern und sollten vermieden werden.