[hibernate-dev] release

Strong Liu stliu at redhat.com
Thu Mar 11 10:32:55 EST 2010


IMO, this is the point, XIncludeHelper.locateInclusions use FileUtil.createSAXSource to create a Source object, after you enabled this loading feature, it will tries to resolve the dtd from remote
here is what i changed.

http://fisheye.jboss.org/browse/Maven/plugins/jdocbook/maven-jdocbook-plugin/trunk/src/main/java/org/jboss/jdocbook/util/XIncludeHelper.java?r1=428&r2=442

428   442   442   XIncludeHelper.java
	37	37	 	
 	38	38	 	import org.apache.xerces.jaxp.SAXParserFactoryImpl;
 	39	39	 	import org.jboss.jdocbook.JDocBookProcessException;
	 	40	+	import org.jboss.jdocbook.xslt.resolve.entity.EntityResolverChain;
 	 	41	+	import org.jboss.jdocbook.xslt.resolve.entity.LocalDocBookEntityResolver;
40	42	 	import org.xml.sax.EntityResolver;
 	41	43	 	import org.xml.sax.InputSource;
 	42	44	 	import org.xml.sax.SAXException;
 	 	 	
 
…
72	74	 	                                return null;
 	73	75	 	                        }
 	74	76	 	                };
	 	77	+	                
 	 	78	+	                EntityResolverChain entityResolverChain = new EntityResolverChain(entityResolver);
 	 	79	+	                entityResolverChain.addEntityResolver(new LocalDocBookEntityResolver());
75	80	 	
 	76	81	 	                try {
 	77	82	 	                        SAXParserFactory parserFactory = new SAXParserFactoryImpl();
 	78	83	 	            parserFactory.setXIncludeAware( true  );
 	79	84	 	
	80	 	-	                        Source transformationSource = FileUtils.createSAXSource( root, entityResolver, true, null );
 	 	85	+	                        Source transformationSource = FileUtils.createSAXSource( root, entityResolverChain, true, null );
	81	86	 	                        Result transformationResult = new StreamResult( new NoOpWriter() );
 	82	87	 	
 	83	88	 	                        javax.xml.transform.TransformerFactory transformerFactory = new com.icl.saxon.TransformerFactoryImpl();


On Mar 11, 2010, at 11:26 PM, Steve Ebersole wrote:

> That's not true afaik:
> reader.setFeature( Constants.DTD_LOADING_FEATURE, true );
> reader.setFeature( Constants.DTD_VALIDATION_FEATURE, false );
> 
> Loading *needs* to happen.
> 
> Anyway that code was the same before I updated and after I updated. This is in FileUtils#createSAXSource.  So where is it that you changed?
> 
> On 03/11/2010 08:49 AM, Strong Liu wrote:
>> I have fixed this issue https://jira.jboss.org/jira/browse/MPJDOCBOOK-52, and released a 2.2.3-SNAPSHOT
>> 
>> i'd think this is because Steve enabled the dtd validation in this 2.2.3 release
>> 
>> i will try to build hibernate trunk later (for the sync) with this snapshot release
>> Steve, would you consider release a 2.2.3?
>> 
>> 
>> -------------------------------
>> Best Regards,
>> 
>> Strong Liu
>> 
>> 
>> 
>> On Mar 11, 2010, at 12:12 PM, Gail Badner wrote:
>> 
>>> The build seems to hang while building the doc, so this is not going to
>>> happen tonight.
>>> 
>>> Here is the build log when it stops:
>>> 
>>> [INFO] Building Hibernate Manual
>>> [INFO]    task-segment: [clean, test]
>>> [INFO]
>>> ------------------------------------------------------------------------
>>> [INFO] [clean:clean {execution: default-clean}]
>>> [INFO] Deleting directory
>>> /NotBackedUp/gbadner/hibernate-core/trunk-before-3.5.0-CR-3/documentation/manual/target
>>> [INFO] [enforcer:enforce {execution: enforce-java}]
>>> [INFO] [jdocbook:resources {execution: default-resources}]
>>> [INFO] Expanding:
>>> /home/gbadner/NotBackedUp/.m2/repository-trunk/org/jboss/jbossorg-jdocbook-style/1.1.0/jbossorg-jdocbook-style-1.1.0.jdocbook-style
>>> into
>>> /NotBackedUp/gbadner/hibernate-core/trunk-before-3.5.0-CR-3/documentation/manual/target/docbook/staging
>>> [INFO] Expanding:
>>> /home/gbadner/NotBackedUp/.m2/repository-trunk/org/jboss/jbossorg-fonts/1.0.0/jbossorg-fonts-1.0.0.jdocbook-style
>>> into
>>> /NotBackedUp/gbadner/hibernate-core/trunk-before-3.5.0-CR-3/documentation/manual/target/docbook/staging
>>> [INFO] Expanding:
>>> /home/gbadner/NotBackedUp/.m2/repository-trunk/org/hibernate/hibernate-jdocbook-style/2.0.1/hibernate-jdocbook-style-2.0.1.jdocbook-style
>>> into
>>> /NotBackedUp/gbadner/hibernate-core/trunk-before-3.5.0-CR-3/documentation/manual/target/docbook/staging
>>> [INFO] Expanding:
>>> /home/gbadner/NotBackedUp/.m2/repository-trunk/org/jboss/jbossorg-jdocbook-style/1.1.0/jbossorg-jdocbook-style-1.1.0.jdocbook-style
>>> into
>>> /NotBackedUp/gbadner/hibernate-core/trunk-before-3.5.0-CR-3/documentation/manual/target/docbook/staging
>>> [INFO] Expanding:
>>> /home/gbadner/NotBackedUp/.m2/repository-trunk/org/jboss/jbossorg-fonts/1.0.0/jbossorg-fonts-1.0.0.jdocbook-style
>>> into
>>> /NotBackedUp/gbadner/hibernate-core/trunk-before-3.5.0-CR-3/documentation/manual/target/docbook/staging
>>> [INFO] [jdocbook:translate {execution: default}]
>>> 
>>> 
>>> 
>>> Gail Badner wrote:
>>>> Just to let you know, I'm making preparations for 3.5.0-CR-3.
>>>> -- Gail
>>>> Hardy Ferentschik wrote:
>>>> 
>>>>> Well, there are still a few code changes and if you tar up all the
>>>>> different documentation
>>>>> bundles and attach them to a helpdesk request you get them uploaded quite
>>>>> quickly (and yes
>>>>> an automated upload would be nice!)
>>>>> 
>>>>> If the majority of people is for a delay then that's fine with me, but I
>>>>> just wanted to
>>>>> remind how you yourself said how important it is to stick to the boxed
>>>>> releases.
>>>>> 
>>>>> --Hardy
>>>>> 
>>>>> 
>>>>> 
>>>>> 
>>>>> On Wed, 10 Mar 2010 19:02:46 -0300, Steve Ebersole<steve at hibernate.org>
>>>>> wrote:
>>>>> 
>>>>> 
>>>>> 
>>>>>> I agree in theory.  But here we have very little actual code changes:
>>>>>> http://opensource.atlassian.com/projects/hibernate/secure/IssueNavigator.jspa?reset=true&jqlQuery=project+%3D+HHH+AND+fixVersion+%3D+%223.5.0.Next%22+AND+status+in+%28Resolved%2C+Closed%29+ORDER+BY+priority+DESC
>>>>>> 
>>>>>> That coupled with the fact the uploading docs is still painful is it
>>>>>> worth it?
>>>>>> 
>>>>>> On 03/10/2010 02:33 PM, Hardy Ferentschik wrote:
>>>>>> 
>>>>>> 
>>>>>>> Given the discussion we had the other day about boxed releases, we
>>>>>>> should have another CR candidate
>>>>>>> in this case.
>>>>>>> 
>>>>>>> 
>>>>>>> On Wed, 10 Mar 2010 17:27:05 -0300, Steve Ebersole<steve at hibernate.org>
>>>>>>> wrote:
>>>>>>> 
>>>>>>> 
>>>>>>> 
>>>>>>>> I want to push the release originally scheduled for today to next week.
>>>>>>>> We are down to docs and website.
>>>>>>>> 
>>>>>>>> There are still a few issues with the docs. After implementing the
>>>>>>>> monospace fonts for programlistigns I am now noticing that the callouts
>>>>>>>> are all no longer aligned properly (and actually its been suggested
>>>>>>>> many
>>>>>>>> of these be migrated away from programlisting/programlistingco
>>>>>>>> altogether. So I'd like the get that all cleaned up. There is also
>>>>>>>> still the minor issue of line spacing for programlisting in the pdf.
>>>>>>>> 
>>>>>>>> As for the site we are actually pretty close to done with what we need
>>>>>>>> to do for the new site.
>>>>>>>> 
>>>>>>>> In total the idea to push back to next week is to tie up those loose
>>>>>>>> ends and deliver them all at once along with the Final.
>>>>>>>> 
>>>>>>>> +/- ?
>>>>>>>> 
>>>>>>>> 
>>>>> _______________________________________________
>>>>> hibernate-dev mailing list
>>>>> hibernate-dev at lists.jboss.org
>>>>> https://lists.jboss.org/mailman/listinfo/hibernate-dev
>>>>> 
>>>>> 
>>>> 
>>>> _______________________________________________
>>>> hibernate-dev mailing list
>>>> hibernate-dev at lists.jboss.org
>>>> https://lists.jboss.org/mailman/listinfo/hibernate-dev
>>>> 
>>> 
>>> _______________________________________________
>>> hibernate-dev mailing list
>>> hibernate-dev at lists.jboss.org
>>> https://lists.jboss.org/mailman/listinfo/hibernate-dev
>> 
>> 
>> _______________________________________________
>> hibernate-dev mailing list
>> hibernate-dev at lists.jboss.org
>> https://lists.jboss.org/mailman/listinfo/hibernate-dev
> 
> -- 
> steve at hibernate.org
> http://hibernate.org





More information about the hibernate-dev mailing list