[Geoserver-devel] netcdf and slf4j

Hi all,

sorry for the build break on 2.8.x and master due to the slf4j upgrade. It was approved by Jody but I admit I didn’t run a full build. Looked easy enough ™ as the commit was a cherry pick from 2.7.x where I did run a full build, and netcdf there is a community module that has been promoted to an extension.

So for the time being I’ve reverted the commit on 2.8.x and master.

Looks like the netcdf modules require slf4j 1.6.4, and I think we should still homogenize the slf4j versions as we have currently 3 or 4 different ones spread across different poms.

That said, I’m totally fine with setting the slf4j version to 1.6.4 as required by netcdf for all the slf4j dependencies and call it a day. Is that ok?

Cheers,
Gabriel

···

Gabriel Roldán
Software Developer | Boundless
groldan@anonymised.com39…
@boundlessgeo

Gabriel, no good deed goes unpunished. :slight_smile:

+1 for pinning slf4j to 1.6.4.

Starting from your previous fix on master (1e84d25037d256b9777239f4743e43498daeec78), if I change slf4j.version to 1.6.4, both gs-grib and gs-netcdf-out build correctly (and I think these were the only ones failing).

Kind regards,
Ben.

On 02/02/16 08:44, Gabriel Roldan wrote:

Hi all,

sorry for the build break on 2.8.x and master due to the slf4j upgrade. It
was approved by Jody but I admit I didn't run a full build. Looked easy
enough (tm) as the commit was a cherry pick from 2.7.x where I did run a
full build, and netcdf there is a community module that has been promoted
to an extension.

So for the time being I've reverted the commit on 2.8.x and master.

Looks like the netcdf modules require slf4j 1.6.4, and I think we should
still homogenize the slf4j versions as we have currently 3 or 4 different
ones spread across different poms.

That said, I'm totally fine with setting the slf4j version to 1.6.4 as
required by netcdf for all the slf4j dependencies and call it a day. Is
that ok?

Cheers,
Gabriel

------------------------------------------------------------------------------
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=267308311&iu=/4140

_______________________________________________
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

--
Ben Caradoc-Davies <ben@anonymised.com>
Director
Transient Software Limited <http://transient.nz/&gt;
New Zealand

On Mon, Feb 1, 2016 at 5:22 PM, Ben Caradoc-Davies <ben@anonymised.com> wrote:

Gabriel, no good deed goes unpunished. :slight_smile:

+1 for pinning slf4j to 1.6.4.

Starting from your previous fix on master
(1e84d25037d256b9777239f4743e43498daeec78), if I change slf4j.version to
1.6.4, both gs-grib and gs-netcdf-out build correctly (and I think these
were the only ones failing).

right, okay so lets do that.

Thanks Ben

Kind regards,
Ben.

On 02/02/16 08:44, Gabriel Roldan wrote:

Hi all,

sorry for the build break on 2.8.x and master due to the slf4j upgrade. It
was approved by Jody but I admit I didn't run a full build. Looked easy
enough (tm) as the commit was a cherry pick from 2.7.x where I did run a
full build, and netcdf there is a community module that has been promoted
to an extension.

So for the time being I've reverted the commit on 2.8.x and master.

Looks like the netcdf modules require slf4j 1.6.4, and I think we should
still homogenize the slf4j versions as we have currently 3 or 4 different
ones spread across different poms.

That said, I'm totally fine with setting the slf4j version to 1.6.4 as
required by netcdf for all the slf4j dependencies and call it a day. Is
that ok?

Cheers,
Gabriel

------------------------------------------------------------------------------
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=267308311&iu=/4140

_______________________________________________
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

--
Ben Caradoc-Davies <ben@anonymised.com>
Director
Transient Software Limited <http://transient.nz/&gt;
New Zealand

--

Gabriel Roldán
Software Developer | Boundless <http://boundlessgeo.com/&gt;
groldan@anonymised.com
@boundlessgeo <http://twitter.com/boundlessgeo/&gt;