Wie einen funktionierenden Patch ins Release bekommen?
Eingetragen von sry (8)
am 17.08.2014 - 16:09 Uhr in
am 17.08.2014 - 16:09 Uhr in
Servus,
im Modul Realname konnte man in 6.x die Sichtbarkeit von der Berechtigung abhängig machen, in 7.x nicht (vgl. auch http://www.drupalcenter.de/node/39134 )
Inzwischen war die Maintainerin von Realname 6.x so nett, das in einem (gut funktionierenden) Patch umzusetzen, aber er wird nicht übernommen: https://www.drupal.org/node/2068195
Was muss/kann ich tun, um das zu erreichen? Kann mir vielleicht ein Contributer aus dem Kreis hier weiterhelfen?
TIA,
Sándor
- Anmelden oder Registrieren um Kommentare zu schreiben
Code muss wohl noch weiter entwickelt werden
am 17.08.2014 - 17:46 Uhr
Ich habe mal kurz in den Thread geschaut: Du hast einen Feature Request gestellt und die Haupt-Maintainerin des Moduls (NancyDru) hat die Funktion portiert, aber erst in einem ersten Schritt. Ein anderer der Maintainer hat an der Lösung etwas bemängelt, was NacncyDru sogar bestätigt hat. Ok, Du hast den Patch getestet, das ist schon mal hilfreich für die Maintainer, aber offensichtlich hat noch niemand anderes die Zeit gefunden weiter am Code zu arbeiten. Du hast ja sogar schon Donation angeboten, was auch ein Weg ist weiter zu kommen. Du kannst außer nett nachfragen bei diesen drei Maintainern evtl. einen anderen Programmierer bitten und evtl. für seinen Aufwand bezahlen, den Code weiter zu entwickeln. Aber unterm Strich haben im Moment erstmal nur alle drei Maintainer, die schon in der Issue aktiv sind, die Berechtigung, Code zum Modul zu committen, also einen Patch einzupflegen.
Funktionalität ist übrigens für verantwortungsvolle Maintainer nicht das einzige Kriterium zur Übernahme von Code. Und es spricht für die Qualität dieses Moduls, daß sich die Haupt-Maintainerin nicht einfach über die Bedenken der anderen Maintainer hinweg setzt.
Update: Ich habe mal Rückfragen in der Issue gestellt und den Status zu "needs work" geändert, da die Maintainer dies wohl wollen, wie ich "zwischen den Zeilen" gelesen habe. Auf jeden Fall kommt die Issue damit erstmal auch in deren Listen wieder nach oben.
Update 2: Übrigens landet ein Patch der committed wird nicht automatisch gleich in einem Release. Er wird erstmal Teil der Dev-Version. Daraus wird dann evtl. früher oder später ein Release erzeugt.
# DrupalCenter-Moderator # https://www.drupal.org/u/c-logemann
# CTO der Nodegard GmbH: CMS Security & Availability Operations / Wir unterstützen IT-Abteilungen, Agenturen, Freiberufler:innen
Danke schonmal...
am 17.08.2014 - 17:45 Uhr
...für die Erläuterungen, und dass Du auf drupal.org in den Thread eingestiegen bist.
CU,
Sándor
Verhalten der Maintainer nicht eindeutig in diesem Fall
am 17.08.2014 - 17:50 Uhr
...für die Erläuterungen, und dass Du auf drupal.org in den Thread eingestiegen bist.
Ich finde das Verhalten der Maintainer in diesem Fall auch nicht eindeutig. Nicht, daß es Aufgabe der Maintainer ist das selbst zu machen, aber wenn Sie Verbesserungen wünschen, sollten sie meiner Meinung nach auch den Status der Issue entsprechend ändern. Dies habe ich dann wiederum mit meinem Kommentar angesprochen.
# DrupalCenter-Moderator # https://www.drupal.org/u/c-logemann
# CTO der Nodegard GmbH: CMS Security & Availability Operations / Wir unterstützen IT-Abteilungen, Agenturen, Freiberufler:innen
Mist, ein Doppel-Post
am 17.08.2014 - 17:53 Uhr
Oh man, jetzt mir das auch mal passiert mit einem Doppel-Post.
Dann nutze ich das mal zu einem kleinen Hinweis: Leute, wenn Ihr Änderungen auf der Berechtigungs-Seite oder der Module-Seite speichert, hütet Euch vor mehrmaligen Drücken des Senden-Buttons. Das kann großes Chaos in Eurer Konfiguration nach sich ziehen.
# DrupalCenter-Moderator # https://www.drupal.org/u/c-logemann
# CTO der Nodegard GmbH: CMS Security & Availability Operations / Wir unterstützen IT-Abteilungen, Agenturen, Freiberufler:innen
Leider Stillstand
am 19.10.2014 - 15:05 Uhr
Servus,
außer dass ein "willzzz" den Status auf "Needs Review" geändert hat, blieb alles wie es war.
Du kannst außer nett nachfragen bei diesen drei Maintainern evtl. einen anderen Programmierer bitten und evtl. für seinen Aufwand bezahlen, den Code weiter zu entwickeln.
Nett nachgefragt habe ich IMO durchaus, passiert aber nix. Kannst Du mir sagen, wie die Alternative "einen anderen Programmierer bitten..." läuft, v. a. im Hinblick darauf, dass wohl nur die Maintainer etwas committen dürfen?
TIA,
Sándor
Patch überarbeitet
am 19.10.2014 - 18:10 Uhr
Jetzt kannst Du auch noch mal testen und vllt. andere motivieren auch Tests durchzuführen.
# DrupalCenter-Moderator # https://www.drupal.org/u/c-logemann
# CTO der Nodegard GmbH: CMS Security & Availability Operations / Wir unterstützen IT-Abteilungen, Agenturen, Freiberufler:innen
Danke schonmal
am 19.10.2014 - 23:19 Uhr
werde gerne Tests lostreten.
CU,
Sándor
Patch needs work (still)
am 19.10.2014 - 23:24 Uhr
Nun, einer der Maintainer möchte gerne noch ein paar Verbesserungen. Dazu gehören nun auch Tests (was er auch schon mal früher hätte erwähnen können). Diese Tests sind zwar grundsätzlich sehr sinnvoll aber mindestens so aufwendig wie die bisherige Arbeit, die schon von den anderen in diesen Patch geflossen sind. Ich will hier gerade nicht versprechen, daß ich das so bald liefern kann in meiner Freizeit.
# DrupalCenter-Moderator # https://www.drupal.org/u/c-logemann
# CTO der Nodegard GmbH: CMS Security & Availability Operations / Wir unterstützen IT-Abteilungen, Agenturen, Freiberufler:innen