benibela / xidel

Command line tool to download and extract data from HTML/XML pages or JSON-APIs, using CSS, XPath 3.0, XQuery 3.0, JSONiq or pattern matching. It can also create new or transformed XML/HTML/JSON documents.
http://www.videlibri.de/xidel.html
GNU General Public License v3.0
674 stars 42 forks source link

[Bug] r8308 tarball doesn't include PasDblStrUtils #83

Closed Reino17 closed 2 years ago

Reino17 commented 2 years ago

r8285:

$ curl -OL "https://sourceforge.net/projects/videlibri/files/Xidel/Xidel%20development/xidel-0.9.9.20211225.8285.b90e197a0a24.src.tar.gz"

$ tar -xf xidel-0.9.9.20211225.8285.b90e197a0a24.src.tar.gz

$ cd xidel-0.9.9.20211225.8285.b90e197a0a24-src/

$ programs/internet/xidel/build.sh -s
-- Building...
Free Pascal Compiler version 3.2.2 [2021/05/15] for i386
Copyright (c) 1993-2021 by Florian Klaempfl and others
Target OS: Win32 for i386
[...]
-- Done.

Everything fine.

r8308 on the other hand (and probably because of "default to using Bero's double parser for JSON"):

$ curl -OL "https://sourceforge.net/projects/videlibri/files/Xidel/Xidel%20development/xidel-0.9.9.20220102.8308.be361bf9a08e.src.tar.gz"

$ tar -xf xidel-0.9.9.20220102.8308.be361bf9a08e.src.tar.gz

$ cd xidel-0.9.9.20220102.8308.be361bf9a08e-src/

$ programs/internet/xidel/build.sh -s
-- Building...
Free Pascal Compiler version 3.2.2 [2021/05/15] for i386
Copyright (c) 1993-2021 by Florian Klaempfl and others
Target OS: Win32 for i386
[...]
Compiling [...]\xidel-0.9.9.20220102.8308.be361bf9a08e-src\components\pascal\data\xquery.internals.common.pas
xquery.internals.common.pas(434,80) Fatal: Can't find unit PasDblStrUtils used by xquery.internals.common
Fatal: Compilation aborted

$ mkdir -p components/pascal/import/pasdblstrutils/src

$ curl -o components/pascal/import/pasdblstrutils/src/PasDblStrUtils.pas "https://raw.githubusercontent.com/BeRo1985/pasdblstrutils/master/src/PasDblStrUtils.pas"

$ programs/internet/xidel/build.sh -s
-- Building...
Free Pascal Compiler version 3.2.2 [2021/05/15] for i386
Copyright (c) 1993-2021 by Florian Klaempfl and others
Target OS: Win32 for i386
[...]
-- Done.

Of course you can also do git clone https://github.com/BeRo1985/pasdblstrutils.git, but as the testdata/parser map is a whopping 261MB(!) I figured this would be less "intrusive".


Related issue: Your mercurial repository is missing PasDblStrUtils, but also FLRE!

benibela commented 2 years ago

now it is there

Related issue: Your mercurial repository is missing PasDblStrUtils, but also FLRE!

I do not include them because they were already on github

Reino17 commented 2 years ago

Thanks. Compiles without errors now.

Fair enough.