[MAGNOLIA-1211] Generate TLD files Created: 15/Nov/06 Updated: 23/Jan/13 Resolved: 26/Mar/09 |
|
| Status: | Closed |
| Project: | Magnolia |
| Component/s: | taglibs |
| Affects Version/s: | 3.0 Final |
| Fix Version/s: | 4.1 |
| Type: | Improvement | Priority: | Major |
| Reporter: | Magnolia International | Assignee: | Magnolia International |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
||||||||||||||||||||||||||||||||||||||||
| Issue Links: |
|
||||||||||||||||||||||||||||||||||||||||
| Template: |
|
||||||||||||||||||||||||||||||||||||||||
| Patch included: |
Yes
|
||||||||||||||||||||||||||||||||||||||||
| Acceptance criteria: |
Empty
|
||||||||||||||||||||||||||||||||||||||||
| Task DoD: |
[ ]*
Doc/release notes changes? Comment present?
[ ]*
Downstream builds green?
[ ]*
Solution information and context easily available?
[ ]*
Tests
[ ]*
FixVersion filled and not yet released
[ ] 
Architecture Decision Record (ADR)
|
||||||||||||||||||||||||||||||||||||||||
| Date of First Response: | |||||||||||||||||||||||||||||||||||||||||
| Description |
Can be done with xdoclet2+maven plugin |
| Comments |
| Comment by Fabrizio Giustina [ 15/Dec/06 ] |
|
should I say that I am strongly against this?
|
| Comment by Magnolia International [ 18/Dec/06 ] |
|
> - makes working with an IDE a lot more difficult (you will have to run maven or configure xdoclet, etc) I don't see why it would be a problem, at least in the context of Magnolia, since both our taglibs are separate modules. > - makes tags documentation and java code less clean: That will still be the case. > and that's easier to write in a xml file instead that in javadoc comments I don't see how. > - not everything can be done using xdoclet (not sure how BeanInfo classes could be handled) BeanInfo support could be implemented very easily if its not the case yet. > - and generally it makes build and site generation more complex... Especially considering the advantage that
|
| Comment by Fabrizio Giustina [ 19/Dec/06 ] |
|
>> - makes working with an IDE a lot more difficult (you will have to run maven or configure xdoclet, etc) Well, using eclipse you usually load all the modules as linked project and classes/tld are loaded directly from the classes dir, without the need for building/installing a full jar... you only have to compile classes, without running maven or xdoclet. Maybe you are using IDEA or a different setup? |
| Comment by Magnolia International [ 19/Dec/06 ] |
|
I'm using IDEA. Sure, if the module is clean, I wont have the tld either, but I dont think running maven once on just the taglib modules is a showstopper if you work on other modules ? (since there's not much else in the module itself that would require the tld ...) I'm not trying to enforce the use of xdoclet here, I'm just bothered by duplicated information (java source and tld) and thus potential synch problems. xdoclet is not the most exciting piece of technology (although xd2 is a lot better/easier/faster in many respects than xd1), but tld are clearly "generate-me" shouting files |
| Comment by Jan Haderka [ 23/Nov/07 ] |
|
In light of latest problems with strict jsp compilers (javelin/WL) and need of review out of sync tags/tlds (/ |
| Comment by Magnolia International [ 30/Jul/08 ] |
|
I'd like to tackle this one of these days, and if results are encouraging (i.e generated files compare well to current one) I'd like to do this sooner than later, for I've noticed once more that the tld doc says that the taglib version is "3.5", which makes no sense. |
| Comment by Magnolia International [ 14/Aug/08 ] |
|
All tag classes for magnolia-taglib-cms have just been xdoclet-tagged, and the comparison between generated tld and manually maintained tld yields zero difference. I'll commit the changes to the build (usage of the xdoclet-plugin in pom.xml) as soon as the latest version of the xdoclet web-plugin is available. (currently built locally) |
| Comment by Magnolia International [ 19/Aug/08 ] |
|
Same for magnolia-taglib-utility is done. |
| Comment by Magnolia International [ 26/Sep/08 ] |
|
Will not have time to test latest snapshots and probably the release won't be made before we release 3.6.2. Attaching the patch for pom files. (repositories only necessary for snapshots) I also have a little xmldiff script that shows the differences (none except the version, since i added and fixed all the tags) |
| Comment by Magnolia International [ 20/Jan/09 ] |
|
Pushing this further once more; looks like the xdoclet 2.0.7 final release won't be ready in time for 4.0 |
| Comment by Magnolia International [ 25/Mar/09 ] |
|
linking to |
| Comment by Magnolia International [ 26/Mar/09 ] |
|
Yay ! For the record, I compared our original TLDs with the generated ones using a custom little tool : http://svn.magnolia-cms.com/svn/internal/xml-diff/ |