diff options
| author | Shulhan <ms@kilabit.info> | 2023-10-12 01:30:19 +0700 |
|---|---|---|
| committer | Shulhan <ms@kilabit.info> | 2023-10-12 01:40:22 +0700 |
| commit | 207e7f016bf96ab037f510837940c037111cd42f (patch) | |
| tree | 1f28958f818da44ebcf6548bb081418e60fb11c2 /awwan.go | |
| parent | 6b756d144c5cdb0a0c9b5f8a70b3c936a75f28c9 (diff) | |
| download | awwan-207e7f016bf96ab037f510837940c037111cd42f.tar.xz | |
all: return error from executeScriptOnLocal
Instead of breaking the statement when its failed to execute, return
the error to the caller, Local.
Diffstat (limited to 'awwan.go')
| -rw-r--r-- | awwan.go | 13 |
1 files changed, 8 insertions, 5 deletions
@@ -207,9 +207,9 @@ func (aww *Awwan) Local(req *Request) (err error) { return fmt.Errorf("%s: %s: %w", logp, ses.dirTmp, err) } defer func() { - err = os.RemoveAll(ses.dirTmp) - if err != nil { - log.Printf("%s: %s", logp, err) + var errRemove = os.RemoveAll(ses.dirTmp) + if errRemove != nil { + log.Printf(`%s: %s`, logp, errRemove) } }() @@ -217,10 +217,13 @@ func (aww *Awwan) Local(req *Request) (err error) { for _, pos = range req.lineRange.list { err = ses.executeRequires(req, pos) if err != nil { - return fmt.Errorf("%s:%w", logp, err) + return fmt.Errorf(`%s: %w`, logp, err) } - ses.executeScriptOnLocal(req, pos) + err = ses.executeScriptOnLocal(req, pos) + if err != nil { + return fmt.Errorf(`%s: %w`, logp, err) + } } return nil |
