NLCR / komplexni-validator

Komplexní validátor
http://www.ndk.cz/aktuality/novinky-titulni-strana/seminar-komplexni-validator-v-praxi
Apache License 2.0
8 stars 3 forks source link

Wiki - Instalace - návod #39

Open mirek-khodl opened 7 years ago

mirek-khodl commented 7 years ago

Rád bych doporučil změnu v odkazování na zdroje ve Wiki instalačním návodu.

Příklad: v době psaní byla pravděpodobně určitá verze ImageMagicu (platí ale obecně) ta, na kterou odkaz vede. Velmi pravděpodobně obsahovala chyby, pro kterou byl daný build rychle nahrazen jiným...

Pokud tedy vývojář odladil funkce svého produktu na omezené a dobře definované množině zájmových parametrů pluginu, mohl by odkaz na zdroj formou názvu platformy a balíku + cestu ke zdroji na úrovni statické url k adresáři buildů postačit, současně se tak předejde situaci, kdy link přímo na exe soubor vyvolá chybové hlášení o neexistenci zdroje.

Konkrétně:

Stáhněte se instalátor z https://www.imagemagick.org/download/binaries/ImageMagick-7.0.3-9-Q16-x64-dll.exe, případně jinou verzi (32b systém, staticky linkované knihovny). Spusťte instalátor a následujte instrukce.<<

Verze sama již není v nabídce, x64 v názvu ve srovnání se závorkovým doplněním u mne vede k logické kolizi ;), nevím, co mi tedy autor textu doporučuje jako plně funkční vyzkoušenou verzi. Spokojím se s jednou, k níž se pak mohou vztahovat i poznatky ostatních uživatelů.

Sám jsem občasným uživatelem Win verze "ImageMagick-7.0.5-Q16-HDRI" a validátor hlásí, že mi chybí JP2 knihovna, která je tam, kde být má... Najde ji i v cmd. Také cesta ve Validátoru je nastavena správně, kontrolka svítí zeleně ;)

Možná tedy další problém s mezerami v cestě? My kontroly na obrazy vypneme, jsou pro dávkové kontroly extrémně náročné na čas,, a protože děláme binární komparace nekomprimovaných dat mezi formáty přímo během zpracování, byla by to duplicitní činnost, nicméně někomu by se mohlo toto upozornění hodit.

Originální hlášení validátoru: checkImageFilesValidByExternalUtil: no_errors: zakázaná hodnota nalezena: identify: unable to load module 'C:\Program Files\ImageMagick-7.0.5-Q16-HDRI\modules\coders\IM_MOD_RLJP2.dll': Uvedený modul nebyl nalezen. @ error/module.c/OpenModule/1279. .

FoxKyong commented 7 years ago

Já jsem taky narazil na chybu, zase ke spuštění validátoru. Na wiki je napsán příklad pro psp, zip a skupinu psp balíčků, ale není není v tom příkladu uveden parametr --action VALIDATE_PSP nebo --action VALIDATE_PSP_GROUP, bez kterého ten validátor nejde spustit a píše, že to je povinný parametr, což podle popisu instalace není.

FoxKyong commented 7 years ago

A ještě k tomu openjfx, co se mi na wiki moc nelíbí, dopsal jsem to k tomu closed issue, ale když je tady další issue na popis instalace, kopíruji i tady:

V dokumentaci by mělo být uvedeno pouze, že je potřeba openjfx, případně uvedeno, že apt-get je pouze u distribucí, které používají .deb balíčky a to nevím, jestli všechny distribuce tu verzi mají stejně pojmenovanou. Jak jsem psal v komentáři, na Arch Linuxu je instalace "pacman -S java-openjfx", takže je tam jiný postup instalace i jiný název. Není podle mě dobré to psát takhle, jako by se předpokládalo, že každý má pouze Debian/Ubuntu. U jiných software je většinou vždy postup instalace uveden pro různé typy distribucí, tak to podle mě nevypadá takhle dobře.

godnat commented 7 years ago

Dobrý den,

děkujeme za upozornění, upravíme.

rzeh4n commented 5 years ago

@FoxKyong

  1. Pokud jde o parametr --action, informace na WIKI byly zastaralé, povinný parametr action přibyl ve verzi 1.5. Na příkladech ve Wiki jsem parametr doplnil.
  2. To, že je problém ve chybějící javaFx tam uvedeno je. A pak příklad, jak se to dát nejčastěji doinstalovat. Dobře, trochu jsem tu formulaci upravil, ale určitě tady nebude aktuální návod instalace JavaFx pro každou Linuxovou distribuci.

Ještě takový technický dotaz - jedno issue by rozhodně nemělo řešit 3 různé problémy, i když nějakým způsobem si blízké. Příště bych vás poprosil pro každý problém založit nové issue. Díky

rzeh4n commented 5 years ago

@mirek-khodl Teď je to na Wiki formulováno jinak a není tam odkaz přímo na konkrétní verzi ImageMagick.

případně jinou verzi (32b systém, staticky linkované knihovny)

Myslel jsem to tak, že případně jinou verzi, třeba 32b alternativu, nebo třeba alternativní verzi se staticky linkovanými knihovnami. Oproti doporučované 64b dll.

Ale ano, asi to upravíme ve smyslu "otestováno s verzí xyz".