Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 381629

Summary:	Missing media type for font files
Product:	z_Archived	Reporter:	Torkild Resheim <torkildr>
Component:	Mylyn	Assignee:	Torkild Resheim <torkildr>
Status:	CLOSED FIXED	QA Contact:
Severity:	major
Priority:	P3
Version:	1.7
Target Milestone:	1.7.0
Hardware:	All
OS:	All
Whiteboard:
Bug Depends on:
Bug Blocks:	380104

Description Torkild Resheim

2012-06-04 15:33:02 EDT

Just found a problem with the EPUB specification which does not allow TTF fonts as it's not a core media type. This causes the EPUB validation to barf when such fonts are present. The validator should instead issue a warning as this font type is fairly common and one cannot expect a fallback to be presented. See http://www.daisy.org/epub/issues/missing-media-type-font-files

Comment 1 Torkild Resheim

2012-06-04 16:03:28 EDT

Done.

Comment 2 Steffen Pingel

2012-06-04 16:20:08 EDT

Not sure if it's related but two tests appear to be failing now: https://hudson.eclipse.org/hudson/user/spingel/my-views/view/Mylyn/job/mylyn-docs-nightly/lastCompletedBuild/testReport/org.eclipse.mylyn.docs.epub.tests.api/TestOPS2Publication/

junit.framework.AssertionFailedError: Exception should be thrown
	at junit.framework.Assert.fail(Assert.java:47)
	at org.eclipse.mylyn.docs.epub.tests.api.TestOPS2Publication.test_Bug358671_Illegal_Fallback(TestOPS2Publication.java:253)
	
junit.framework.AssertionFailedError: Exception should be thrown
	at junit.framework.Assert.fail(Assert.java:47)
	at org.eclipse.mylyn.docs.epub.tests.api.TestOPS2Publication.test_Bug358671_Illegal_Item(TestOPS2Publication.java:226)

Comment 3 Torkild Resheim

2012-06-04 16:23:07 EDT

Yes... I pushed a changed to Gerrit, remembered that I forgot to fix the tests, fixed it, amended the commit and pushed again. However I did not wait for the confirmation message. A few minutes later I merged the change in - not noticing that the latest push was not received. There was a communication error. I've fixed it in another commit that has been merged in - so it should be OK in the next build.

Sorry about the trouble.

Comment 4 Steffen Pingel

2012-06-04 16:25:33 EDT

No problem at all. Thanks for looking into it!

Comment 5 Torkild Resheim

2012-07-23 16:14:41 EDT

Closing resolved issues in released product.