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.

Sign up for GitHub

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

Jump to bottom

Update the dynamic linkage code to expose new cmutils functions #740

Merged
merged 1 commit into from
Jan 28, 2025
Merged

Update the dynamic linkage code to expose new cmutils functions #740

merged 1 commit into from
Jan 28, 2025

Conversation

ifakhrutdinov
Copy link
Contributor

@ifakhrutdinov ifakhrutdinov commented Dec 12, 2024 •
edited
Loading

Proposed changes

This PR updates the corresponding dynamic linkage code to pick up the new zowe-common-c/c/cmutils.c functions along with the changes addressing EXECUTABLE=YES storage.

This PR depends upon the following PRs:

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

PR Checklist

Please delete options that are not relevant.

  • If the changes in this PR are meant for the next release / mainline, this PR targets the "staging" branch.
  • My code follows the style guidelines of this project (see: Contributing guideline)
  • Relevant update to CHANGELOG.md
  • My changes generate no new warnings

Testing

Regression testing of ZIS is needed.

@ifakhrutdinov ifakhrutdinov force-pushed the bugfix/non-exec-storage branch 2 times, most recently from 13a19ff to f3b0714 Compare December 12, 2024 12:42
@ifakhrutdinov ifakhrutdinov self-assigned this Dec 12, 2024
@ifakhrutdinov ifakhrutdinov marked this pull request as ready for review December 12, 2024 12:43
@ifakhrutdinov ifakhrutdinov force-pushed the bugfix/non-exec-storage branch from f3b0714 to 2b6db2a Compare December 13, 2024 09:46
@ifakhrutdinov ifakhrutdinov added the enhancement New feature or request label Dec 13, 2024
@ifakhrutdinov ifakhrutdinov force-pushed the bugfix/non-exec-storage branch from 2b6db2a to 99b253d Compare December 20, 2024 12:12
@ifakhrutdinov ifakhrutdinov force-pushed the bugfix/non-exec-storage branch from 99b253d to 7eb3fed Compare January 10, 2025 10:33
@ifakhrutdinov ifakhrutdinov force-pushed the bugfix/non-exec-storage branch from 7eb3fed to 277688b Compare January 22, 2025 14:50
JoeNemo
JoeNemo approved these changes Jan 28, 2025
View reviewed changes
Copy link
Contributor

@JoeNemo JoeNemo left a comment

Choose a reason for hiding this comment

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

Approved.

@ifakhrutdinov
Update dynlink to expose new cmutils functions; pick up EXECUTABLE=YE…
b24eaf1 
…S storage changes

Signed-off-by: Irek Fakhrutdinov <ifakhrutdinov@rocketsoftware.com>
@ifakhrutdinov ifakhrutdinov force-pushed the bugfix/non-exec-storage branch from 277688b to b24eaf1 Compare January 28, 2025 16:09
@JoeNemo JoeNemo merged commit bca9f12 into v3.x/staging Jan 28, 2025
1 check passed
@ifakhrutdinov ifakhrutdinov deleted the bugfix/non-exec-storage branch January 28, 2025 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JoeNemo JoeNemo JoeNemo approved these changes

Assignees

@ifakhrutdinov ifakhrutdinov

Labels
enhancement New feature or request
Projects
zOS Squad Board
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ifakhrutdinov @JoeNemo