Skip to content

Basic 'starter support' for PL/I #16

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
Draft
wants to merge 23 commits into
base: main
Choose a base branch
from
Draft

Basic 'starter support' for PL/I #16

wants to merge 23 commits into from

Conversation

MikeFultonDev
Copy link
Collaborator

Requires a PL/I modified front end that prints out dependencies
See https://github.com/ZOSOpenTools/cmakeport/blob/altcmp/tests/basictest for what is expected to work with the modified front end.

Mike Fulton added 22 commits December 1, 2022 00:21
getting started with PL/I experiment
d6e0a52
basic starter for describing PL/I
c7ed102
fix some typos and add new patch file
379a5f7
fix some typos
e68f3aa
switch order of parms around
ff0bb26
remove -o object for compile
3b1501a
compile and link not quite right
80ffa37
add in a pli.cmake file
e25fbc4
fix compilation errors in source
9b2f118
tweaks to improve PL/I ... added -I option but still doesn't manage d…
03f27db 
…ependencies
create simple hello-world for C to do tracing
603627e
get PL/I compiler to determine version info
64cedd1
add version info for the PL/I compiler
606db7e
switch to ninja
a426cb3
use ninja
c4c917f
try a different approach akin to zoslib
8db5310
switch to clang compiler and use better names for the scripts
aa3f3f8
add basic test to validate that dependency (touch) of C and H files w…
94bed44 
…orks
complete up config script
b301cca
fix up code for dep specification
1c9d2f7
forgot quotes
7a99255
move CompilerId.cmake partial patch into separate patch file outside …
91a2afb 
…zos-v3.24.2.patch along with other patch
@MikeFultonDev MikeFultonDev added the enhancement New feature or request label Dec 24, 2022
perry-ca
perry-ca reviewed Jan 3, 2023
View reviewed changes
@MikeFultonDev MikeFultonDev mentioned this pull request Jan 4, 2023
Open
perry-ca
perry-ca approved these changes Jan 6, 2023
View reviewed changes
Copy link
Collaborator

@perry-ca perry-ca left a comment

Choose a reason for hiding this comment

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

LGTM

@MikeFultonDev MikeFultonDev marked this pull request as draft January 31, 2024 17:10
@MikeFultonDev
Copy link
Collaborator Author

@perry-ca @IgorTodorovskiIBM converting this to draft and unassigning myself.
I expect much has changed in this area, but the patches might be useful if/when we resurrect this idea.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@perry-ca perry-ca perry-ca approved these changes

@IgorTodorovskiIBM IgorTodorovskiIBM Awaiting requested review from IgorTodorovskiIBM IgorTodorovskiIBM is a code owner

@fanbo-meng fanbo-meng Awaiting requested review from fanbo-meng fanbo-meng is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@MikeFultonDev @perry-ca