Bug#594270: libjcharts-java should depend on javaX-runtime

2010-08-26 Thread Thadeu Lima de Souza Cascardo
On Thu, Aug 26, 2010 at 12:34:46AM +0530, Onkar Shinde wrote:
 A look at the source tells me that there is only one file using Sun
 specific APIs - src/org/jCharts/encoders/JPEGEncoder13.java. Also it
 looks like this file has been superseeded by
 src/org/jCharts/encoders/JPEGEncoder.java which uses java standard
 APIs. So it makes sense to not compile JPEGEncoder13.java file.
 
 Other files referring to JPEGEncoder13.java are
 1. src/org/jCharts/test/ChartTestDriver.java - tests the encoding
 using JPEGEncoder13.
 2. src/org/jCharts/encoders/ServletEncoderHelper.java - provides an
 API to be used by servlets/jsps using JPEGEncoder13.
 It should be easy to patch the sources to exclude these sources.
 
 By the way, we currently do not run tests. Now may be good time to enable 
 them.
 
 Onkar

Hello, folks.

I could not run jmeter using gcj, after rebuilding jcharts with a simple
change in the Depends to allow the removal of openjdk. I don't think
jcharts is to blame here. I've got a problem with awt gtk, with a
division by zero.

Anyway, thanks for investigating this. I may be reporting another bug
regarding this division by zero to the proper package as soon as I have
a chance to investigate it.

Shall we change the severity of the bug to wishlist?

Regards,
Cascardo.


signature.asc
Description: Digital signature
__
This is the maintainer address of Debian's Java team
http://lists.alioth.debian.org/mailman/listinfo/pkg-java-maintainers. Please 
use
debian-j...@lists.debian.org for discussions and questions.


Bug#594270: libjcharts-java should depend on javaX-runtime

2010-08-25 Thread Onkar Shinde
On Wed, Aug 25, 2010 at 10:41 AM, tony mancill tmanc...@debian.org wrote:
 On 08/24/2010 08:01 PM, Thadeu Lima de Souza Cascardo wrote:
 Package: libjcharts-java
 Version: 0.7.5-1
 Severity: normal

 libjcharts-java depends on openjdk or sun jre explictly. It does not
 allow the use of other java runtime (like gcj), which would be possible
 if it depended on java5-runtime, for example. I don't know what version
 is needed for libjcharts-java, so I am not providing a patch.

 It already depends on another java library (libbatik-java), which
 depends on java2-runtime-headless. I am not sure libjcharts-java is
 headless, but this should also be considered.

 Given that this source package only generates a single library package
 which doesn't appear that it's used as a stand-alone app (I may be
 incorrect on that, but I didn't see anything about it on the jCharts
 site), there needn't be a runtime JRE dependency at all.

 Any objections to removing the JRE from Depends completely?

IIRC, jcharts uses some Sun specific APIs which are only available in
Sun JRE and OpenJDK JRE.
If this isn't the case then the restricted dependencies is a packaging
mistake on my part. Please feel free to remove JRE dependency after
verification.


Onkar
-- 
Passion - Some people climb mountains - others write Free software.
Don't ask why - the reason is the same.



__
This is the maintainer address of Debian's Java team
http://lists.alioth.debian.org/mailman/listinfo/pkg-java-maintainers. Please 
use
debian-j...@lists.debian.org for discussions and questions.


Bug#594270: libjcharts-java should depend on javaX-runtime

2010-08-25 Thread tony mancill
On 08/25/2010 01:29 AM, Onkar Shinde wrote:
 On Wed, Aug 25, 2010 at 10:41 AM, tony mancill tmanc...@debian.org wrote:
 On 08/24/2010 08:01 PM, Thadeu Lima de Souza Cascardo wrote:
 Package: libjcharts-java
 Version: 0.7.5-1
 Severity: normal

 libjcharts-java depends on openjdk or sun jre explictly. It does not
 allow the use of other java runtime (like gcj), which would be possible
 if it depended on java5-runtime, for example. I don't know what version
 is needed for libjcharts-java, so I am not providing a patch.

 It already depends on another java library (libbatik-java), which
 depends on java2-runtime-headless. I am not sure libjcharts-java is
 headless, but this should also be considered.

 Given that this source package only generates a single library package
 which doesn't appear that it's used as a stand-alone app (I may be
 incorrect on that, but I didn't see anything about it on the jCharts
 site), there needn't be a runtime JRE dependency at all.

 Any objections to removing the JRE from Depends completely?
 
 IIRC, jcharts uses some Sun specific APIs which are only available in
 Sun JRE and OpenJDK JRE.
 If this isn't the case then the restricted dependencies is a packaging
 mistake on my part. Please feel free to remove JRE dependency after
 verification.
 
 
 Onkar

Hi Onkar,

That would be a good reason for the runtime dependency.  :)  Since I
don't have any sort of a comprehensive test suite for jcharts and you
have already looked at the issue at least once when packaging it
initially, I think it's best to leave the package as is for the short term.

Thadeu, have you tested jcharts with gcj?  If so, do you have an idea of
how much coverage that testing provides?

Thank you,
tony



signature.asc
Description: OpenPGP digital signature
__
This is the maintainer address of Debian's Java team
http://lists.alioth.debian.org/mailman/listinfo/pkg-java-maintainers. Please 
use
debian-j...@lists.debian.org for discussions and questions.


Bug#594270: libjcharts-java should depend on javaX-runtime

2010-08-25 Thread Thadeu Lima de Souza Cascardo
On Wed, Aug 25, 2010 at 07:18:52AM -0700, tony mancill wrote:
 On 08/25/2010 01:29 AM, Onkar Shinde wrote:
  On Wed, Aug 25, 2010 at 10:41 AM, tony mancill tmanc...@debian.org wrote:
  On 08/24/2010 08:01 PM, Thadeu Lima de Souza Cascardo wrote:
  Package: libjcharts-java
  Version: 0.7.5-1
  Severity: normal
 
  libjcharts-java depends on openjdk or sun jre explictly. It does not
  allow the use of other java runtime (like gcj), which would be possible
  if it depended on java5-runtime, for example. I don't know what version
  is needed for libjcharts-java, so I am not providing a patch.
 
  It already depends on another java library (libbatik-java), which
  depends on java2-runtime-headless. I am not sure libjcharts-java is
  headless, but this should also be considered.
 
  Given that this source package only generates a single library package
  which doesn't appear that it's used as a stand-alone app (I may be
  incorrect on that, but I didn't see anything about it on the jCharts
  site), there needn't be a runtime JRE dependency at all.
 
  Any objections to removing the JRE from Depends completely?
  
  IIRC, jcharts uses some Sun specific APIs which are only available in
  Sun JRE and OpenJDK JRE.
  If this isn't the case then the restricted dependencies is a packaging
  mistake on my part. Please feel free to remove JRE dependency after
  verification.
  
  
  Onkar
 
 Hi Onkar,
 
 That would be a good reason for the runtime dependency.  :)  Since I
 don't have any sort of a comprehensive test suite for jcharts and you
 have already looked at the issue at least once when packaging it
 initially, I think it's best to leave the package as is for the short term.
 
 Thadeu, have you tested jcharts with gcj?  If so, do you have an idea of
 how much coverage that testing provides?
 
 Thank you,
 tony
 

I am a newbie when it comes to Java. But I may run some tests here. One
of the tests I could make is running jmeter, which is the software I am
currently using. If it does generate the graphics we need, that would be
OK for us. But I agree that would not be complete coverage. Is running
the test suites in the package enough?

Thanks,
Cascardo.


signature.asc
Description: Digital signature
__
This is the maintainer address of Debian's Java team
http://lists.alioth.debian.org/mailman/listinfo/pkg-java-maintainers. Please 
use
debian-j...@lists.debian.org for discussions and questions.


Bug#594270: libjcharts-java should depend on javaX-runtime

2010-08-25 Thread Onkar Shinde
A look at the source tells me that there is only one file using Sun
specific APIs - src/org/jCharts/encoders/JPEGEncoder13.java. Also it
looks like this file has been superseeded by
src/org/jCharts/encoders/JPEGEncoder.java which uses java standard
APIs. So it makes sense to not compile JPEGEncoder13.java file.

Other files referring to JPEGEncoder13.java are
1. src/org/jCharts/test/ChartTestDriver.java - tests the encoding
using JPEGEncoder13.
2. src/org/jCharts/encoders/ServletEncoderHelper.java - provides an
API to be used by servlets/jsps using JPEGEncoder13.
It should be easy to patch the sources to exclude these sources.

By the way, we currently do not run tests. Now may be good time to enable them.


Onkar



__
This is the maintainer address of Debian's Java team
http://lists.alioth.debian.org/mailman/listinfo/pkg-java-maintainers. Please 
use
debian-j...@lists.debian.org for discussions and questions.


Bug#594270: libjcharts-java should depend on javaX-runtime

2010-08-24 Thread Thadeu Lima de Souza Cascardo
Package: libjcharts-java
Version: 0.7.5-1
Severity: normal

libjcharts-java depends on openjdk or sun jre explictly. It does not
allow the use of other java runtime (like gcj), which would be possible
if it depended on java5-runtime, for example. I don't know what version
is needed for libjcharts-java, so I am not providing a patch.

It already depends on another java library (libbatik-java), which
depends on java2-runtime-headless. I am not sure libjcharts-java is
headless, but this should also be considered.

-- System Information:
Debian Release: squeeze/sid
  APT prefers testing
  APT policy: (990, 'testing'), (500, 'unstable')
Architecture: i386 (x86_64)

Kernel: Linux 2.6.36-rc1 (SMP w/2 CPU cores; PREEMPT)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages libjcharts-java depends on:
ii  libbatik-java 1.7-6  xml.apache.org SVG Library
ii  openjdk-6-jre 6b18-1.8-1 OpenJDK Java runtime, using Hotspo

libjcharts-java recommends no packages.

libjcharts-java suggests no packages.

-- no debconf information



__
This is the maintainer address of Debian's Java team
http://lists.alioth.debian.org/mailman/listinfo/pkg-java-maintainers. Please 
use
debian-j...@lists.debian.org for discussions and questions.


Bug#594270: libjcharts-java should depend on javaX-runtime

2010-08-24 Thread tony mancill
On 08/24/2010 08:01 PM, Thadeu Lima de Souza Cascardo wrote:
 Package: libjcharts-java
 Version: 0.7.5-1
 Severity: normal
 
 libjcharts-java depends on openjdk or sun jre explictly. It does not
 allow the use of other java runtime (like gcj), which would be possible
 if it depended on java5-runtime, for example. I don't know what version
 is needed for libjcharts-java, so I am not providing a patch.
 
 It already depends on another java library (libbatik-java), which
 depends on java2-runtime-headless. I am not sure libjcharts-java is
 headless, but this should also be considered.

Given that this source package only generates a single library package
which doesn't appear that it's used as a stand-alone app (I may be
incorrect on that, but I didn't see anything about it on the jCharts
site), there needn't be a runtime JRE dependency at all.

Any objections to removing the JRE from Depends completely?

Thanks,
tony



signature.asc
Description: OpenPGP digital signature
__
This is the maintainer address of Debian's Java team
http://lists.alioth.debian.org/mailman/listinfo/pkg-java-maintainers. Please 
use
debian-j...@lists.debian.org for discussions and questions.