Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Zero-length environment variables reported as null #786

Closed
tiymat opened this issue Jan 8, 2024 · 1 comment · Fixed by #785
Closed

Zero-length environment variables reported as null #786

tiymat opened this issue Jan 8, 2024 · 1 comment · Fixed by #785
Labels
bug Unexpected behavior deployed to `develop` Feature built. Currently BETA testing in the `develop` branch

Comments

@tiymat
Copy link
Contributor

tiymat commented Jan 8, 2024

Describe the bug:
is-null reports environment variables set to zero-length strings as null, inconsistent with its treatment of local and global variables.

Expected behaviour:
Environment variables set to the empty string should be treated by is-null as non-null.

Screenshots:

~ » set x=""
~ » global y=""
~ » export z=""
~ » is-null x y z
x: defined and not null
y: defined and not null
z: undefined or null

Platform (please complete the following information):

  • OS, output from uname -a if supported: Ubuntu 23.04
  • Terminal Emulator: GNOME Terminal
  • Murex version, output from version --no-app-name: 5.4.1000
@tiymat tiymat added the bug Unexpected behavior label Jan 8, 2024
@lmorg lmorg added the in progress Issue is currently being worked on (possibly in a feature branch) label Jan 9, 2024
@lmorg
Copy link
Owner

lmorg commented Jan 9, 2024

Another good spot. Just pushed a fix up to develop

@lmorg lmorg added the deployed to `develop` Feature built. Currently BETA testing in the `develop` branch label Jan 9, 2024
@lmorg lmorg linked a pull request Jan 9, 2024 that will close this issue
@lmorg lmorg mentioned this issue Feb 17, 2024
@lmorg lmorg removed the in progress Issue is currently being worked on (possibly in a feature branch) label Jul 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Unexpected behavior deployed to `develop` Feature built. Currently BETA testing in the `develop` branch
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants