Skip to content

Fetch library ASP and include in member path for CCSID conversion#3370

Merged
sebjulliand merged 1 commit into
codefori:masterfrom
buzzia2001:fix/ccsidIasp
Jul 18, 2026
Merged

Fetch library ASP and include in member path for CCSID conversion#3370
sebjulliand merged 1 commit into
codefori:masterfrom
buzzia2001:fix/ccsidIasp

Conversation

@buzzia2001

Copy link
Copy Markdown
Member

Changes

During the save, the iasp was not included in the calculation of the IFS path, so run the attr command failed.

How to test this PR

  1. open a source member on iASP
  2. save it
  3. in output log now you should see something similar:
[2026-07-18 15:38:47.627] /usr/bin/attr -p /WHATANIASP/QSYS.LIB/SAMPLEIASP.LIB/DUMMYSRCPF.FILE CCSID
[2026-07-18 15:38:47.930] {
    "code": 0,
    "signal": null,
    "stdout": "CCSID=280",
    "stderr": ""
}

Checklist

  • have tested my change
  • have created one or more test cases
  • updated relevant documentation
  • Remove any/all console.logs I added
  • have added myself to the contributors' list in CONTRIBUTING.md

@buzzia2001
buzzia2001 requested a review from a team July 18, 2026 13:39
@buzzia2001 buzzia2001 self-assigned this Jul 18, 2026
@buzzia2001 buzzia2001 added the bug A confirmed issue when something isn't working as intended label Jul 18, 2026

@sebjulliand sebjulliand left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch, thanks @buzzia2001 !

@sebjulliand
sebjulliand merged commit 15f31d6 into codefori:master Jul 18, 2026
1 check passed
@buzzia2001
buzzia2001 deleted the fix/ccsidIasp branch July 20, 2026 11:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug A confirmed issue when something isn't working as intended

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants