Project

General

Profile

Feature #5630

add missing classes referenced during parsing to the conversion list in file list mode

Added by Greg Shah over 2 years ago. Updated over 2 years ago.

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

0%

billable:
No
vendor_id:
GCD

Related issues

Related to Conversion Tools - Bug #5135: replace -s, -f and -x options of ConversionDriver with a single combined mode Closed
Related to Conversion Tools - Bug #6082: automatically add to conversion list all non-skeleton .cls dependencies New

History

#1 Updated by Greg Shah over 2 years ago

From Marian:

I'm stil having issues to get everything to convert though, the process takes a lot of time only to greet me with a NPE on `classLoad` that doesn't give any pertinent information regarding what was it trying to load and couldn't find it :(

I see the conversion process has several steps, no idea if that can somehow be extended to do something like a PROPATH resolution so the missing classes could be somehow automagically added to the conversion list or anything like that.

From Greg:

We already do resolve classes via the PROPATH during parsing. I think the problem here is that the list of classes to convert is only automatically generated if we use the "filespec" mode or blacklist mode to convert. If we use an explicit conversion list, then we don't automatically add referenced classes to it since you are telling us the exact procedures/classes to convert.

From Marian:

I see, this makes sense only it will definitively be helpful if those could be somehow combined... not convert everything, start from the list given but then add missing classes as needed, resolved from PROPATH. This will reduce the scope of conversion (and time) while still avoiding those nasty NPE exceptions if a 'dependency' was left out of the conversion list.

#2 Updated by Greg Shah over 2 years ago

This is for full conversion mode, though we will also need to consider the implications for incremental conversion mode.

#3 Updated by Greg Shah over 2 years ago

  • Related to Bug #5135: replace -s, -f and -x options of ConversionDriver with a single combined mode added

#4 Updated by Greg Shah over 2 years ago

Consider that when we implement #5135, we will require changes for this feature.

We probably also should implement this even in blacklist mode. In other words: what should we do if there is a reference to a class that is excluded from conversion?

#5 Updated by Greg Shah about 2 years ago

  • Related to Bug #6082: automatically add to conversion list all non-skeleton .cls dependencies added

Also available in: Atom PDF