[Bug 855331] Review Request: tesla-polyglot - Modules to enable Maven usage in others JVM languages

2014-10-21 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855331

Michael Simacek msima...@redhat.com changed:

   What|Removed |Added

 CC||msima...@redhat.com
   Assignee|m...@redhat.com |msima...@redhat.com
  Flags||fedora-review?



--- Comment #13 from Michael Simacek msima...@redhat.com ---
If you're still interested in packaging it, I'll take the review.

Some issues:
It seems that it doesn't build with current rawhide version of snakeyaml.
Change summary to: Modules to enable Maven usage in other JVM languages
(there was gramatical error)
Is the depmap still needed?

Looks good otherwise, but I didn't run fedora-review on it yet.

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855331] Review Request: tesla-polyglot - Modules to enable Maven usage in others JVM languages

2014-10-21 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855331



--- Comment #14 from Michael Simacek msima...@redhat.com ---
Created attachment 948965
  -- https://bugzilla.redhat.com/attachment.cgi?id=948965action=edit
build.log

build.log for the failing build

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855331] Review Request: tesla-polyglot - Modules to enable Maven usage in others JVM languages

2014-10-21 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855331



--- Comment #15 from gil cattaneo punto...@libero.it ---
(In reply to Michael Simacek from comment #13)
 If you're still interested in packaging it, I'll take the review.
 
 Some issues:
 It seems that it doesn't build with current rawhide version of snakeyaml.
Disabled, i package T-P for gradle, and need only common groovy modules.
 Change summary to: Modules to enable Maven usage in other JVM languages
 (there was gramatical error)
Fix
 Is the depmap still needed?
Yes, for this reason:
- Force clojure-compat usage. Require clojure 1.2.x
- clojure-maven-plugin don't work with clojure = 1.5.1
 
 Looks good otherwise, but I didn't run fedora-review on it yet.
Thanks!


Spec URL: http://gil.fedorapeople.org/tesla-polyglot.spec
SRPM URL: http://gil.fedorapeople.org/tesla-polyglot-0.1.0-1.fc19.src.rpm

Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=7925643

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855331] Review Request: tesla-polyglot - Modules to enable Maven usage in others JVM languages

2014-06-21 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855331

gil cattaneo punto...@libero.it changed:

   What|Removed |Added

Summary|Review Request: |Review Request:
   |polyglot-maven - Modules to |tesla-polyglot - Modules to
   |enable Maven usage in   |enable Maven usage in
   |others JVM languages|others JVM languages
  Alias|polyglot-maven  |



--- Comment #12 from gil cattaneo punto...@libero.it ---
Spec URL: http://gil.fedorapeople.org/tesla-polyglot.spec
SRPM URL: http://gil.fedorapeople.org/tesla-polyglot-0.1.0-1.fc19.src.rpm

- moved from
  https://github.com/tobrien/polyglot-maven
  to
  https://github.com/takari/maven-polyglot/

Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=7064170

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review