Bugfix/drop isnull - Githubissues

atecon commented 5 years ago

replace isnull() by exists() function
create 'src' folder where new code is located
put old versions into the archive folder
update the spec-file and compile the new gfn-file

atecon commented 5 years ago

Hi nochmal, also ich kann nicht wirklich (im Webinterface von github) sehen, was Du in den .inp-Dateien geändert hast. Irgendwie scheint sich die Editierung der Dateien mit der Verschiebung ins neue src-Verzeichnis beim commit zu vermischen. Wahrscheinlich hätte man das in mehrere commits aufteilen müssen. Egal. Ich glaube Dir natürlich, dass Du das gemacht hast, was Du schreibst, aber wirklich prüfen kann ich es eigentlich nicht. Soll ich dann einfach mehr oder weniger blind freigeben?

Ah, ja, stimmt. 2 commits wären da angebracht gewesen. Ich habe gerade noch mal "manuell" die "FEP_public.inp" Dateien verglichen. Die 2 Änderungen umfassen einzig die Blöcke:

if isnull(fpath) fpath = "display" endif if !exists(fpath) fpath = "display" endif

Ja, wäre wohl ausreichend, wenn du den pull request einfach genehmigst. Dann kannst du ja auch lokal noch mal das neue Paket bei dir testen.

atecon commented 5 years ago

Hi Artur, mir ist nicht klar, warum Du als Grundlage von src/FEP.lyx die Backup-Datei FEP_2.2/FEP.lyx~ genommen hast (also die mit der Tilde, statt die normale .lyx).

Hi Sven, das ist mir gar nicht aufgefallen. Aber ich habe gerade noch mal "FEP.lyx" mit "FEP.lyx~" mittels meld verglichen: Der Unterschied ist einzig ein Kommata. Denke, dass kann man vernachlässigen.

atecon / FEP

Bugfix/drop isnull #1