Project

General

Profile

Feature #4162

enhance existing SESSION:STARTUP-PARAMETERS support to honor -ininame value

Added by Greg Shah almost 5 years ago. Updated about 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
Due date:
% Done:

100%

billable:
No
vendor_id:
GCD
version_reported:
version_resolved:

Related issues

Related to Base Language - Feature #4391: enhance existing SESSION:STARTUP-PARAMETERS support to honor -H -db -S -ld -N values Closed

History

#1 Updated by Greg Shah almost 5 years ago

  • Subject changed from enhance existing SESSION:STARTUP-PARAMETERS support to honor additional values to enhance existing SESSION:STARTUP-PARAMETERS support to honor -ininame value

#2 Updated by Greg Shah about 4 years ago

See SessionUtils.getStartupParameters() and LegacyCLI.

We should add a specific capability for directory configuration of -ininame.

In addition, we should add support for passing something as a bootstrap config override on the FWD client command line. This can be passed up as part of the client params and then used in LegacyCLI. This would be a general purpose argument passing "thing".

#3 Updated by Roger Borrello about 4 years ago

The comments in LegacyCLI.getStartupParameters indicate "Returns a character string containing a comma-separated list...", yet the return doesn't use commas to separate. Rather it uses spaces: return s1 + (s1.isEmpty() ? "" : " ") + s2;

Are there any testcases for the most recent addition of -icfparam I could leverage to validate behavior? Is there any documentation about how to format the directory for STARTUP-PARAMETERS? I only get a hit on some old Lexer documentation when I search.

#4 Updated by Greg Shah about 4 years ago

The comments in LegacyCLI.getStartupParameters indicate "Returns a character string containing a comma-separated list...", yet the return doesn't use commas to separate. Rather it uses spaces: return s1 + (s1.isEmpty() ? "" : " ") + s2;

Check the 4GL. If the docs are wrong, fix them.

Are there any testcases for the most recent addition of -icfparam I could leverage to validate behavior?

The changes were in #3812. The "new" testcases project would have something, probably.

Is there any documentation about how to format the directory for STARTUP-PARAMETERS? I only get a hit on some old Lexer documentation when I search.

No. You have read the LegacyCLI code to see how each one is being read.

#5 Updated by Roger Borrello about 4 years ago

Ovidiu, you had made changes to ClientCore in 2014 to emulate the -ininame command-line parameter. Is there any chance you might recall if that would have allowed a web client to be started with "-ininame <inifile>" passed through the directory?

#6 Updated by Roger Borrello about 4 years ago

Roger Borrello wrote:

Ovidiu, you had made changes to ClientCore in 2014 to emulate the -ininame command-line parameter. Is there any chance you might recall if that would have allowed a web client to be started with "-ininame <inifile>" passed through the directory?

+Ovidiu

#7 Updated by Roger Borrello about 4 years ago

  • % Done changed from 0 to 90

Changes to support -ininame are checked into 4231b-11608 (LegacyCLI.java). Testcases indicated there should be commas between multiple parameters, but the code was using spaces. I updated that, as well.

#8 Updated by Roger Borrello about 4 years ago

Is #4391 basically additional parsing in the same manner as this task?

#9 Updated by Greg Shah about 4 years ago

Roger Borrello wrote:

Is #4391 basically additional parsing in the same manner as this task?

Yes.

#10 Updated by Roger Borrello about 4 years ago

  • Status changed from New to WIP
  • % Done changed from 90 to 100

Ready for test.

#11 Updated by Greg Shah about 4 years ago

  • Status changed from WIP to Closed

Task branch 4231b has been merged to trunk as revision 11347.

#12 Updated by Roger Borrello about 1 year ago

  • Related to Feature #4391: enhance existing SESSION:STARTUP-PARAMETERS support to honor -H -db -S -ld -N values added

Also available in: Atom PDF