Merge ~dirk.zimoch/epics-base:epicsEnvUnset into ~epics-core/epics-base/+git/epics-base:3.15
Proposed by
Dirk Zimoch
Status: | Merged |
---|---|
Approved by: | Ralph Lange |
Approved revision: | 3cb72ec2091bd3e03b742ee3d8e8da426c09b210 |
Merge reported by: | Ralph Lange |
Merged at revision: | 3cb72ec2091bd3e03b742ee3d8e8da426c09b210 |
Proposed branch: | ~dirk.zimoch/epics-base:epicsEnvUnset |
Merge into: | ~epics-core/epics-base/+git/epics-base:3.15 |
Diff against target: |
225 lines (+117/-0) (has conflicts) 9 files modified
documentation/RELEASE_NOTES.html (+6/-0) src/libCom/env/envDefs.h (+1/-0) src/libCom/iocsh/libComRegister.c (+16/-0) src/libCom/osi/os/Darwin/osdEnv.c (+10/-0) src/libCom/osi/os/default/osdEnv.c (+12/-0) src/libCom/osi/os/iOS/osdEnv.c (+10/-0) src/libCom/osi/os/vxWorks/osdEnv.c (+19/-0) src/libCom/test/Makefile (+5/-0) src/libCom/test/epicsEnvUnsetTest.c (+38/-0) Conflict in documentation/RELEASE_NOTES.html |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Ralph Lange | Approve | ||
Review via email: mp+356070@code.launchpad.net |
Description of the change
Allow to unset variables:
epics> echo $(ABC=default)
default
epics> epicsEnvSet ABC 123
epics> echo $(ABC=default)
123
epics> epicsEnvSet ABC ""
epics> echo $(ABC=default)
epics> epicsEnvUnset ABC
epics> echo $(ABC=default)
default
Tested on Linux and vxWorks 5.
To post a comment you must log in.
I would like to see a unit test go with that.
(For a while, the rule has been "new features must have tests - if feasible with reasonable effort".)
Also, a paragraph in the release notes would be good, and maybe a matching one in the App Developer's Guide to the Galaxy.