Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

define package conflicts #55

Merged
merged 2 commits into from Feb 19, 2017
Merged

define package conflicts #55

merged 2 commits into from Feb 19, 2017

Conversation

schuer
Copy link
Member

@schuer schuer commented Feb 18, 2017

fixes #51

@staabm
Copy link
Member

staabm commented Feb 19, 2017

//cc @gharlan

@gharlan
Copy link
Member

gharlan commented Feb 19, 2017

👍

Nur noch mal zur Klarstellung: Folge ist, sobald irgendeines der Addons in Version 3 erscheint, können die Leute, die Cache Warmup installiert haben, das jeweilige Addon nicht updaten.
Die müssen dann entweder Cache Warmup deinstallieren, oder updaten (falls schon neue Version da, die auch die Version 3 unterstützt), oder halt das andere Addon nicht updaten.

Kann also auch nerven, vor allem, wenn Cache Warmup eigentlich doch mit der Version 3 kompatibel wäre.

Aber da wir das ja jetzt nicht wissen können, finde ich das alles so richtig. Wie Markus ja sagte, offene Grenzen nach oben sind ein Antipattern.
Und ich denke so ist es auch besser, als wenn man dann zwar updaten darf, aber dann plötzlich komische Dinge passieren (bzw. nichts mehr passiert).

package.yml Outdated
php: '>=5.5'

conflicts:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Vllt solltest du hier nen kommentar reinmachen was diese regel bewirkt... conflict rules sind bisher in FOR nicht üblich

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done, danke.

@staabm
Copy link
Member

staabm commented Feb 19, 2017

@gharlan haben wir eine ordentliche message beim installieren wenn es wg einem conflikt nicht klappt? (Damit die leute wissen was das problem ist und die ursache erkennen kann)?

@gharlan
Copy link
Member

gharlan commented Feb 19, 2017

Dies sind die Meldungen, die auftreten können, je nach Konstellation:
https://github.com/redaxo/redaxo/blob/master/redaxo/src/core/lang/de_de.lang#L284-L291

Da drüber steht dann noch sowas wie "Addon X konnte aus folgendem Grund nicht aktualisiert werden:".
Im oben genannten Fall, also jemand versucht eines der anderen Addons auf Version 3 zu aktualisieren, würden die _reverse_conflict_-Messages greifen.

php: '>=5.5'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@gharlan brauchts auch ne conflict rule für php >7 (da wir hier noch kein || können?)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bei der PHP-Version scheint es üblicher zu sein, die Grenze doch nach oben offen zu lassen. Zumindest sehe ich das meist so in den composer.json's.
Ein Beispiel, Symfony: https://github.com/symfony/symfony/blob/master/composer.json#L19

@schuer
Copy link
Member Author

schuer commented Feb 19, 2017

Danke euch! Wollen wir dann mergen?

@staabm staabm merged commit 949ec87 into FriendsOfREDAXO:master Feb 19, 2017
@schuer schuer deleted the patch-1 branch February 19, 2017 11:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

package.yml: Conflicts definieren
3 participants