Github user asfgit commented on the issue:
https://github.com/apache/ant/pull/63
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Linux/75/
---
--
Github user asfgit commented on the issue:
https://github.com/apache/ant/pull/63
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Windows/81/
---
Github user asfgit commented on the issue:
https://github.com/apache/ant/pull/63
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Linux/74/
---
--
Github user asfgit commented on the issue:
https://github.com/apache/ant/pull/63
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Windows/80/
---
Github user asfgit commented on the issue:
https://github.com/apache/ant/pull/63
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Windows/77/
---
Github user twogee commented on the issue:
https://github.com/apache/ant/pull/63
Thanks for reviewing. I hope my last refactoring clarifies the point about
the string field in the Text class (or the string attribute in the text
element).
---
Github user asfgit commented on the issue:
https://github.com/apache/ant/pull/63
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Linux/71/
---
--
Github user bodewig commented on the issue:
https://github.com/apache/ant/pull/63
`addImageOperation` indeed looks strange (as does the manual when it talks
about `ImageOperation`, By virtue of `add(ImageOperation)` you can add any type
as nested element that implements `ImgeOperation
Github user asfgit commented on the issue:
https://github.com/apache/ant/pull/63
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Windows/75/
---
Github user asfgit commented on the issue:
https://github.com/apache/ant/pull/63
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Linux/69/
---
--
Github user asfgit commented on the issue:
https://github.com/apache/ant/pull/63
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Windows/74/
---
Github user asfgit commented on the issue:
https://github.com/apache/ant/pull/63
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Linux/68/
---
--
Github user twogee commented on the issue:
https://github.com/apache/ant/pull/63
I will do move `addRectangle` and `addText` to `Draw`, it makes no sense to
have them in an abstract class. The manual needs correction regarding
`ImageOperation`, both for image and imageio tasks: it sho
Github user bodewig commented on the issue:
https://github.com/apache/ant/pull/63
Looks good to me. I probably wouldn't have touched the image task's diagram
at all, but that's your choice.
No idea about `addRectangle`. If things work if you move the method over to
`Draw` I s
Github user twogee commented on the issue:
https://github.com/apache/ant/pull/63
Looks like disk is full in CI server...
---
-
To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org
For additional commands, e-mail: d
Github user asfgit commented on the issue:
https://github.com/apache/ant/pull/63
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Linux/67/
---
--
Github user asfgit commented on the issue:
https://github.com/apache/ant/pull/63
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Windows/73/
---
Github user twogee commented on the issue:
https://github.com/apache/ant/pull/63
Looking at the class diagram, I wonder why `addRectangle()` is in
TransformOperation rather than only in Draw?
---
-
To unsubscribe, e
Github user asfgit commented on the issue:
https://github.com/apache/ant/pull/63
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Linux/66/
---
--
Github user asfgit commented on the issue:
https://github.com/apache/ant/pull/63
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Windows/72/
---
Github user twogee commented on the issue:
https://github.com/apache/ant/pull/63
I decided to redraw the class diagram as GraphML (continuing Ivy tradition
of using yEd), because it's quite informative.
---
-
To uns
Github user bodewig commented on the issue:
https://github.com/apache/ant/pull/63
or live without any UML diagram :-) - no, I don't think we've got anything
but the image.
---
-
To unsubscribe, e-mail: dev-unsubscr.
Github user twogee commented on the issue:
https://github.com/apache/ant/pull/63
I could almost reuse the class diagram in the documentation if it wasn't
referring to PlanarImage... so I have to draw a new one unless the original UML
is available somewhere (a long shot, I know...)
-
Github user asfgit commented on the issue:
https://github.com/apache/ant/pull/63
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Linux/63/
---
--
Github user asfgit commented on the issue:
https://github.com/apache/ant/pull/63
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Windows/69/
---
Github user bodewig commented on the issue:
https://github.com/apache/ant/pull/63
Looks as if only the manual was missing. :-)
---
-
To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org
For additional commands, e-m
Github user asfgit commented on the issue:
https://github.com/apache/ant/pull/63
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Linux/62/
---
--
Github user asfgit commented on the issue:
https://github.com/apache/ant/pull/63
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Windows/68/
---
Github user twogee commented on the issue:
https://github.com/apache/ant/pull/63
"imageio" is fine by me, then we skip a potential debacle about naming of
java-jai.jar ð
Whenever JAI is laid to rest, "image" can be reclaimed as an alias.
FWIW Java 8 is an LTS release, but ho
Github user bodewig commented on the issue:
https://github.com/apache/ant/pull/63
My scenario is people using Java 8, where JAI still works - Java 8 is far
less dead than Java 9 in my view :-)
I'd prefer a new "imageio" (or whatever it is called) task that doesn't
suggest it
Github user twogee commented on the issue:
https://github.com/apache/ant/pull/63
JAI is dead. It depends on internal classes that are no longer there in
Java 9. I propose dropping JAI API support completely, just like apt or javah.
I don't like the like the idea of shipping dead code
Github user bodewig commented on the issue:
https://github.com/apache/ant/pull/63
I'm a bit torn as this PR breaks public APIs and may break build
environments where somebody has managed to install JAI. for example because
they've downloaded it long ago. Maybe it would be better to ma
Github user asfgit commented on the issue:
https://github.com/apache/ant/pull/63
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Linux/61/
---
--
Github user asfgit commented on the issue:
https://github.com/apache/ant/pull/63
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Windows/67/
---
34 matches
Mail list logo