[PATCH] Neue configure-Option --with-archivedir

[rhabarber1848]

Hi,

http://forum.tuxbox-cvs.sourceforge.net ... 08#p369108

Für alle möglichen Verzeichnisse gibt es configure-Optionen,
nur für cdk/Archive nicht. Wäre da eine zusätzliche Option
--with-archivedir nicht sinnvoll? Ich stimme für ja, dann fällt
der zusätzlich anzulegende Link weg.

Dieser Patch sollte Abhilfe schaffen: EDIT: Patch ist im CVS

Wenn cdk/Archive bereits durch vorhandene Skripte angelegt ist, wird
dieses Verzeichnis genutzt, unabhängig davon, ob --with-archivedir
gesetzt wurde oder nicht, das garantiert Rückwärtskompatibilität.

Default ist, wie bisher, cdk/Archive.

Wenn mittels --with-archivedir ein anderes Verzeichnis angegeben wurde,
wird von cdk/configure automatisch ein Link cdk/Archive dorthin angelegt.
Damit wird vermieden, rules.pl und rules-archive.pl ändern zu müssen.

[Barf]

Grundsätzlich finde ich es eine gute Idee; damit wird symlinks nicht notwendig. Symlinks ist (IMHO) für das Filesystem was das GOTO für die strukturierte Programmierung ist: Die hierarchische Stuktur wird verletzt, ein Kommando wie "cd .." wird nicht mehr semantisch eindeutig, etc.

Wenn cdk/Archive bereits durch vorhandene Skripte angelegt ist, wird
dieses Verzeichnis genutzt, unabhängig davon, ob --with-archivedir
gesetzt wurde oder nicht, das garantiert Rückwärtskompatibilität.

WHAT?!! Unter bestimmte Umständen ignoriert configure die explizite Anweisungen des Users, wegen ... "Rückwärtskompabilität"??. Unakzeptabel. (Möglicherweise eine Warnung wäre OK.)

Default ist, wie bisher, cdk/Archive.

Könnte man nicht gleichzeitig in cdk/archive umbenennen?, alle andere Verzeichnisse in Projekt, augenommen Patches (auch umbenennen), sind klein geschrieben.

Wenn mittels --with-archivedir ein anderes Verzeichnis angegeben wurde,
wird von cdk/configure automatisch ein Link cdk/Archive dorthin angelegt.
Damit wird vermieden, rules.pl und rules-archive.pl ändern zu müssen.

Wenn, dann richtig. Also symlinklos.

[rhabarber1848]

(Möglicherweise eine Warnung wäre OK.)

Die Warnung ist im Patch enthalten:

Code: Alles auswählen

AC_MSG_NOTICE([$buildprefix/Archive found, overriding --with-archivedir])

Was soll ich stattdessen machen, cdk/Archive löschen und
damit evtl. mehrere 100MB Userdownloads plattmachen?

Könnte man nicht gleichzeitig in cdk/archive umbenennen?

Wäre möglich.

Patches (auch umbenennen)

Das würde im CVS bedeuten, dass alle Dateien dort
gelöscht und in cdk/patches neuangelegt werden
müssen, was gleichzeitig die CVS-history wesentlich
schwerer zugänglich macht, imho keine gute Idee.

Wenn, dann richtig. Also symlinklos.

Ich möchte den Eingriff im CVS möglichst klein halten.

[Barf]

Was soll ich stattdessen machen, cdk/Archive löschen und
damit evtl. mehrere 100MB Userdownloads plattmachen?

Man soll davon ausgehen, dass der User der die Option benutzen sich diese Frage gestellt hat, und entweder Gegenmaßnahmen getroffen hat, oder über die Folgen klar ist. Eine Warnung ist OK, eine Entmündigung des Users nicht. Die "Warnung"

Code: Alles auswählen

+AC_ARG_WITH(archivedir,
+	[  --with-archivedir=DIR   where to store downloaded files [[Archive/]]],
+	[if test -d "$buildprefix/Archive"; then
+		archivedir="$buildprefix/Archive"
+		AC_MSG_NOTICE([$buildprefix/Archive found, overriding --with-archivedir])
+	else
+		archivedir="$with_archivedir"
+	fi],[archivedir="$buildprefix/Archive"])
+

ist keine Warnung, es ist eine Entmündigung ("overriding...").

Barf hat geschrieben:
Patches (auch umbenennen)

Das würde im CVS bedeuten, dass alle Dateien dort
gelöscht und in cdk/patches neuangelegt werden
müssen, was gleichzeitig die CVS-history wesentlich
schwerer zugänglich macht, imho keine gute Idee.

OK, war nicht überlegt.

Barf hat geschrieben:
Wenn, dann richtig. Also symlinklos.

Ich möchte den Eingriff im CVS möglichst klein halten.

Verständlich, wir sind alle faul... Was dann bleibt ist eine Konfigurationsoption, die nicht anderes macht als ein Symlink anzulegen. Dazu kann man selbst ln -s eingeben; keine Änderungen sind dafür notwendig.

Meine Meinung ist deswegen: entweder keine Änderung, oder es richtig zu machen.

[rhabarber1848]

Patch aktualisiert.

Wenn --with-archivedir genutzt wird und cdk/Archive ein existierender Link
ist, wird dieser entfernt und eine Infomeldung diesbezüglich ausgegeben.

Wenn --with-archivedir genutzt wird und cdk/Archive ein existierendes Verzeichnis
ist, wird eine Fehlermeldung ausgegeben und configure abgebrochen.

configure: error: directory /root/tuxbox/work_glibc2/compile/cdk/Archive exists, please remove if you want to use --with-archivedir

Somit wird --with-archivedir nicht mehr übergangen.

[Barf]

Ein Schritt in die richtige Richtung, trotzden finde ich ein Abbruch nicht richtig. Ich sehe ich nicht ein, warum der configure-Verfasser sich die Aufgabe stellt, potentiell "doofe" Usecase zu identifizieren und in diese Fälle den Dienst verweigern. Es gibt so viele unterschiedliche Fälle, configure kann und soll nicht versuchen Oberlehrer zu spielen. (Anders ist natürlich falls ein (echtes!) Problem entdeckt wird, wo Weitermachen nicht möglich oder sinnvoll ist.)

Intressant ist übrigens das Ergebnis bei ./configure .... --with-archivedir=.../cdk/Archive

Wichtiger ist noch dass es ist und bleibt eine Symlinkbasierte Halblösung.

[Striper]

Patch aktualisiert.

Wenn --with-archivedir genutzt wird und cdk/Archive ein existierender Link
ist, wird dieser entfernt und eine Infomeldung diesbezüglich ausgegeben.

Wenn --with-archivedir genutzt wird und cdk/Archive ein existierendes Verzeichnis
ist, wird eine Fehlermeldung ausgegeben und configure abgebrochen.

configure: error: directory /root/tuxbox/work_glibc2/compile/cdk/Archive exists, please remove if you want to use --with-archivedir

Somit wird --with-archivedir nicht mehr übergangen.

Finde das so absolut i.O. -> Bitte einchecken! :)

P.S. Wer das so nicht haben will, soll dann bitte konkrete Gegenvorschläge machen, oder es einfach nicht nutzen. :wink:

[rhabarber1848]

Da es für diesen Patch keinen Konsens gibt und die
Einwände von Barf nachvollziehbar sind, habe ich
den Patch nicht eingecheckt.
Diskutiert ruhig noch ein wenig weiter darüber,
ob es bessere Lösungen als meine gibt. Für mich
ist das Thema im Moment erledigt, ich nutze den
aktuellen Patch, der reicht mir.

[Barf]

Wie man sich leicht überzeugen kann (frühere Beiträte sorgfällig durchlesen), war ich (und bin ich) nicht gegen eine --with-archivedir-Option als solche, sondern nur gegen die vorgeschlagene symlinkbasierte Schummellösung, sowie unnötige Benutzerentmündigungen (ist eigentlich ein anderes Thema). Der konkrete Gegenvorschlag ist also es richtig zu machen, mit Make-Variabel und -Substitutionen. Es gibt sicherlich mehrere Leute hier, die sowas in ein Paar Stunden machen könnte... Striper...?

[Striper]

Wie gesagt. Ich finde den Patch von rhabarber1848 so wie er jetzt ist völlig OK und sehe keinen Bedarf für Änderungen.

Ich empfinde dies

Wenn --with-archivedir genutzt wird und cdk/Archive ein existierender Link
ist, wird dieser entfernt und eine Infomeldung diesbezüglich ausgegeben.

Wenn --with-archivedir genutzt wird und cdk/Archive ein existierendes Verzeichnis
ist, wird eine Fehlermeldung ausgegeben und configure abgebrochen.

übrigens auch nicht als Entmündigung oder gar Schummellösung sondern eher als: Macht das was es soll. :wink:

P.S. Ich sehe mich persönlich nicht als Entwickler sondern eher als Heavy User. Meine Programmierkenntnisse sind die letzten Jahre über, dank eines völlig anderen Aufgabengebietes, sehr eingerostet.

[rhabarber1848]

mit Make-Variabel und -Substitutionen

Ich schau mal, was sich da machen lässt.
Ein neuer Punkt am Ende meiner To-Do-Liste

[rhabarber1848]

es richtig zu machen, mit Make-Variabel und -Substitutionen.

In etwa so? EDIT: Patch ist im CVS

Ich habe damit soeben Neutrino, Enigma und einige weitere
targets kompiliert, nachdem ich mein Archiv-Verzeichnis
geleert habe. Von meiner Seite ist der Patch CVS-tauglich.

[Barf]

Ich habe es nicht getestet, nur angeschaut. Seht i.O. aus. Daumen hoch!

[rhabarber1848]

committed:
http://article.gmane.org/gmane.comp.vid ... x.scm/1354
http://article.gmane.org/gmane.comp.vid ... x.scm/1355

[rhabarber1848]

--with-archivedir findet nun auch im VMWare-Howto Verwendung:
http://forum.tuxbox-cvs.sourceforge.net ... 05#p364805

[Striper]

Super. Danke für den Patch.