Re: svn commit: r1234877 - in /xmlgraphics/fop/trunk: examples/embedding/java/embedding/ examples/embedding/java/embedding/atxml/ src/java/org/apache/fop/cli/ src/java/org/apache/fop/render/ src/java/

2012-01-23 Thread J.Pietschmann
Hi Mehdi! Am 23.01.2012 17:15, schrieb me...@apache.org: > This breaks the public API but for good reasons: Good reasons or not, there are a few guidelines when it comes to changing public interfaces: - There has to be a discussion of the change. Alternatively, open a bugzilla ticket and provid

RE: svn commit: r1234877 - in /xmlgraphics/fop/trunk: examples/embedding/java/embedding/ examples/embedding/java/embedding/atxml/ src/java/org/apache/fop/cli/ src/java/org/apache/fop/render/ src/java/

2012-01-23 Thread Jonathan Levinson
I have no vote, but I’m not happy with the change since it will break our RenderServer.jar which is a client-server we use for giving rendering tasks to FOP. In the worst case, we will have to build two versions of the jar – one for fop 1.0 and the other for people who are using trunk. This is

Re: svn commit: r1234877 - in /xmlgraphics/fop/trunk: examples/embedding/java/embedding/ examples/embedding/java/embedding/atxml/ src/java/org/apache/fop/cli/ src/java/org/apache/fop/render/ src/java/

2012-01-23 Thread Glenn Adams
I'm disturbed that such a change has been committed without a public discussion of the merits, risks, etc., of making such a breaking change. Please revert this commit, conduct a public discussion, then, based on the results, implement the consensus. I have no idea if there is a consensus for this

DO NOT REPLY [Bug 52499] [PATCH] non embeddable TTF fonts -- throw exception if trying to embed

2012-01-23 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=52499 Mehdi Houshmand changed: What|Removed |Added Status|NEW |RESOLVED Resolution|

DO NOT REPLY [Bug 52499] [PATCH] non embeddable TTF fonts -- throw exception if trying to embed

2012-01-23 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=52499 --- Comment #2 from Mehdi Houshmand 2012-01-23 12:08:38 UTC --- Patch applied to r1234761. Thanks for this patch. -- Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail becau

DO NOT REPLY [Bug 52499] [PATCH] non embeddable TTF fonts -- throw exception if trying to embed

2012-01-23 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=52499 --- Comment #1 from Luis Bernardo 2012-01-23 10:58:22 UTC --- Created attachment 28189 --> https://issues.apache.org/bugzilla/attachment.cgi?id=28189 the patch Note: to see the issue one needs a TTF that is non embeddable (usually due t

DO NOT REPLY [Bug 52499] New: [PATCH] non embeddable TTF fonts -- throw exception if trying to embed

2012-01-23 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=52499 Bug #: 52499 Summary: [PATCH] non embeddable TTF fonts -- throw exception if trying to embed Product: Fop Version: all Platform: All OS/Version: All