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

v1.1.0-beta-01: TRACE N behaves like TRACE E instead of TRACE F #85

Closed
RossPatterson opened this issue Nov 4, 2024 · 2 comments
Closed
Assignees
Labels
bug Something isn't working Fixed in release 1.1.0

Comments

@RossPatterson
Copy link
Collaborator

RossPatterson commented Nov 4, 2024

Martin Scheffler reported this on the h390-vm list:

I have installed the binary distribution via VMARC on the MECAFF user. There is a change how REXX tracing works which is disturbing EENEW's internal machinery. My workaround is to begin all REXX programs with "TRACE 'Off'".

TRACE 'Negative' behaves the same as TRACE 'Errors' but these should differ. The new bREXX has changed in the way, that TRACE 'Negative' does trace host commands with a positive return code but these should not be traced by TRACE 'Negative'.

I have checked this on SDFVM's VM/SP5 .

@RossPatterson
Copy link
Collaborator Author

TRACE 'Negative' behaves the same as TRACE 'Errors' but these should differ. The new bREXX has changed in the way, that TRACE 'Negative' does trace host commands with a positive return code but these should not be traced by TRACE 'Negative'.

I did indeed change stuff in the area of host command return code processing, because Cowlishaw's Rexx separates SIGNAL ON FAILURE (negative RC) from SIGNAL ON ERROR (non-zero RC). I wasn't aware of TRACE NEGATIVE, and in fact it isn't documented in The Rexx Language 2nd ed. (TRL2) or in the ANSI Rexx Standard. Both of those have TRACE NORMAL, and since they both say to only test the first character (i.e., "N"), TRACE NEGATIVE is legal. TRL2 documents TRACE NORMAL as the same as TRACE FAILURE - i.e., display commands with negative return codes.

So, yes, you're right, TRACE N should behave the same as TRACE F, not like TRACE E.

@RossPatterson RossPatterson self-assigned this Nov 4, 2024
@RossPatterson RossPatterson added the bug Something isn't working label Nov 4, 2024
@RossPatterson
Copy link
Collaborator Author

Fix is complete and will be part of v1.1.0-beta-02.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Fixed in release 1.1.0
Projects
None yet
Development

No branches or pull requests

1 participant