mike-mcgann commented on PR #919:
URL: https://github.com/apache/daffodil/pull/919#issuecomment-1406689243

   > However, there were some misses that may need to be corrected.
   
   I didn't include the TDML packages in the refactoring effort because I 
didn't think they would be loaded into an OSGI container but I went ahead and 
refactored these too and this is included in the recent updates. 
   
   The macros seem to be a special case. They are in a separate package, span 
multiple packages, and get copied into the packages that use them. I'm not sure 
what we should be doing here with those. When I was talking to Mike B. he was 
interested in having these scripts do most of the heavy lifting and additional 
work to finalize could be done after. If you are okay with it, I think we 
should address the macros in a separate PR as that might require some research 
and additional discussion.     
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to