loader command interpreter should reset command_errmsg
ClosedPublic
Actions

Authored by tsoome on Sep 26 2016, 8:11 AM.

Details

Reviewers

allanjude
imp

Commits

rS306380: loader command interpreter should reset command_errmsg

Summary

The error message from previous command is repeated in case the forth
word is defined as builtin.

Illumos issue: https://www.illumos.org/issues/7405

Test Plan

I did use the following to reproduce the issue in fbsd:
OK include /boot/pcibios.4th
OK lszfs <-- this will result with error message being printed
OK pci-device-count 1

last command will do it's output *and* will print error message from
lszfs.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 5321
Build 5505: arc lint + arc unit

Event Timeline

tsoome updated this revision to Diff 20700.Sep 26 2016, 8:11 AM

tsoome retitled this revision from to loader command interpreter should reset command_errmsg.

tsoome updated this object.

tsoome edited the test plan for this revision. (Show Details)

tsoome added reviewers: allanjude, imp.

Approved for commit

This revision is now accepted and ready to land.Sep 27 2016, 7:00 PM

Closed by commit rS306380: loader command interpreter should reset command_errmsg (authored by tsoome). · Explain WhySep 27 2016, 8:41 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

sys/

boot/

common/

interp_forth.c

8 lines

Diff 20700

View Options

sys/boot/common/interp_forth.c

loader command interpreter should reset command_errmsgClosedPublicActions