[GeoNetwork-devel] Upgrading jetty to version 8 on develop

On 2.10.x we upgraded from Jetty 7.3.1 to Jetty 7.6.8 in order to fix a JSP
compilation issue. I had suggested that we should be going even further in
the develop branch by upgrading to a more recent version of Jetty. So I have
made a quick attempt to upgrade to 8.1.12 and I have it mostly working
however I have hit a bug with the current version of ICU4J

http://icu-project.org/trac/ticket/6505

The fix involves upgrading ICU4J from 2.6 to 2.8 on geonetwork and
geoserver. Could someone point me to where and how to proceed with this
upgrade -I can't seem to find it. Also I don't know much regarding ICU4J so
does anyone know if it will cause any issues if we perform this upgrade? I
did manage to replace all occurrences of icu4j-2.6.1.jar by icu4j_2_8.jar in
the lib folders and it solved the issue with Jetty 8 - I'm just not sure if
it introduced any new bugs?

Note: I originally attempted to upgrade to Jetty 9 however it required Java
1.7. I did think it was worth forcing the requirement of Java 1.7 based on
Jetty. When future changes change the requirements to Java 1.7 then we can
look at upgrading Jetty to version 9 and beyond.

--
View this message in context: http://osgeo-org.1560.x6.nabble.com/Upgrading-jetty-to-version-8-on-develop-tp5072040.html
Sent from the GeoNetwork developer mailing list archive at Nabble.com.

Ian,

2013/8/12 ianwallen <ianwallen@anonymised.com>:

On 2.10.x we upgraded from Jetty 7.3.1 to Jetty 7.6.8 in order to fix a JSP
compilation issue. I had suggested that we should be going even further in
the develop branch by upgrading to a more recent version of Jetty. So I have
made a quick attempt to upgrade to 8.1.12 and I have it mostly working
however I have hit a bug with the current version of ICU4J

http://icu-project.org/trac/ticket/6505

The fix involves upgrading ICU4J from 2.6 to 2.8 on geonetwork and
geoserver. Could someone point me to where and how to proceed with this
upgrade -I can't seem to find it. Also I don't know much regarding ICU4J so
does anyone know if it will cause any issues if we perform this upgrade?

ICU4J is a dependency of Jaxen.
Upgrading jaxen to 1.1.4 in pom.xml fixed the issue. See
https://github.com/geonetwork/core-geonetwork/issues/223

So far, I did not noticed any issue.

Cheers.

Francois

I
did manage to replace all occurrences of icu4j-2.6.1.jar by icu4j_2_8.jar in
the lib folders and it solved the issue with Jetty 8 - I'm just not sure if
it introduced any new bugs?

Note: I originally attempted to upgrade to Jetty 9 however it required Java
1.7. I did think it was worth forcing the requirement of Java 1.7 based on
Jetty. When future changes change the requirements to Java 1.7 then we can
look at upgrading Jetty to version 9 and beyond.

--
View this message in context: http://osgeo-org.1560.x6.nabble.com/Upgrading-jetty-to-version-8-on-develop-tp5072040.html
Sent from the GeoNetwork developer mailing list archive at Nabble.com.

------------------------------------------------------------------------------
Get 100% visibility into Java/.NET code with AppDynamics Lite!
It's a free troubleshooting tool designed for production.
Get down to code-level detail for bottlenecks, with <2% overhead.
Download for free and get started troubleshooting in minutes.
http://pubads.g.doubleclick.net/gampad/clk?id=48897031&iu=/4140/ostg.clktrk
_______________________________________________
GeoNetwork-devel mailing list
GeoNetwork-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geonetwork-devel
GeoNetwork OpenSource is maintained at http://sourceforge.net/projects/geonetwork

Francois,

Thank you for that information. - I knew it was an easy fix - just could not
find it.

I noticed that you called the issue "Upgrade Jetty to 8.x or 9x version".
Are you planning on going with 9 (and make geonetwork dependent on Java 1.7)
or are you going with version 8? Just curious - thanks.

--
View this message in context: http://osgeo-org.1560.x6.nabble.com/Upgrading-jetty-to-version-8-on-develop-tp5072040p5072049.html
Sent from the GeoNetwork developer mailing list archive at Nabble.com.

2013/8/12 ianwallen <ianwallen@anonymised.com>:

Francois,

Thank you for that information. - I knew it was an easy fix - just could not
find it.

I noticed that you called the issue "Upgrade Jetty to 8.x or 9x version".
Are you planning on going with 9 (and make geonetwork dependent on Java 1.7)
or are you going with version 8? Just curious - thanks.

8 is fine for me and I only find jetty maven plugin for 8 version.

Francois

--
View this message in context: http://osgeo-org.1560.x6.nabble.com/Upgrading-jetty-to-version-8-on-develop-tp5072040p5072049.html
Sent from the GeoNetwork developer mailing list archive at Nabble.com.

------------------------------------------------------------------------------
Get 100% visibility into Java/.NET code with AppDynamics Lite!
It's a free troubleshooting tool designed for production.
Get down to code-level detail for bottlenecks, with <2% overhead.
Download for free and get started troubleshooting in minutes.
http://pubads.g.doubleclick.net/gampad/clk?id=48897031&iu=/4140/ostg.clktrk
_______________________________________________
GeoNetwork-devel mailing list
GeoNetwork-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geonetwork-devel
GeoNetwork OpenSource is maintained at http://sourceforge.net/projects/geonetwork