Bug 143433 - dev-java/commons-fileupload-1.1.1 version bump
Bug#: 143433 Product:  Gentoo Linux Version: 2006.0 Platform: All
OS/Version: Linux Status: RESOLVED Severity: enhancement Priority: P2
Resolution: FIXED Assigned To: java@gentoo.org Reported By: kleini76@gmail.com
Component: Ebuilds
URL: 
Summary: dev-java/commons-fileupload-1.1.1 version bump
Keywords:  InOverlay
Status Whiteboard: 
Opened: 2006-08-10 02:13 0000
Description:   Opened: 2006-08-10 02:13 0000
On http://jakarta.apache.org/commons/fileupload/ version 1.1.1 of
commons-fileupload is available. An ebuild is currently only available for
version 1.0. Version 1.1 is available for half a year but no ebuild has been
made for it.

------- Comment #1 From Petteri Räty 2006-08-12 06:03:39 0000 -------
Initial ebuild here:
https://overlays.gentoo.org/svn/proj/java/migrated-java-experimental-overlay/dev-java/commons-fileupload

This does not work yet because we don't have portlet-api packaged. 

------- Comment #2 From Miroslav Šulc 2006-09-19 09:58:14 0000 -------
Created an attachment (id=97433) [details]
commons-fileupload-1.1.1.ebuild

Has dependency to dev-java/portletapi-1.0 which is not in portage and might
even change name or version in future.

------- Comment #3 From Miroslav Šulc 2006-09-19 09:58:44 0000 -------
Created an attachment (id=97434) [details]
commons-fileupload-1.1.1-gentoo.patch

------- Comment #4 From Miroslav Šulc 2006-09-22 15:57:38 0000 -------
The most up-to-date ebuild can be found here:
http://overlays.gentoo.org/svn/proj/java/migrated-java-experimental-overlay/dev-java/commons-fileupload/

------- Comment #5 From Josh Nichols (RETIRED) 2006-10-08 19:29:45 0000 -------
Added to portage. Thanks for contributing!

One thing to watch out for: make sure you run 'repoman full' in the directory
of the ebuild. This does some things like checking the sanity in the depends.
In particular, your ebuild for portletapi didn't have keywords for ppc and
ppc64, but this ebuild did, and commons-io was missing ppc64 keywords.

------- Comment #6 From Marcus Klein 2006-10-10 02:55:23 0000 -------
Please don't blame me because I'm reopening this bug. But I don't want to open
an new bug for the following issue:

commons-fileupload-1.1.1 depends on commons-io-1.1 but you added the dependency

=dev-java/commons-io-1*

If you download the source tar.gz from commons-fileupload you can see this
dependency in the project.xml file for Maven line 165. So I think the
dependency should be something like:

>=dev-java/commons-io-1.1

------- Comment #7 From Miroslav Šulc 2006-10-10 11:45:23 0000 -------
Unfortunately portage doesn't support depending on a specific SLOT (used for
compatible versions of the package) so here I used the dependancy
=dev-java/commons-io-1* to make the package depend on version 1.x which is in
SLOT="1". This is safer than depending on >=dev-java/commons-io-1.1 which could
cause problems when commons-io-2.0 arises which may (or may not) have
incompatible API.

So, you are correct, but because portage doesn't support depending on a
specific SLOT, I used that dependancy.

------- Comment #8 From Miroslav Šulc 2006-10-10 12:06:30 0000 -------
Created an attachment (id=99316) [details]
patch for commons-fileupload-1.1.1 (in tree)

So you were right with the dependency and my solution doesn't work.
commons-fileupload-1.1.1 doesn't compile with commons-io-1.0. Here is a patch
that solves it. Thanks.

------- Comment #9 From Vlastimil Babka (Caster) 2006-10-10 13:45:22 0000 -------
Fixed, thanks. Hopefully we will have bug 150368 solved before some new slot of
commons-io hits the tree :) Thanks for giving me an example for that bug.