Bug#856693: Drop ant dependency

2017-03-05 Thread Rene Engelhard
Hi,

On Sun, Mar 05, 2017 at 11:32:53AM +0100, Emmanuel Bourg wrote:
> The ant dependency is only used for the excelant tasks [1] (which are
> never used in Debian [2]). Any use of these tasks implicitly means Ant
> is already on the classpath. In this context Ant can be seen as a
> runtime, much like the Servlet API for web based stuff. So I think it
> makes sense to remove the dependency from the binary package.

Yeah. Or making it a Suggests only.

> Splitting ant into ant+libant-java is a good idea, but this can probably
> wait for the Buster development cycle.

True, since it has the dependency already in jessie. Then again, the libreoffice
metapackage got improvements this release cycle:

libreoffice (1:5.2.1-2) unstable; urgency=medium

  * debian/rules: install typesizes.h into
usr/include/$(DEB_HOST_MULTIARCH)/libreoffice/sal/typesizes.h
  * debian/control.in:
- remove libreoffice-report-builder-bin [kfreebsd-i386 kfreebsd-amd64]
  Conflicts: from -core, thanks Jon Boden (closes: #836887)
- rework libreoffice metapackage "dependencies":
  + Recommend everything (except GNOME/KDE integration and similar) which
the upstream .tar.gz contains:
 libreoffice-script-provider-bsh,
 libreoffice-script-provider-js,
 libreoffice-script-provider-python,
 libreoffice-ogltrans,
 libreoffice-librelogo,
 libreoffice-nlpsolver,
 libreoffice-pdfimport,
 libreoffice-report-builder,
 libreoffice-wiki-publisher
 libreoffice-sdbc-postgresql
  + make libreoffice only Recommend the fonts. Mention it recommends
extra stuff like fonts to resemble a upstream install, thanks
Fabian Greffrath (closes: #836416)
- remove obsolete libreoffice-gtk conflicts which makes -core Break: it
  with bin-NMUs; add (intended) -gtk2/-gtk3 ones...

  * merge from Ubuntu:
- uno-libs: Create the libuno_* symlinks in the multiarch libdir

 -- Rene Engelhard   Tue, 13 Sep 2016 04:28:17 +0200

so that libreoffice now pulls this in via libreoffice-report-builder via its 
dependencies.

But given the minimal size saved, yeah, I agree the split isn't that important. 
The dependency
in libapache-poi-java itself can be changed, though, especially given the 
reasons stated by
Emmanuel.

Regards,

Rene

__
This is the maintainer address of Debian's Java team
. 
Please use
debian-j...@lists.debian.org for discussions and questions.


Bug#856693: Drop ant dependency

2017-03-05 Thread Emmanuel Bourg
The ant dependency is only used for the excelant tasks [1] (which are
never used in Debian [2]). Any use of these tasks implicitly means Ant
is already on the classpath. In this context Ant can be seen as a
runtime, much like the Servlet API for web based stuff. So I think it
makes sense to remove the dependency from the binary package.

Splitting ant into ant+libant-java is a good idea, but this can probably
wait for the Buster development cycle.

Emmanuel Bourg

[1] https://poi.apache.org/spreadsheet/excelant.html
[2] https://codesearch.debian.net/search?q=excelant

__
This is the maintainer address of Debian's Java team
. 
Please use
debian-j...@lists.debian.org for discussions and questions.


Bug#856693: Drop ant dependency

2017-03-04 Thread Rene Engelhard
Hi,

On Sat, Mar 04, 2017 at 11:40:53AM -0800, tony mancill wrote:
> were causing a serious issue, we could discuss splitting ant into
> something like "ant-bin" (maybe there's a better name) and libant-java,
> so only the latter library package is a dependency of
> libapache-poi-java.  
>
> In that case, I think we would still want to have an "ant" package that
> depended on both the library and the "ant-bin" cli build tools that are
> found in /usr/bin and /usr/share/ant/bin.  Perhaps another reason to

Or simply ant -> libant-java. IMHO no need for a extra ant-bin.

> But in terms of overhead, I don't think there's anything to be concerned
> about, because the ant wrapper scripts are tiny when compared to the jar
> file itself:
> 
> $ du -sh /usr/share/ant/bin
> 40K   /usr/share/ant/bin
> 
> $ du -shL /usr/share/java/ant.jar
> 2.0M  /usr/share/java/ant.jar
> 
> So this seems more like a question of style.  Do you still think the
> severity is important?

Think so, yes. Doesn't the Java policy also say you should split the library
out to a -java, anyways?

Regards,

Rene

__
This is the maintainer address of Debian's Java team
. 
Please use
debian-j...@lists.debian.org for discussions and questions.


Bug#856693: Drop ant dependency

2017-03-04 Thread tony mancill
On Sat, Mar 04, 2017 at 01:30:47AM +0100, Michael Biebl wrote:
> Source: libapache-poi-java
> Version: 3.10.1-3
> Severity: important
> 
> libapache-poi-java is an indirect dependency of libreoffice, so pulled
> in on every desktop installation.
> 
> Having a build tool like ant being pulled because of a library
> dependency is unwanted in such a case. Please consider dropping the
> dependency on ant.

Hi Michael,

Thank you for noticing this.  I agree that it look odd.  However, the
apache-poi sources use classes from ant.jar - for example, refer to the
classes in this package [1] - and currently that jar is shipped along
with the ant package.  I question whether it would worth it, but if this
were causing a serious issue, we could discuss splitting ant into
something like "ant-bin" (maybe there's a better name) and libant-java,
so only the latter library package is a dependency of
libapache-poi-java.  

In that case, I think we would still want to have an "ant" package that
depended on both the library and the "ant-bin" cli build tools that are
found in /usr/bin and /usr/share/ant/bin.  Perhaps another reason to
consider a split is because ant depends on a JRE (although I think
you're going to need one anyway in order to do anything useful with
apache-poi).

But in terms of overhead, I don't think there's anything to be concerned
about, because the ant wrapper scripts are tiny when compared to the jar
file itself:

$ du -sh /usr/share/ant/bin
40K /usr/share/ant/bin

$ du -shL /usr/share/java/ant.jar
2.0M/usr/share/java/ant.jar

So this seems more like a question of style.  Do you still think the
severity is important?

Cheers,
tony

[1] 
https://anonscm.debian.org/cgit/pkg-java/libapache-poi-java.git/tree/src/excelant/java/org/apache/poi/ss/excelant


signature.asc
Description: PGP signature
__
This is the maintainer address of Debian's Java team
. 
Please use
debian-j...@lists.debian.org for discussions and questions.

Bug#856693: Drop ant dependency

2017-03-03 Thread Michael Biebl
Source: libapache-poi-java
Version: 3.10.1-3
Severity: important

libapache-poi-java is an indirect dependency of libreoffice, so pulled
in on every desktop installation.

Having a build tool like ant being pulled because of a library
dependency is unwanted in such a case. Please consider dropping the
dependency on ant.


-- System Information:
Debian Release: 9.0
  APT prefers unstable-debug
  APT policy: (500, 'unstable-debug'), (500, 'unstable'), (500, 'stable'), 
(200, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 4.9.0-2-amd64 (SMP w/4 CPU cores)
Locale: LANG=de_DE.utf8, LC_CTYPE=de_DE.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

__
This is the maintainer address of Debian's Java team
. 
Please use
debian-j...@lists.debian.org for discussions and questions.