Activity
From 04/05/2022 to 05/04/2022
05/04/2022
- 10:55 AM Feature #6320: sharded schemata
- We have multiple customers that split up a large 4GL application into multiple independent modules, each of which can...
- 10:51 AM Feature #6320 (WIP): sharded schemata
- 10:47 AM Feature #6319: IntelliJ plugin
- IntelliJ plugin for integration with syntax check and running conversion (et al). This must also provide the full sup...
- 10:44 AM Feature #6319 (Rejected): IntelliJ plugin
05/02/2022
- 11:02 AM Feature #6256: improved profile support
- If there were a need for 2 main database configured:
# a *merged* configuration with all schemas in one database, na... - 10:25 AM Feature #6256: improved profile support
- Sorry, failed refresh.
- 10:24 AM Feature #6256: improved profile support
- Eric Faulhaber wrote:
> Ovidiu, do I understand correctly that when the @importFile@ attribute of @namespace@ is spe... - 10:26 AM Feature #6302: Improve conversion of empty OCX arguments
- This issue does not appear in known customer scenarios, so its severity reduced to LOW.
05/01/2022
- 11:43 AM Feature #6256: improved profile support
- The current changes are in 6129a/13823.
04/30/2022
- 05:20 PM Feature #6256: improved profile support
- Greg Shah wrote:
> We can defer any conversion issues until a second phase of work. The front end part is what is n... - 02:53 PM Feature #6256: improved profile support
- We can defer any conversion issues until a second phase of work. The front end part is what is needed this week.
- 02:49 PM Feature #6256: improved profile support
- Greg Shah wrote:
> My idea was to add an extra layer at the ConversionDriver level, which would associate the profil... - 02:32 PM Feature #6256: improved profile support
- My idea was to add an extra layer at the ConversionDriver level, which would associate the profile with some sub-set ...
- 02:30 PM Feature #6256: improved profile support
- > What if a file is part of multiple profiles?
The profiles are intended to be mutually-exclusive. We can define ... - 02:26 PM Feature #6256: improved profile support
- Greg Shah wrote:
> The files specified for the profile should be parsed/converted with the same settings as if the p... - 02:15 PM Feature #6256: improved profile support
- When I said the multiple profile merge was needed, I meant a different idea.
Each profile can be run standalone. ... - 02:07 PM Feature #6256: improved profile support
- So, the approach is this:
* multiple profiles are allowed (only for conversion time), and they can be specified via ... - 02:04 PM Feature #6256: improved profile support
- Eric: I think the shard idea makes good sense. We have multiple customers using this approach in OE today. The idea...
- 01:59 PM Feature #6256: improved profile support
- A useful addition (medium term, not immediately) would be to be able to load multiple DF files into a single schema n...
- 01:03 PM Feature #6256: improved profile support
- Greg Shah wrote:
> The multi-profile merge mode is needed as part of this task.
This will be only for conversion. ... - 01:00 PM Feature #6256: improved profile support
- It is a good plan.
- 12:22 PM Feature #6256: improved profile support
- Constantin Asofiei wrote:
> * if no profile is set as default, let the global configuration be the default, unless o... - 12:20 PM Feature #6256: improved profile support
- Greg Shah wrote:
> > The @SchemaConfig$Profile@ has a @isDefault@, which suggests that there can be a @isDefault@ at... - 12:13 PM Feature #6256: improved profile support
- Greg Shah wrote:
> The multi-profile merge mode is needed as part of this task.
Understood.
- 12:12 PM Feature #6256: improved profile support
- > Do you see a need to make the profile/schema node optional? In this case, it would be inherited from the global con...
- 12:10 PM Feature #6256: improved profile support
- Do you see a need to make the @profile/schema@ node optional? In this case, it would be inherited from the global con...
- 11:27 AM Feature #6256: improved profile support
- I don't think there is a hard requirement for that at the profile level.
- 11:26 AM Feature #6256: improved profile support
- > The @SchemaConfig$Profile@ has a @isDefault@, which suggests that there can be a @isDefault@ attribute at the @prof...
- 11:26 AM Feature #6256: improved profile support
- Greg Shah wrote:
> The namespace should assume @default="true"@ so that it doesn't have to be duplicated. But it sh... - 11:24 AM Feature #6256: improved profile support
- The namespace should assume @default="true"@ so that it doesn't have to be duplicated. But it should be possible to ...
- 11:12 AM Feature #6256: improved profile support
- The @profile@ node has only @name@ attribute in #6256-8.
The @SchemaConfig$Profile@ has a @isDefault@, which sugge... - 10:03 AM Feature #6256: improved profile support
- > At least the syntax in note 5 is no longer valid, right?
Right. - 09:56 AM Feature #6256: improved profile support
- Greg Shah wrote:
> Is there some other use case I'm missing?
I don't think so.
> We already handle the namespace... - 09:49 AM Feature #6256: improved profile support
- > When running in single profile mode, the profile will need to overwrite default @parameter@ and @file-set@ nodes,
... - 09:48 AM Feature #6256: improved profile support
- > each profile has the same syntax as the 'global' profile (so the namespace nodes would be added in a schema node)
... - 09:39 AM Feature #6256: improved profile support
- When running in single profile mode, the profile will need to overwrite default @parameter@ and @file-set@ nodes, and...
- 09:35 AM Feature #6256: improved profile support
- Greg/Eric: my thought would be the profiles to work like this:
* there is the 'global' profile which is used implici...
04/29/2022
- 01:49 PM Bug #6308: post string literal alternative coding quirk in 4GL preprocessor
- The function issue is actually a 4GL preprocessor quirk. I've created a testcase and checked it into [[p2j:Old UAST ...
- 01:46 PM Bug #6308 (Internal Test): post string literal alternative coding quirk in 4GL preprocessor
- 03:25 AM Feature #6302: Improve conversion of empty OCX arguments
- Vladimir, is there a customer app failing test case for this? If not I suggest you make this lower priority.
- 03:06 AM Feature #6302: Improve conversion of empty OCX arguments
- Vladimir Tsichevski wrote:
> The @.ast@ file seems to be correct (all parameters in the call are present, but in @.j...
04/28/2022
- 07:29 PM Feature #6302: Improve conversion of empty OCX arguments
- The @.ast@ file seems to be correct (all parameters in the call are present, but in @.jast@ file parameters for the @...
- 07:08 PM Feature #6302: Improve conversion of empty OCX arguments
- Vladimir Tsichevski wrote:
> Tijs Wickardt wrote:
>
> > The OpenEdge documentation states explicitly that OCX's a... - 06:18 PM Feature #6302: Improve conversion of empty OCX arguments
- Tijs Wickardt wrote:
> FYI: see @testcases/oxc_to_fwd/mscomctl-treeview-ocx.w@ :
> @hNode = chCfTreeView:TreeView:... - 06:10 PM Feature #6302: Improve conversion of empty OCX arguments
- Tijs Wickardt wrote:
> The OpenEdge documentation states explicitly that OCX's are 0-based, not 1-based like the 4... - 03:19 PM Feature #6302: Improve conversion of empty OCX arguments
- Tijs Wickardt wrote:
> Vladimir Tsichevski wrote:
> > Seems, I was not right here: undefined values *can* be used b... - 03:16 PM Feature #6302: Improve conversion of empty OCX arguments
- Vladimir Tsichevski wrote:
> Seems, I was not right here: undefined values *can* be used by OE to pass the "default"... - 02:50 PM Feature #6302 (WIP): Improve conversion of empty OCX arguments
- Seems, I was not right here: undefined values *can* be used by OE to pass the "default" value to OCX. Moreover, undef...
- 01:15 PM Feature #6302: Improve conversion of empty OCX arguments
- This issue must be fixed along with: #5622-12. Otherwise no compilable code is created by conversion for this call.
- 09:20 AM Feature #6302: Improve conversion of empty OCX arguments
- Hynek Cihlar wrote:
> @hNode.assign(hTree.unwrapTreeView().getNodes().unwrapTreeNodeCollection().add(new integer()... - 09:16 AM Feature #6302: Improve conversion of empty OCX arguments
- FYI: see @testcases/oxc_to_fwd/mscomctl-treeview-ocx.w@ :
@hNode = chCfTreeView:TreeView:Nodes:Add(,,cParentNodeID,... - 08:43 AM Feature #6302: Improve conversion of empty OCX arguments
- The function parameter specification in COM type library may contain a default value. Perhaps this is the value used ...
- 08:32 AM Feature #6302: Improve conversion of empty OCX arguments
- Empty arguments of converted OCX method invocations are converted into "unknown" instances.
For example the follo... - 08:23 AM Feature #6302 (WIP): Improve conversion of empty OCX arguments
- 12:10 PM Feature #6256: improved profile support
- Yes, the syntax is OK.
- 12:00 PM Feature #6256 (WIP): improved profile support
- Incorporating the above suggestions/requirements, I have updated the sample configuration as follows:...
- 07:25 AM Feature #6256: improved profile support
- Greg Shah wrote:
> > I'm assuming (for now) that profile *.df files will be found in ./data/<profile-name>/.
> No, ...
04/27/2022
- 07:27 PM Feature #6256: improved profile support
- > Is this the right vision?
Mostly, yes. The part I don't understand is why there is a @schema@ node inside the @... - 06:07 PM Feature #6256: improved profile support
- At this moment the structure of the configuration file is:...
- 04:59 PM Feature #6256: improved profile support
- To translate the above configuration requirements into a sample @p2j.cfg.xml@, I understand it should look something ...
04/26/2022
- 08:52 AM Feature #6255: simplify the namespace configuration, part deux
- What was frustrating was that in my local testing worked, but with the main project it abended - and this was only be...
04/24/2022
- 09:07 AM Feature #6255: simplify the namespace configuration, part deux
- With 13811, the metadata schema can be loaded twice. This was seen in a customer project which then had ambiguous re...
04/22/2022
- 06:28 PM Feature #6255 (WIP): simplify the namespace configuration, part deux
- The implementation for the initial two requests can be found in FWD starting with r13811 of 3821c.
We should keep ...
04/19/2022
- 12:02 PM Feature #6253: add file-set processing into p2j.cfg.xml
- Please check 3821c/13794.
- 10:41 AM Feature #6253: add file-set processing into p2j.cfg.xml
- > > A command like this:
> > [...]
> > will use the @file-set@ configuration from the @cfg/p2j.cfg.xml@ (will assum...
04/18/2022
- 05:45 AM Bug #5135 (Closed): replace -s, -f and -x options of ConversionDriver with a single combined mode
- 05:45 AM Feature #6253 (Closed): add file-set processing into p2j.cfg.xml
- Code Review Task Branch 3821c Revision 13788
It is good.
04/16/2022
- 02:58 PM Bug #5135: replace -s, -f and -x options of ConversionDriver with a single combined mode
- Greg Shah wrote:
> Any amount of whitespace (not just space char) should be ignored.
Please see 3821c/13788. - 02:58 PM Feature #6253: add file-set processing into p2j.cfg.xml
- The changes are in 3821c/13788.
04/15/2022
- 08:10 AM Bug #5135: replace -s, -f and -x options of ConversionDriver with a single combined mode
- Any amount of whitespace (not just space char) should be ignored.
- 12:55 AM Bug #5135: replace -s, -f and -x options of ConversionDriver with a single combined mode
- I created a list file for a project. The file contained several lines using the @D@ directive. I noticed there can be...
04/14/2022
- 04:57 PM Feature #6253: add file-set processing into p2j.cfg.xml
- > but recursive makes sense for it to default to 'true'.
Let's do it. - 04:56 PM Feature #6253: add file-set processing into p2j.cfg.xml
- Code Review Task Branch 3821c Revision 13778
The changes are good.
1. An "include file" has a special meaning i...
04/13/2022
- 09:20 AM Feature #6203: create cvtpath and move conversion database into that directory
- Constantin Asofiei wrote:
> Thanks, I can duplicate it:
> * run @ant convert.list@ with 3821c - name is @MultiLine1... - 09:15 AM Feature #6203: create cvtpath and move conversion database into that directory
- Thanks, I can duplicate it:
* run @ant convert.list@ with 3821c - name is @MultiLine1@.
* run @ant clean.all@
* ru... - 09:03 AM Feature #6203: create cvtpath and move conversion database into that directory
- Constantin Asofiei wrote:
> Roger, please upload to devsrv01 your entire testcases project where you do the conversi... - 04:04 AM Feature #6203: create cvtpath and move conversion database into that directory
- Roger, please upload to devsrv01 your entire testcases project where you do the conversion (without the p2j/ folder)....
- 08:34 AM Feature #6253: add file-set processing into p2j.cfg.xml
- Greg Shah wrote:
> Should we make the following common cases the default?
>
> * @recursive="true"@
> * @spec="(*... - 08:32 AM Feature #6253 (Review): add file-set processing into p2j.cfg.xml
- The changes are in 3821c/13778 for both this task and #5135.
- 08:28 AM Feature #6253: add file-set processing into p2j.cfg.xml
- It looks good.
Should we make the following common cases the default?
* @recursive="true"@
* @spec="(*.[pPwW... - 08:15 AM Feature #6253: add file-set processing into p2j.cfg.xml
- The @file-set@ in @p2j.cfg.xml@ will look like this:...
- 08:10 AM Feature #6253: add file-set processing into p2j.cfg.xml
- Constantin Asofiei wrote:
> A command like this:
> [...]
> will use the @file-set@ configuration from the @cfg/p2j... - 08:09 AM Feature #6253: add file-set processing into p2j.cfg.xml
- > I've used @<file-set filename="file-set.txt" />@ (@filename@ instead of @input-file@) because of the dash in @input...
- 08:08 AM Feature #6253: add file-set processing into p2j.cfg.xml
- A command like this:...
- 08:07 AM Feature #6253: add file-set processing into p2j.cfg.xml
- I've used @<file-set filename="file-set.txt" />@ (@filename@ instead of @input-file@) because of the dash in @input-f...
- 08:32 AM Bug #5135 (Review): replace -s, -f and -x options of ConversionDriver with a single combined mode
- The changes are in 3821c/13778 for both this task and #6253.
- 08:28 AM Bug #5135: replace -s, -f and -x options of ConversionDriver with a single combined mode
- OK
- 08:19 AM Bug #5135: replace -s, -f and -x options of ConversionDriver with a single combined mode
- Z mode is stripping the @./@ prefix, because of some peculiarities when resolving the path (I need the real OS filena...
04/12/2022
- 04:18 PM Feature #6203: create cvtpath and move conversion database into that directory
- Ovidiu Maxiniuc wrote:
> This is odd, at least. Does it happens for a single project? It does not happen for me. It ... - 03:42 PM Feature #6203: create cvtpath and move conversion database into that directory
- This is odd, at least. Does it happens for a single project? It does not happen for me. It is a configuration issue, ...
- 03:27 PM Feature #6203: create cvtpath and move conversion database into that directory
- I'm not sure if this was an intended change, but with the 13732 revision, generated filenames are no longer Camel-Cas...
04/11/2022
- 11:25 AM Feature #6270: annotations-based per-file hints
- Greg Shah wrote:
> To clarify, I meant that to maintain the WINE environment one must have an original Windows funct... - 08:54 AM Feature #6270: annotations-based per-file hints
- > > a working Windows configuration
> This seems to be a misunderstanding. I meant:
> Only the binaries that are n... - 08:48 AM Feature #6270: annotations-based per-file hints
- No need for apologies, these are all valid points.
> a working Windows configuration
This seems to be a misunders... - 08:17 AM Feature #6270: annotations-based per-file hints
- Every time we add *any* dependency, we add friction in the process of getting everything running properly. For conve...
- 02:41 AM Feature #6270: annotations-based per-file hints
- Greg Shah wrote:
> I don't see the same value in maintaining the @.wrx@ as a resource. It cannot be easily edited, ...
04/10/2022
- 07:41 PM Feature #6270: annotations-based per-file hints
- I don't see the same value in maintaining the @.wrx@ as a resource. It cannot be easily edited, it can only be edite...
- 02:36 PM Feature #6270: annotations-based per-file hints
- Note: https://pkgs.alpinelinux.org/package/edge/community/x86/wine
(and winetricks)
Non edge version (stable): ... - 02:16 PM Feature #6270: annotations-based per-file hints
- Hynek Cihlar wrote:
> Greg Shah wrote:.
> > > .. interpret .wrx files (for example) the related annotations should ... - 12:15 PM Feature #6270: annotations-based per-file hints
- Greg Shah wrote:
> > It should be limited only to information that FWD cannot deduce at the conversion.
>
> This... - 12:04 PM Feature #6270: annotations-based per-file hints
- Greg Shah wrote:
> > It should be limited only to information that FWD cannot deduce at the conversion.
>
> This... - 11:14 AM Feature #6270: annotations-based per-file hints
- > It should be limited only to information that FWD cannot deduce at the conversion.
This is already the case. W... - 11:08 AM Feature #6270: annotations-based per-file hints
- I like the idea of using 4GL annotations for FWD runtime hints.
It reduces the chance of an OE developer forgetting ... - 09:48 AM Feature #6270: annotations-based per-file hints
- Today, we have multiple types of hints at the individual source-file level.
* "UAST Hints" are read from a file th... - 09:14 AM Feature #6270 (New): annotations-based per-file hints
04/08/2022
- 10:00 AM Bug #5135: replace -s, -f and -x options of ConversionDriver with a single combined mode
- > I can either use java.nio.Path.toRealPath (if it works) to resolve the real filename at OS level, or otherwise rely...
- 09:17 AM Bug #5135: replace -s, -f and -x options of ConversionDriver with a single combined mode
- Another issue: lets assume we are under Windows, where file names are case-insensitive; you have a @F ./Build.p@ line...
- 07:45 AM Feature #6202: allow .df files to exist in a path that is not directly in data/
- > I think the schema @.hints@ should be stored with the @.df@ file, not with @.schema@.
Correct.
> More than t...
04/07/2022
- 07:05 PM Feature #6202: allow .df files to exist in a path that is not directly in data/
- This note was added because of the #5034-1965. It was caused because the conversion got confused about the location o...
- 01:42 PM Feature #6202: allow .df files to exist in a path that is not directly in data/
- > Note that this defines the location where the .d files are, to allow the import for custom locations. This is not t...
- 11:48 AM Feature #6202: allow .df files to exist in a path that is not directly in data/
- Greg Shah wrote:
> * The @dataPath@ parameter for import could be optional. I like to have the option to override t... - 06:53 AM Feature #6202: allow .df files to exist in a path that is not directly in data/
- All of this is very good. A couple of items:
* The @dataPath@ parameter for import could be optional. I like to ... - 02:25 PM Bug #5135: replace -s, -f and -x options of ConversionDriver with a single combined mode
- And yes, it is OK to leave @-s@.
- 02:25 PM Bug #5135: replace -s, -f and -x options of ConversionDriver with a single combined mode
- I was assuming that all of the modes would remain until we've migrated all projects into the @-z@.
- 02:16 PM Bug #5135: replace -s, -f and -x options of ConversionDriver with a single combined mode
- Greg, I'll remove the whitelist and blacklist modes in current work, but the 'FILESPEC' (-s mode) should remain, this...
- 11:19 AM Bug #5135: replace -s, -f and -x options of ConversionDriver with a single combined mode
- Greg Shah wrote:
> Yes, this makes sense. We should also require double quoted strings when there are embedded spac... - 11:14 AM Bug #5135: replace -s, -f and -x options of ConversionDriver with a single combined mode
- Yes, this makes sense. We should also require double quoted strings when there are embedded space characters.
<pr... - 10:25 AM Bug #5135: replace -s, -f and -x options of ConversionDriver with a single combined mode
- Greg, in the case where we use a file, there will be problems if there are space characters in either the file spec o...
- 06:55 AM Bug #5135: replace -s, -f and -x options of ConversionDriver with a single combined mode
- As with other flags @-z@ and @-Z@ should be equivalent.
- 06:55 AM Bug #5135: replace -s, -f and -x options of ConversionDriver with a single combined mode
- > Greg, to make something clear: the @X@ and @I@ filters will work only on the set of files determined from the D, N,...
- 03:04 AM Bug #5135 (WIP): replace -s, -f and -x options of ConversionDriver with a single combined mode
- Greg, to make something clear: the @X@ and @I@ filters will work only on the set of files determined from the D, N, F...
- 11:26 AM Feature #6253 (WIP): add file-set processing into p2j.cfg.xml
04/06/2022
- 07:04 PM Feature #6202: allow .df files to exist in a path that is not directly in data/
- With r13746 we have the abilities to:
* pick the .df from any subdirectory under the project's root. The path is spe... - 07:03 PM Feature #6256: improved profile support
- For complex projects, the OpenEdge compilation will often be done in multiple different compile runs. Each one might...
- 06:46 PM Feature #6256 (Closed): improved profile support
- 06:45 PM Feature #6255: simplify the namespace configuration, part deux
- In #4105, many improvements were made to simplify the namespace configuration in @p2j.cfg.xml@.
I have 2 additiona... - 06:40 PM Feature #6255 (WIP): simplify the namespace configuration, part deux
- 06:33 PM Feature #6253: add file-set processing into p2j.cfg.xml
- The global section of the @p2j.cfg.xml@ should allow the definition of a @file-set@ element which will allow the expl...
- 06:14 PM Feature #6253 (Closed): add file-set processing into p2j.cfg.xml
- 06:17 PM Bug #5135: replace -s, -f and -x options of ConversionDriver with a single combined mode
- This new file processing approach can be activated with the @-Z<input_file>@ command line option. I know this is dif...
- 05:04 PM Bug #5135: replace -s, -f and -x options of ConversionDriver with a single combined mode
- I've been thinking about the syntax for this approach. Using a list of @X@ (exclude) and @I@ (include) filters is a ...
- 06:10 PM Feature #6252: cleanup of p2j.cfg.xml
- Make the global section of @p2j.cfg.xml@ simpler with the following changes:
* Set good default values for @P2J_RU... - 06:06 PM Feature #6252 (New): cleanup of p2j.cfg.xml
- 05:20 PM Feature #3699: improve call-graph visualization to make it much more usable
- I found an interesting article on the "ForceAtlas2":https://journals.plos.org/plosone/article?id=10.1371/journal.pone...
04/05/2022
- 10:06 AM Feature #6237: OEUnit support
- "OEUnit":https://github.com/CameronWills/OEUnit is an open source project (Eclipse Public License) which preceded ABL...
- 10:03 AM Feature #6237 (Closed): OEUnit support
Also available in: Atom