https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37038

--- Comment #4 from Janusz Kaczmarek <[email protected]> ---
(In reply to Tomás Cohen Arazi from comment #3)
> Shouldn't it be
> 
> if [ "${clo_commit_size}" -gt "0" ]; then
> 
> ?

Thanks, Tomás, for having a look.

Well, yes, and it was my first shot.  It would be OK, provided that
clo_commit_size is defined and is a integer number.

Otherwise we get:

./debian/scripts/koha-elasticsearch: 93: [: Illegal number:

I have tried several options (including nested ifs) -- the proposed solution
seems to be a reasonable, working trade-off.  But I'm not a bash expert ;)

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to