ts-hosts-allocate-Executive: Fix handling of failed preps for same sharing

This code was previously unreachable.  It ought to be executed when
all the shares are allocatable or prep: in that case, we can unshare
and re-share the host.

Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>

diff --git a/ts-hosts-allocate-Executive b/ts-hosts-allocate-Executive
index 78b94c6da1e31bb8ff0a57268b4cc2f6b82f69d1..2c18a739f54013489778108bfcf6a7dc60e35056 100755 (executable)
--- a/ts-hosts-allocate-Executive
+++ b/ts-hosts-allocate-Executive
@@ -1044,12 +1044,11 @@ sub actual_allocation ($) {
                # now-dead task.  If so that share will become allocatable
                # (and therefore not be counted in `ntasks') in due course.
                return undef;
-
-                # someone was preparing it but they aren't any more
-                push @allocwarnings,
-                    "previous preparation apparently abandoned";
-                $allocatable= 1;
-            }
+           }
+           # someone was preparing it but they aren't any more
+           push @allocwarnings,
+               "previous preparation apparently abandoned";
+           $allocatable= 1;
         }
     }