After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 652666 - ant modtype broken
ant modtype broken
Status: RESOLVED FIXED
Product: jhbuild
Classification: Infrastructure
Component: general
unspecified
Other All
: Normal major
: ---
Assigned To: Jhbuild maintainers
Jhbuild QA
Depends on:
Blocks:
 
 
Reported: 2011-06-15 16:23 UTC by deignacio
Modified: 2011-07-10 03:43 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
pass the jhbuild config to modtypes.get_branch (1.14 KB, patch)
2011-06-15 16:28 UTC, deignacio
none Details | Review

Description deignacio 2011-06-15 16:23:02 UTC
the get_branch call does not get passed the config, thus resulting in a

TypeError: get_branch() takes exactly 4 arguments (3 given)

this prevents any ant modules from being built using jhbuild
Comment 1 deignacio 2011-06-15 16:28:58 UTC
Created attachment 189988 [details] [review]
pass the jhbuild config to modtypes.get_branch
Comment 2 Colin Walters 2011-06-15 18:13:06 UTC
You're actually using ant modules?  I am about to remove the modtype; see bug 647231.

The 'install' currently does nothing, so how can you be using it?
Comment 3 Sandy Armstrong 2011-06-17 21:08:08 UTC
We aren't yet using ant modules, we were thinking to integrate them into our jhbuild moduels when David ran across this bug.

If you're removing the modtype, that's fine, feel free to close this.  We have a wrapper script we have been using to call ant that works fine.
Comment 4 Colin Walters 2011-06-18 15:01:33 UTC
(In reply to comment #3)
> We aren't yet using ant modules, we were thinking to integrate them into our
> jhbuild moduels when David ran across this bug.
> 
> If you're removing the modtype, that's fine, feel free to close this.  We have
> a wrapper script we have been using to call ant that works fine.

Ok, what I suggest you do in situations like this (if you aren't already) is create a patch that adds ./configure and Makefile files that Do The Right Thing.

The ./configure accepts --prefix and writes it to a file Makefile.config, and the Makefile can just be:

include Makefile.config

all:
    ant

install:
   cp foo.jar $(DESTDIR)$(prefix)/share/java

See also http://people.gnome.org/~walters/docs/build-api.txt
Comment 5 Craig Keogh 2011-07-10 03:43:59 UTC
Thank you for the report & patch. Ant has now been removed from JHBuild. A wrapper script is recommended if you require Ant modules.

http://git.gnome.org/browse/jhbuild/commit/?id=8db8d874daa030a0ca6a550f94726f3f6cf96dc0
http://git.gnome.org/browse/jhbuild/commit/?id=aefcb2fb7d53cdd1f0dcc83d122d988afeb94c21