rototor commented on pull request #67:
URL: https://github.com/apache/freemarker/pull/67#issuecomment-633005977


   Using ```#import``` in the first place would have made sense. But changing 
our 65k lines freemarker template codebase, which built up the last 10 years,  
to use ```#import``` instead of ```#include``` is not gonna happen... As you 
must change all affected places to use the namespace defined by ```#import```. 
And ensure that the namespaces are named exactly the same in every file. 
Otherwise you get in trouble as soon as you search for something...
   
   There should at least be a note in the ```#include``` documentation, that 
you should seriously consider using ```#import``` if you are including 
macros/functions. And maybe even an option in the ```#ftl``` directive to 
specify a "import only" file. Also some way to specify and enforce the "target" 
namespace name would be helpful.
   
   This pull request is more a fix/workaround for codebases you simply can't 
change to ```#import```.


----------------------------------------------------------------
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.

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


Reply via email to