Re: [VOTE] Merge Temp_FontMerging to trunk

2014-06-23 Thread Pascal Sancho
Hi,

Very good work, Simon.

+1

2014-06-23 17:34 GMT+02:00 Clay Leeds :
> Will there be 'one last release using Java 5’ enabling the most widespread 
> use of the latest FOP development or does this mean that the last version of 
> Java 5 support will be FOP 1.1?
>
> Either way, I think this is a great improvement!
>
> Great work, Simon!
>
> +1 from me.
>
> On Jun 23, 2014, at 8:11 AM, Chris Bowditch  
> wrote:
>> Hi Simon,
>>
>> This branch represents many months of hard work. Merging fonts within 
>> separate PDFs is far from a trivial task, and has been requested by almost 
>> everyone who uses the PDF Plug-in. Thanks you and well done on bringing the 
>> implementation to this point.
>>
>> I downloaded the branch and ran a couple of basic tests, seems ok from a 
>> high level. Therefore I am +1 to merge this branch into trunk
>>
>> Thanks,
>>
>> Chris
>>
>> On 20/06/2014 12:23, Simon Steiner wrote:
>>>
>>> Hi,
>>>
>>> I have been working on merging fonts inside pdf external graphic using the 
>>> pdf-plugin. The feature is disabled by default and can be enabled using 
>>> fop.xconf:
>>>
>>> 
>>>
>>>  
>>>
>>>
>>>
>>> true
>>>
>>>
>>>
>>>  
>>>
>>> 
>>>
>>> It is using pdfbox 2.0 snapshot which requires Java 6 or later so you vote 
>>> needs to agree to end support for Java 5 on trunk.
>>>
>>> It is supporting fonts of type CFF, Truetype, CID and Type1.
>>>
>>> The vote will last 5 working days, ending next Friday.
>>>
>>> https://issues.apache.org/jira/browse/FOP-2302
>>>
>>> https://svn.apache.org/repos/asf/xmlgraphics/fop-pdf-images/branches/Temp_FontMerging
>>>
>>> https://svn.apache.org/repos/asf/xmlgraphics/fop/branches/Temp_FontMerging
>>>
>>> Here is my vote: +1
>>>
>>> Thanks
>>>
>>
>



-- 
pascal


Re: [VOTE] Merge Temp_FontMerging to trunk

2014-06-23 Thread Clay Leeds
Will there be 'one last release using Java 5’ enabling the most widespread use 
of the latest FOP development or does this mean that the last version of Java 5 
support will be FOP 1.1?

Either way, I think this is a great improvement!

Great work, Simon!

+1 from me.

On Jun 23, 2014, at 8:11 AM, Chris Bowditch  wrote:
> Hi Simon,
> 
> This branch represents many months of hard work. Merging fonts within 
> separate PDFs is far from a trivial task, and has been requested by almost 
> everyone who uses the PDF Plug-in. Thanks you and well done on bringing the 
> implementation to this point.
> 
> I downloaded the branch and ran a couple of basic tests, seems ok from a high 
> level. Therefore I am +1 to merge this branch into trunk
> 
> Thanks,
> 
> Chris
> 
> On 20/06/2014 12:23, Simon Steiner wrote:
>> 
>> Hi,
>> 
>> I have been working on merging fonts inside pdf external graphic using the 
>> pdf-plugin. The feature is disabled by default and can be enabled using 
>> fop.xconf:
>> 
>> 
>> 
>>  
>> 
>>
>> 
>> true
>> 
>>
>> 
>>  
>> 
>> 
>> 
>> It is using pdfbox 2.0 snapshot which requires Java 6 or later so you vote 
>> needs to agree to end support for Java 5 on trunk.
>> 
>> It is supporting fonts of type CFF, Truetype, CID and Type1.
>> 
>> The vote will last 5 working days, ending next Friday.
>> 
>> https://issues.apache.org/jira/browse/FOP-2302
>> 
>> https://svn.apache.org/repos/asf/xmlgraphics/fop-pdf-images/branches/Temp_FontMerging
>> 
>> https://svn.apache.org/repos/asf/xmlgraphics/fop/branches/Temp_FontMerging
>> 
>> Here is my vote: +1
>> 
>> Thanks
>> 
> 



Re: [VOTE] Merge Temp_FontMerging to trunk

2014-06-23 Thread Chris Bowditch

Hi Simon,

This branch represents many months of hard work. Merging fonts within 
separate PDFs is far from a trivial task, and has been requested by 
almost everyone who uses the PDF Plug-in. Thanks you and well done on 
bringing the implementation to this point.


I downloaded the branch and ran a couple of basic tests, seems ok from a 
high level. Therefore I am +1 to merge this branch into trunk


Thanks,

Chris

On 20/06/2014 12:23, Simon Steiner wrote:


Hi,

I have been working on merging fonts inside pdf external graphic using 
the pdf-plugin. The feature is disabled by default and can be enabled 
using fop.xconf:




  



true



  



It is using pdfbox 2.0 snapshot which requires Java 6 or later so you 
vote needs to agree to end support for Java 5 on trunk.


It is supporting fonts of type CFF, Truetype, CID and Type1.

The vote will last 5 working days, ending next Friday.

https://issues.apache.org/jira/browse/FOP-2302

https://svn.apache.org/repos/asf/xmlgraphics/fop-pdf-images/branches/Temp_FontMerging

https://svn.apache.org/repos/asf/xmlgraphics/fop/branches/Temp_FontMerging

Here is my vote: +1

Thanks





Re: svn commit: r1600897 - /xmlgraphics/fop-pdf-images/branches/Temp_FontMerging/build.xml

2014-06-23 Thread Chris Bowditch

Thank you Simon

On 20/06/2014 12:20, Simon Steiner wrote:

Hi,

I updated the branch to use jacoco instead of emma.

Thanks

-Original Message-
From: Chris Bowditch [mailto:bowditch_ch...@hotmail.com]
Sent: 18 June 2014 09:02
To: fop-dev@xmlgraphics.apache.org
Subject: Re: svn commit: r1600897 - 
/xmlgraphics/fop-pdf-images/branches/Temp_FontMerging/build.xml

+1

We should try to be consistent in which tools we use across the sub projects

Thanks,

Chris

On 16/06/2014 18:35, Vincent Hennebert wrote:

On 16/06/14 17:50, Simon Steiner wrote:

Hi,

Its there in fop, its not in pdfplugin which is where I added it.

Ah, sorry, I didn’t realise the modification was made in the PDF
Images plug-in. I guess it’s ok then, although it would be better to
use the same tool in every sub-project.

Vincent



Thanks

-Original Message-
From: Vincent Hennebert [mailto:vhenneb...@gmail.com]
Sent: 16 June 2014 16:32
To: fop-dev@xmlgraphics.apache.org
Subject: Re: svn commit: r1600897 -
/xmlgraphics/fop-pdf-images/branches/Temp_FontMerging/build.xml

Hi Simon,

On 16/06/14 17:13, Simon Steiner wrote:

Hi,

Seems we have JaCoCo on trunk, I forgot about that. I added emma to
pdfplugin since i have used that tool in the past. I can revert this
change if emma is not allowed.

It’s not so much that Emma is not allowed, it’s that JaCoCo is
already there that provides the same functionality, plus it’s meant
to be the successor of Emma. So we might as well use it.

Thanks,
Vincent



Thanks

-Original Message-
From: Vincent Hennebert [mailto:vhenneb...@gmail.com]
Sent: 06 June 2014 18:50
To: fop-dev@xmlgraphics.apache.org
Subject: Re: svn commit: r1600897 -
/xmlgraphics/fop-pdf-images/branches/Temp_FontMerging/build.xml

On 06/06/14 16:21, ssteiner wrote:

Author: ssteiner
Date: Fri Jun  6 14:21:57 2014
New Revision: 1600897

URL: http://svn.apache.org/r1600897
Log:
Add emma

Why, when we already have JaCoCo?
https://issues.apache.org/jira/browse/FOP-1991

Vincent