Re: Upgrading freehep to latest upstream (needed by cdk and probably others)

[Giovanni Mascellani <gio@debian.org>]

Hi,

Il 09/07/20 08:38, merkys@debian.org ha scritto:
> * As soon as new FreeHEP is accepted to experimental, I initiate the
> appropriate transition.
> 
> * As GeoGebra will FTBFS with the new FreeHEP, I will ask to remove the
> former from testing to complete the transition.

There is no need for this step. I have already a patched version that
compiles with the new FreeHEP. The thing I ask you is to add this patch
to freehep-vectorgraphics:

> diff --git a/freehep-graphicsio-svg/src/main/java/org/freehep/graphicsio/svg/SVGGraphics2D.java b/freehep-graphicsio-svg/src/main/java/org/freehep/graphicsio/svg/SVGGraphics2D.java
> index 27fcb1d..84004cc 100644
> --- a/freehep-graphicsio-svg/src/main/java/org/freehep/graphicsio/svg/SVGGraphics2D.java
> +++ b/freehep-graphicsio-svg/src/main/java/org/freehep/graphicsio/svg/SVGGraphics2D.java
> @@ -155,7 +155,7 @@ public class SVGGraphics2D extends AbstractVectorGraphicsIO {
>      // The private writer used for this file.
>      private OutputStream ros;
>  
> -    private PrintWriter os;
> +    protected PrintWriter os;
>  
>      // table for gradients
>      Hashtable<GradientPaint, String> gradients = new Hashtable<GradientPaint, String>();

Notice that this patch is harmless to other reverse dependencies of
freehep-vectorgraphics, because it makes a field available to
subclasses, but doesn't change the behavior for subclasses that don't
touch it.

Thanks, Giovanni.
-- 
Giovanni Mascellani <g.mascellani@gmail.com>
Postdoc researcher - Université Libre de Bruxelles