Bugzilla – Full Text Bug Listing
| Summary: | Several about.html have invalid XHTML due to missing </p> tag | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | [Tools] Orbit | Reporter: | Martin Oberhuber <mober.at+eclipse> | ||||||||
| Component: | bundles | Assignee: | Orbit Bundles <orbit.bundles-inbox> | ||||||||
| Status: | NEW --- | QA Contact: | |||||||||
| Severity: | minor | ||||||||||
| Priority: | P4 | CC: | david_williams, dj.houghton, wayne.beaton | ||||||||
| Version: | unspecified | ||||||||||
| Target Milestone: | --- | ||||||||||
| Hardware: | PC | ||||||||||
| OS: | Windows XP | ||||||||||
| Whiteboard: | |||||||||||
| Attachments: |
|
||||||||||
|
Description
Martin Oberhuber
I fixed the issue for org.apache.commons.httpclient org.apache.commons.net org.apache.log4j Created attachment 184737 [details]
patch v1
I have taken a first pass at fixing the XHTML in those bundle's about that we're shipping in our product:
com.jcraft.jsch
javax.servlet.jsp
org.apache.commons.codec
org.apache.commons.el
org.apache.commons.logging
org.apache.jasper
org.apache.lucene
org.apache.lucene.analysis
org.apache.xerces
org.sat4j.core
org.sat4j.pb
Turned out that each bundle had slightly different problems, so it was a manual approach (using WTP's "validate"). When I release those changes, respective bundles will be updated in Indigo. Before I proceed though, I just wanted to check back with the group whether that's really what we want - since the defect itself isn't a major thing.
Created attachment 187300 [details] bundles.map changes Since I did not hear any concerns, I finally committed these changes. Turned out that for some bundles, newer versions are around now: javax.servlet.jsp -- I fixed 2.0.0 but did not check 2.1.0 org.apache.commons.logging -- I fixed 1.0.4 but did not check 1.1.1 org.apache.lucene/analysis -- I fixed 1.9.2 but did not check 2.9.2 com.jcraft.jsch -- I fixed 0.1.41 but did not check 0.1.44 I also noticed that for plugin@org.apache.jasper,5.5.17 the branch tag (,5.5.17) had been missing in bundles.map. This is now fixed. I would appreciate if bundle maintainers could check their about.html files for proper xhtml syntax from now on. Just apply a Webtools validate before checkin. Created attachment 187303 [details]
Additional patch for newer versions
So I fixed the newer versions too:
> javax.servlet.jsp 2.1.0
> org.apache.commons.logging 1.1.1
> org.apache.lucene/analysis/core 2.9.1
> com.jcraft.jsch 0.1.44
This should fix all the about XHTML for Orbit bundlis shipped with the core Platform Runtime. I'm going to leave the bug open though since I cannot check all the bundles in the repo.
Again, ideally each bundle maintainer should derive new about.html files from kosher ones, and run WTP validate on them.
(In reply to comment #5) > So I fixed the newer versions too: > > > javax.servlet.jsp 2.1.0 > > org.apache.commons.logging 1.1.1 > > org.apache.lucene/analysis/core 2.9.1 > > com.jcraft.jsch 0.1.44 > > This should fix all the about XHTML for Orbit bundlis shipped with the core > Platform Runtime. I'm going to leave the bug open though since I cannot > check all the bundles in the repo. > > Again, ideally each bundle maintainer should derive new about.html files > from kosher ones, and run WTP validate on them. Perhaps you could udpate wiki at http://wiki.eclipse.org/Orbit/Adding_Bundles_to_Orbit and then count this as "fixed"? |