Subject: Re: [SM-Commit] GIT changes to master sorcery by Ismael Luceno (85b671af5ddb30d7a1c5682b3d9b5ee73d54ae8f)
Date: Thu, 24 Sep 2020 21:46:30 +0200
Am Tue, 15 Sep 2020 15:39:28 +0000
schrieb Ismael Luceno <scm AT sourcemage.org>:
> + if grep -qs "^$spell$" "$FAILED_LIST"; then
> + message "${MESSAGE_COLOR}The spell ${SPELL_COLOR}$spell$MESSAGE_COLOR"
> \
> + " failed to cast. Remembering that.$DEFAULT_COLOR"
> + rc=1
> + fi
> +
This needs a revision. I've seen, on a box I am still upgrading, that
pre_sub_depends as reason gets you into the FAILED_LIST. So just the
requirement to re-build a dependency because sub deps are not satisfied
in the installed config.
I added that exclusion to the grep in a copy of sorcery on that box.
I'll try to push the change to my branch soon-ish.
Maybe you can think of other cases where an entry in FAILED_LIST is not
serious enough to consider a cast failed.