306 replace guava mediatype

[gnl42]

Replace guavas MediaType

two down, one to go

[gnl42]

@ruspl-afed Any chance you have 'ignore white space' disabled?

[ruspl-afed]

@ruspl-afed Any chance you have 'ignore white space' disabled?

Yes, this is how I was able to look at the distilled change.

[merks]

FYI, the changes here introduced a new dependency on org.apache.tika.core which needs to be in the update site:

https://github.com/eclipse-mylyn/org.eclipse.mylyn/pull/518

[ruspl-afed]

FYI, the changes here introduced a new dependency on org.apache.tika.core which needs to be in the update site:

518

Right, I forgot that we need to add every introduced 3rd party bundle to category.xml

[merks]

@ruspl-afed @gnl42

BTW, I noticed these are available newer versions

• https://repo1.maven.org/maven2/org/apache/tika/tika-core/1.28.5/
• https://repo1.maven.org/maven2/org/apache/tika/tika-core/2.9.2/

These are distributed as OSGi bundles and would be trivial to add to Orbit. It would make more sense to add the 2.x version rather than the 1.x version though. Would it make sense to add the 2.x version and test that it's a good replacement?

[gnl42]

Go for 2.x

[ruspl-afed]

Go for 2.x

[gnl42]

@merks Could you remind me how I can get apache commons-text added to Orbit?

[merks]

Open an issue here and ask nicely 😇

https://github.com/eclipse-orbit/orbit-simrel/issues

[merks]

Of course it pays off to see what's already there:

The tabular summary is quite helpful:

https://download.eclipse.org/tools/orbit/simrel/orbit-aggregation/table.html

[gnl42]

Assuming one even remembers that there is a milestone :)

The summary is exactly what I was about to ask for

Thanks