Test ioBroker Fixer mit JSONL-Compression

[mcm1957]

@apollon77

Auch interessant das der verlinkte artikel davon redet das er ein "fix.bat" finden müsste aber scheinbar ists jetzt "iobroker.bat"

Lt. npm exec docu (https://docs.npmjs.com/cli/v7/commands/npx) gilt:

If no -c or --call option is provided, then the positional arguments are used to generate the command string. If no --package options are provided, then npm will attempt to determine the executable name from the package specifier provided as the first positional argument according to the following heuristic:

If the package has a single entry in its bin field in package.json, or if all entries are aliases of the same command, then that command will be used.
If the package has multiple bin entries, and one of them matches the unscoped portion of the name field, then that command will be used.
If this does not result in exactly one option (either because there are no bin entries, or none of them match the name of the package), then npm exec exits with an error.

Da es genau einen bin Eintrag gibt und der iobroker heißt nimmt npx den Befehl "iobroker". Nur nimmt es dann offenbar nicht den Eintrag in package.json als erste Wahl sondern geht mal mit "iobroker" den PATH abklappern. Und da findet es iobroker.bat.

Eigentlich eine perverse Logic - scheint aber so zu sein. Und npm/npx werden wir schwer ändern können. Ev. sollte man das Command e289hjvzufgcru657otß nennen - da ist eine Kollisio unwahrscheinlicher

[Gaspode]

@mcm57 said in Test ioBroker Fixer mit JSONL-Compression:

In jedem Fall gibts einen Fehler in iob.bat und iobroker.bat wenn keine Parameter angegeben werden. Das wär zu fixen. Gibts dazu schon ein Issue?

Das ist bereits seit gestern gefixt.

[Gaspode]

@mcm57 said in Test ioBroker Fixer mit JSONL-Compression:

Wie erhältst du den Screenoutput? Bei mir löst irgendwas ein Clear Screen aus sodass ich am Ende nur die Zeieln 36 bis 40 deines Outputs sehe...

Sorry, eben erst die Frage gesehen. einfach in Datei umleiten:

npx @iobroker/fix > meinLog.txt

[mcm1957]

@gaspode
Danke.

Weißt du wer das die ClearScreens absetzt? Wär ev. sinnvoll wenn man die verhindern kann.
Oder wenn iob fix gleich ein Logfile schreibt. Ein DAU wird möglicherweise nicht an die Umleitung denken. Ich wollt sie eigentlich auch vermeiden

Zusatz:
Gibts unter Windows ein Äquivalent zu tee unter Linux? Wär ja schön wenn man den output auch live sehen kann - insbesondere wenn's mal wo klemmt.

[apollon77]

@mcm57 warum sollte ein User das im Normalfall brauchen? macht’s nicht so kompliziert

Ansonsten habe wir entschieden mit js-Controller 5 später dieses Jahr die iobroker.bat zu entfernen. Bis dahin ist’s halt noch nicht gescheit funktional. Ist akzeptabel.

[mcm1957]

@apollon77
Was brauchen?
Einen Output des Fixerlaufs? Ich denk schon den würde ein User sehen wollen.
Spätestens wenn er den Outpu des Fixerlaufs posten soll

Und iobroker.bat zu streichen find ich nicht so toll. Aber das ist meine Meinung. Das wird sicher verwirren da dann der Befehl iobroker nicht mehr geht - oder versteh ich da was falsch? Und im Ergebnis wird dann der User ein iobroker.bat anlegen und dann werden erst wieder Fragen auftreten ...

Aber mir wird eh nachgesagt dass ich Probleme kreiere. ergo versuch ich mal still zu sein
Ev. können ja andere Windows User sagen ob / wieoft sie den Befehl iobroker auf der Commandline verwenden.

[apollon77]

@mcm57 wir haben ja noch iob.bat. Die bleibt ;-)) aber ja die interessante Frage ist was die windows User dazu sagen…

[Gaspode]

@apollon77 said in Test ioBroker Fixer mit JSONL-Compression:

aber ja die interessante Frage ist was die windows User dazu sagen…

Ich befürchte, die meisten davon haben diesen Thread noch nicht entdeckt.

Eine Alternative wäre es u.U. auch, unter Windows soweit möglich auf npx zu verzichten.
Der Fixer lässt sich ja so starten:

npm install @iobroker/fix
node node_modules\@iobroker\fix\lib-npx\install.js fix 

Evtl. könnte man ja bei der Installation die benötigten Packages installieren und die entsprechenden node Aufrufe in BAT Files kapseln. in iobroker.bat gibt es ja bereits ein if für den Fixer. Da müsste man den npx Aufruf dann entsprechend ersetzen. Was bräuchte man denn noch? @iobroker/install ist ja noch kein Problem, da vor dem Installieren noch kein iobroker.bat vorhanden ist (trotzdem könnte man das ja in iobroker.bat aufnehmen).

Zum Logging auf der Shell:
@apollon77 said in Test ioBroker Fixer mit JSONL-Compression:

macht’s nicht so kompliziert

Ich fände es auch übersichtlicher, wenn der Screen nicht während einem Scriptlauf gelöscht würde. Ist ja nur ein CLS in iobrokerservice.bat. Ich würde sagen, das hat keinen Mehrwert und man könnte es einfach rauswerfen. Ich habe einen entsprechenden PR angelegt, falls ihr das ähnlich seht.

[apollon77]

@gaspode sagte in Test ioBroker Fixer mit JSONL-Compression:

Ich befürchte, die meisten davon haben diesen Thread noch nicht entdeckt.

Ja das Fragen wir nochmals dedizierter ab

[mcm1957]

@apollon77
Na ja iob ist schon ok. Ich frag mich aber schon warum iobroker.bat entfernt werden soll nur weil der fixer nicht geht. Ich hätte als erstes versucht das Command in package.json umzubenennen.

Zumindest hier hab ich nichts davon gelesen
-) dass das nicht sinnvoll ist, das es dort oder dort Sideeffekts hätte
-) dass das irgendwie / wo getestet wurde (und nicht funktioniert hat)

Ich (und sicher auch @Gaspode) würden eine Version in der statt iobroker z.b iob-install oder irgendwas andere drinnen steht jederzeit testen.

@Gaspode
Ich habe mal deinen PR upgevotet. Abgesehen von üblichen Reviews an solchen Kernkomponenten wie admin sollte der PR asap gemerged werden da ein cls wenige hier absolut Sinn macht und meiner Ansicht nach null negative Auswirkungen haben kann.

---

An alle:
Weiß jemand wie man ein npx bauen / starten in einer privaten Umgebung starten kann? Ich könnte ja iobBorker/ioBroker forken und an package.json "rumspielen". Nur wie komm ich dann zu einem npx fähigen paket???

[Gaspode]

@mcm57 said in Test ioBroker Fixer mit JSONL-Compression:

Ich habe mal deinen PR upgevotet. Abgesehen von üblichen Reviews an solchen Kernkomponenten wie admin sollte der PR asap gemerged werden da ein cls wenige hier absolut Sinn macht und meiner Ansicht nach null negative Auswirkungen haben kann.

Ja, ist jetzt aber doch nicht so dringend. Die neue Installationsmethode für Windows ist ja noch im Testbetrieb. Ich denke, da gibt es aktuell wichtigere Baustellen, um die Sache rund zu bekommen.

[mcm1957]

@gaspode
Ja sie ist im Testbetrieb.
ABER sie ist meines Wissens nach die einzige die derzeit funktioniert (unter Windows )
Außerdem heißt gemerged ja nicht sofort eine neue Release ...

[Gaspode]

@mcm57 said in Test ioBroker Fixer mit JSONL-Compression:

ABER sie ist meines Wissens nach die einzige die derzeit funktioniert (unter Windows )

Nö, der "alte" von Stabilostick ist noch nutzbar, wenn man weiß wie (Beschreibung steht mehrfach im Forum und funktioniert noch, gerade letzte Woche nochmal getetstet).

@mcm57 said in Test ioBroker Fixer mit JSONL-Compression:

Außerdem heißt gemerged ja nicht sofort eine neue Release ...

Dann bekommt es ja aber niemand mit und somit ist das Ergebnis das Gleiche.

[mcm1957]

@gaspode said in Test ioBroker Fixer mit JSONL-Compression:

@mcm57 said in Test ioBroker Fixer mit JSONL-Compression:

ABER sie ist meines Wissens nach die einzige die derzeit funktioniert (unter Windows )

Nö, der "alte" von Stabilostick ist noch nutzbar, wenn man weiß wie (Beschreibung steht mehrfach im Forum und funktioniert noch, gerade letzte Woche nochmal getetstet).

OK, dann zieh ich die Anmerkung zurück. Ich hatte im Kopf, dass der alte nur mit veraltetem node funktioniert und der node upgrade dann klemmt. Oder sowas ähnliches. Hab ich so hier im Forum mitbekommen.

Wenn er noch funktioniert soll es mir natürlich recht sein. Sorry für die Falschmeldung.

[mcm1957]

@mcm57 said in Test ioBroker Fixer mit JSONL-Compression:

An alle:
Weiß jemand wie man ein npx bauen / starten in einer privaten Umgebung starten kann? Ich könnte ja iobBorker/ioBroker forken und an package.json "rumspielen". Nur wie komm ich dann zu einem npx fähigen paket???

Teilweise kann ich es mir beantworten:

npx github:iobroker/iobroker scheint ident mit npx @iobroker/fix zu sein. Es kommt jedenfalls die ")" Fehlermeldung.

Forke ich das iobroker/iobroker repo, so erhalte ich mit npx mcm1957/iobroker dasselbe ergebnis, d.h. die ")" Fehlermeldung.

Wenn ich nun in mcm1957/iobroker das bin command ändere in

"bin": {
    "iobnpx": "./lib-npx/install.js"
  },

bekomm ich auf einmal die Fehlermeldung

C:\ioBroker>npx github:mcm1957/iobroker
Der Befehl "iobnpx" ist entweder falsch geschrieben oder
konnte nicht gefunden werden.

Offenbar ermittelt npx das command wie erwartet. Nur warum npx das command dann nicht findet ist mir schleierhaft.

[Dr. Bakterius]

@gaspode sagte in Test ioBroker Fixer mit JSONL-Compression:

Ich befürchte, die meisten davon haben diesen Thread noch nicht entdeckt.

Dann sollte man diesen Thread vielleicht splitten. Da gibt es jetzt einige Beiträge die sich alleine mit dem Windows-Problem beschäftigen. Davor waren aber die meisten nur für Linux gültig. Da scheuen Windows-User kurz rein und meinen, dass sie das nichts angeht.

[apollon77]

@mcm57 aus produkt strategischen Gründen ist halt @iobroker/install Bzw /fix durchaus zu bevorzugen

Und im Daily Business ist doch iob.bat viel praktischer.

[mcm1957]

@apollon77
Ja. Für den User Input ist dass schon OK und sollte auch so bleiben.

Nur ich seh den Zusammenhang zwischen iobroker/fix bzw iobroker/install und dem Eintrag im package nicht. "fix" steht - wenn ich mich nicht irre - gar nirgends in package.json.

Steh wohl irgendwie auf der Leitung. Muss mich da mal genauer einarbeiten / einstudieren. Aber das ist hier mal OT

Ansonsten würd ich nach wie vor eine Lösung bevorzugen die von Dateinamen unabhängig sind. Sonst suchen wir uns zum Elch wenn bei einem User ws nicht geht und dann der Fixer auch meckert und alles nur weil er sich ein harmlos scheinendes iobroker.bat (oder etwas anderes) wo im Pfad abgelegt hat. Das muss ja nicht mal im cwd sein - wahrscheinlich reicht es irgendwo im Pfad (hab das aber nicht getestet)

Aber wie schon gaspode geschrieben hat - wir wissen was passiert. Das reicht im Moment mal. Ist ja kein 1A prio Problem. Ev. gibts ja wo nen npm / node / windows Fachmann der die Lösung aus dem kleinen Finger schüttelt.

[apollon77]

@mcm57 sagte in Test ioBroker Fixer mit JSONL-Compression:

"fix" steht - wenn ich mich nicht irre - gar nirgends in package.json.

Das passt schon beim Deployment wird das paket quasi doppelt deployes mot einem name change weil es der gleiche Code ist der sich bei fix nur leicht anders verhält.