]> git.koha-community.org Git - koha.git/commit
Bug 17532: Make koha-shell propagate the error code
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Thu, 13 Feb 2020 11:03:36 +0000 (12:03 +0100)
committerLucas Gass <lucas@bywatersolutions.com>
Thu, 9 Apr 2020 19:36:17 +0000 (19:36 +0000)
commitc779db2d75d2808b8a4b36f53bf467ec8dd5a33d
tree07dbe162ba5f86ec82a31de87aafb34535c23906
parent97df645697f238f503d6afe1105d203eacd6fc51
Bug 17532: Make koha-shell propagate the error code

Context: I am trying to make koha-misc4dev stop when something wrong happens.
Basically we do not want the process to continue of the DB has not been populated correctly.
It will make the errors easier to catch.

The issue: Say a script that will return an error (die in perl for an easy example),
koha-shell will not return the 255 error code, but 0 instead.
The caller cannot know something wrong happened

The solution is to propagate the error and make koha-shell return the same error code
as the command it executed

An example:
=== t.pl ===
die('something wrong');

% perl t.pl;echo $?
something wrong at t.pl line 1.
255

% sudo koha-shell kohadev -p -c 'perl xxx.pl' ; echo $?
something wrong at xxx.pl line 1.
0

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Tested it with
  $ sudo debian/scripts/koha-shell kohadev -c 'exit 8'
  $ echo $?
  8
The unpatched version returns 0 (forgets about the error code).

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Joy Nelson <joy@bywatersolutions.com>
(cherry picked from commit 5d22e0b3a331e35b8f1e494bc366c1624c982cf8)

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
debian/scripts/koha-shell