Skip to content

Conversation

@jwoehr
Copy link
Contributor

@jwoehr jwoehr commented May 10, 2025

  • Make USRPRF an option in rules.mk
  • Update documentation to reflect this

- Update documentation to reflect this

Signed-off-by: jwoehr <[email protected]>
@robinsg
Copy link

robinsg commented May 11, 2025

This did not work for me but I got it working by amending the following: line:

CRTBNDCLFLAGS = AUT($(AUT)) USRPRF($(USRPRF)) DBGVIEW($(DBGVIEW)) OPTION($(OPTION)) TEXT('$(TEXT)') TGTRLS($(TGTRLS)) INCDIR($(INCDIR))

So, as well as changing the CRTPGMFLAGS I variable I needed to change the CRTBNDCLFLAGS variable too.

…lled in

 by MODULE_TO_PGM_RECIPE and not by PGM.CBLLE_TO_PGM_RECIPE.

Signed-off-by: jwoehr <[email protected]>
@jwoehr
Copy link
Contributor Author

jwoehr commented May 11, 2025

This did not work for me but I got it working by amending the following: line:

CRTBNDCLFLAGS = AUT($(AUT)) USRPRF($(USRPRF)) DBGVIEW($(DBGVIEW)) OPTION($(OPTION)) TEXT('$(TEXT)') TGTRLS($(TGTRLS)) INCDIR($(INCDIR))

So, as well as changing the CRTPGMFLAGS I variable I needed to change the CRTBNDCLFLAGS variable too.

You're right, @robinsg, the $(CRTPGMFLAGS) variable I changed gets pulled in by $(MODULE_TO_PGM_RECIPE) but not by the target PGM.CBLLE_TO_PGM_RECIPE. Thank you, I've made that change.

@robinsg
Copy link

robinsg commented May 11, 2025

In know this is a bit of a pain but, for completeness, I think USRPRF needs adding to these command variables:

CRTPGMFLAGS
CRTRPGPGMFLAGS
CRTCBLPGMFLAGS
CRTSQLCIFLAGS
CRTSQLCPPIFLAGS
CRTSQLCBLIFLAGS
CRTSQLRPGIFLAGS
CRTSRVPGMFLAGS
CRTBNDRPGFLAGS
CRTBNDCBLFLAGS
CRTBNDCFLAGS
CRTBNDCLFLAGS
CRTCLPGMFLAGS

@jwoehr
Copy link
Contributor Author

jwoehr commented May 12, 2025

In know this is a bit of a pain but, for completeness, I think USRPRF needs adding to these command variables:

You are probably right. Let's see what the maintainers comment. It's not a lot of work but I want to keep the PR simple until I get maintainer feedback.

@robinsg
Copy link

robinsg commented May 12, 2025

In know this is a bit of a pain but, for completeness, I think USRPRF needs adding to these command variables:

You are probably right. Let's see what the maintainers comment. It's not a lot of work but I want to keep the PR simple until I get maintainer feedback.

Agree.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants