diff options
author | Sean Whitton <spwhitton@spwhitton.name> | 2021-07-05 16:29:46 -0700 |
---|---|---|
committer | Sean Whitton <spwhitton@spwhitton.name> | 2021-07-10 20:45:21 -0700 |
commit | 56bc5a2c24b0fe56c72ced9a5ac85d982d592567 (patch) | |
tree | d01d1820afeadd4d50794e45007bba9d9533b373 /src/deployment.lisp | |
parent | ec508517bc89d3934afa9ec91f3787839b14be2d (diff) | |
download | consfigurator-56bc5a2c24b0fe56c72ced9a5ac85d982d592567.tar.gz |
signal SKIPPED-PROPERTIES & factor out interpreting exit codes
Unconditionally signalling FAILED-CHANGE does not make sense because perhaps
the type of condition C is not a subtype of SIMPLE-CONDITION. Moreover, when
we invoke the SKIP-PROPERTY restart we do not actually pass the condition.
For simplicity, and since all we need is notification that a SKIP-PROPERTY
restart was invoked, instead define and signal a special-purpose condition.
Additionally, use an exit code to pass the signal between Lisp images.
Signed-off-by: Sean Whitton <spwhitton@spwhitton.name>
Diffstat (limited to 'src/deployment.lisp')
-rw-r--r-- | src/deployment.lisp | 16 |
1 files changed, 0 insertions, 16 deletions
diff --git a/src/deployment.lisp b/src/deployment.lisp index ce4fb95..a845168 100644 --- a/src/deployment.lisp +++ b/src/deployment.lisp @@ -91,22 +91,6 @@ will not be discarded." (make-propspec :propspec propspec-expression))) :collect-at-end collect-at-end)) -(defmacro with-deployment-report (&rest forms) - (with-gensyms (failures) - `(let (,failures) - (handler-bind ((failed-change (lambda (c) (setq ,failures t)))) - (let ((result (progn ,@forms))) - (inform - t - (cond - ((eql :no-change result) - "No changes were made.") - (,failures - "There were failures while attempting to apply some properties.") - (t - "Changes were made without any reported failures."))) - result))))) - (defun deploy* (connections host &optional additional-properties) "Execute the deployment which is defined by the pair (CONNECTIONS . HOST), except possibly with the property application specification |