You are viewing a plain text version of this content. The canonical link for it is here.
Posted to fop-dev@xmlgraphics.apache.org by Vincent Hennebert <vh...@gmail.com> on 2014/06/06 19:50:06 UTC

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

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

Posted by Chris Bowditch <bo...@hotmail.com>.
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_chris@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:vhennebert@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:vhennebert@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
>>>>
>>
>
>
>


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

Posted by Simon Steiner <si...@gmail.com>.
Hi,

I updated the branch to use jacoco instead of emma.

Thanks

-----Original Message-----
From: Chris Bowditch [mailto:bowditch_chris@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:vhennebert@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:vhennebert@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
>>>
>>
>
>



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

Posted by Chris Bowditch <bo...@hotmail.com>.
+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:vhennebert@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:vhennebert@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
>>>
>>
>
>


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

Posted by Vincent Hennebert <vh...@gmail.com>.
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:vhennebert@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:vhennebert@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
>>
>

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

Posted by Simon Steiner <si...@gmail.com>.
Hi,

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

Thanks

-----Original Message-----
From: Vincent Hennebert [mailto:vhennebert@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:vhennebert@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
>


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

Posted by Vincent Hennebert <vh...@gmail.com>.
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:vhennebert@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
>

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

Posted by Simon Steiner <si...@gmail.com>.
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.

Thanks

-----Original Message-----
From: Vincent Hennebert [mailto:vhennebert@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