bodewig commented on issue #1: Fix issue ant-antlibs-common submodule not found
URL: https://github.com/apache/ant-antlibs-props/pull/1#issuecomment-447769681
should be fixed with df20430
Many thanks for reporting it.
bodewig commented on issue #1: Fix issue ant-antlibs-common submodule not found
URL: https://github.com/apache/ant-antlibs-props/pull/1#issuecomment-447768646
Sorry for that, we are in the process of moving our repositories internally
at Apache, expect some hiccups.
Rather than
vgoncharenko closed pull request #1: Fix issue ant-antlibs-common submodule not
found
URL: https://github.com/apache/ant-antlibs-props/pull/1
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As
vgoncharenko opened a new pull request #1: Fix issue ant-antlibs-common
submodule not found
URL: https://github.com/apache/ant-antlibs-props/pull/1
Hello, today during installation of this project I have faced an issue like
that:
+ git submodule init
Submodule 'common'
vgoncharenko commented on issue #1: Fix issue ant-antlibs-common submodule not
found
URL: https://github.com/apache/ant-antlibs-props/pull/1#issuecomment-447769149
ok, thanks for your fast reaction to my problem.
This is an
twogee commented on a change in pull request #89: Added version and mainClass
attributes to task.
URL: https://github.com/apache/ant/pull/89#discussion_r268422972
##
File path: manual/Tasks/jar.html
##
@@ -217,6 +217,23 @@ Parameters
include META-INF unless
twogee commented on a change in pull request #89: Added version and mainClass
attributes to task.
URL: https://github.com/apache/ant/pull/89#discussion_r268423028
##
File path: manual/Tasks/jar.html
##
@@ -385,6 +402,40 @@ service
a JAR file has more that one
jaikiran commented on issue #89: Added version and mainClass attributes to
task.
URL: https://github.com/apache/ant/pull/89#issuecomment-476030934
this is ok to test
This is an automated message from the Apache Git Service.
asfgit commented on issue #89: Added version and mainClass attributes to
task.
URL: https://github.com/apache/ant/pull/89#issuecomment-476032382
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Linux/114/
asfgit commented on issue #89: Added version and mainClass attributes to
task.
URL: https://github.com/apache/ant/pull/89#issuecomment-476032427
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Windows/120/
twogee commented on a change in pull request #89: Added version and mainClass
attributes to task.
URL: https://github.com/apache/ant/pull/89#discussion_r268423359
##
File path: src/main/org/apache/tools/ant/util/jarattr/ClassFormatException.java
##
@@ -0,0 +1,59 @@
+/*
twogee commented on a change in pull request #89: Added version and mainClass
attributes to task.
URL: https://github.com/apache/ant/pull/89#discussion_r268422972
##
File path: manual/Tasks/jar.html
##
@@ -217,6 +217,23 @@ Parameters
include META-INF unless
twogee commented on a change in pull request #89: Added version and mainClass
attributes to task.
URL: https://github.com/apache/ant/pull/89#discussion_r268423686
##
File path: src/main/org/apache/tools/ant/util/jarattr/Constant.java
##
@@ -0,0 +1,147 @@
+/*
Review
jaikiran commented on a change in pull request #89: Added version and mainClass
attributes to task.
URL: https://github.com/apache/ant/pull/89#discussion_r270277284
##
File path: manual/Tasks/jar.html
##
@@ -217,6 +217,23 @@ Parameters
include META-INF unless
jaikiran commented on issue #89: Added version and mainClass attributes to
task.
URL: https://github.com/apache/ant/pull/89#issuecomment-477864675
Thanks @craigpell, this is a very good enhancement and the code looks very
good too. I do have some suggestions/questions about some of the
asfgit commented on issue #89: Added version and mainClass attributes to
task.
URL: https://github.com/apache/ant/pull/89#issuecomment-477865927
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Windows/121/
asfgit commented on issue #89: Added version and mainClass attributes to
task.
URL: https://github.com/apache/ant/pull/89#issuecomment-477866471
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Linux/115/
jaikiran commented on issue #89: Added version and mainClass attributes to
task.
URL: https://github.com/apache/ant/pull/89#issuecomment-477867207
While you are here, a quick suggestion/question - given that these changes
are related to Java modules in general and there probably will be
asfgit commented on issue #90: solve the unconsistent comments
URL: https://github.com/apache/ant/pull/90#issuecomment-480727724
Can one of the admins verify this patch?
This is an automated message from the Apache Git
yuanjumao opened a new pull request #90: solve the unconsistent comments
URL: https://github.com/apache/ant/pull/90
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
asfgit commented on issue #90: solve the unconsistent comments
URL: https://github.com/apache/ant/pull/90#issuecomment-480727726
Can one of the admins verify this patch?
This is an automated message from the Apache Git
asfgit commented on issue #90: solve the unconsistent comments
URL: https://github.com/apache/ant/pull/90#issuecomment-480727727
Can one of the admins verify this patch?
This is an automated message from the Apache Git
asfgit commented on issue #90: solve the unconsistent comments
URL: https://github.com/apache/ant/pull/90#issuecomment-480727725
Can one of the admins verify this patch?
This is an automated message from the Apache Git
janmaterne commented on issue #45: Consider returning a zero length array
rather than null.
URL: https://github.com/apache/ant/pull/45#issuecomment-480534186
While i agree and prefer returning Optional or an empty List, in this
case this would break backwards compatiblity. And therefore
asfgit commented on issue #45: Consider returning a zero length array rather
than null.
URL: https://github.com/apache/ant/pull/45#issuecomment-480534190
Can one of the admins verify this patch?
This is an automated message
asfgit commented on issue #45: Consider returning a zero length array rather
than null.
URL: https://github.com/apache/ant/pull/45#issuecomment-480534188
Can one of the admins verify this patch?
This is an automated message
asfgit commented on issue #45: Consider returning a zero length array rather
than null.
URL: https://github.com/apache/ant/pull/45#issuecomment-480534191
Can one of the admins verify this patch?
This is an automated message
asfgit commented on issue #45: Consider returning a zero length array rather
than null.
URL: https://github.com/apache/ant/pull/45#issuecomment-480534189
Can one of the admins verify this patch?
This is an automated message
bodewig commented on a change in pull request #90: solve the unconsistent
comments
URL: https://github.com/apache/ant/pull/90#discussion_r273119651
##
File path: src/main/org/apache/tools/ant/ComponentHelper.java
##
@@ -452,12 +452,7 @@ public void checkTaskClass(final
bodewig commented on a change in pull request #90: solve the unconsistent
comments
URL: https://github.com/apache/ant/pull/90#discussion_r273119872
##
File path: src/main/org/apache/tools/ant/UnknownElement.java
##
@@ -151,11 +151,7 @@ public RuntimeConfigurable
bodewig commented on a change in pull request #90: solve the unconsistent
comments
URL: https://github.com/apache/ant/pull/90#discussion_r273120166
##
File path: src/main/org/apache/tools/ant/types/XMLCatalog.java
##
@@ -197,10 +197,7 @@ public Path createClasspath() {
bodewig commented on a change in pull request #90: solve the unconsistent
comments
URL: https://github.com/apache/ant/pull/90#discussion_r273121101
##
File path:
src/main/org/apache/tools/ant/types/selectors/modifiedselector/ModifiedSelector.java
##
@@ -686,9 +686,7 @@
asfgit commented on issue #89: Added version and mainClass attributes to
task.
URL: https://github.com/apache/ant/pull/89#issuecomment-475891806
Can one of the admins verify this patch?
This is an automated message from the
craigpell opened a new pull request #89: Added version and mainClass attributes
to task.
URL: https://github.com/apache/ant/pull/89
Added version and mainClass attributes to task, to set binary
attributes of a modular jar file. For Ant 1.10.x. Addresses these tickets:
asfgit commented on issue #89: Added version and mainClass attributes to
task.
URL: https://github.com/apache/ant/pull/89#issuecomment-475891807
Can one of the admins verify this patch?
This is an automated message from the
asfgit commented on issue #89: Added version and mainClass attributes to
task.
URL: https://github.com/apache/ant/pull/89#issuecomment-475891810
Can one of the admins verify this patch?
This is an automated message from the
asfgit commented on issue #89: Added version and mainClass attributes to
task.
URL: https://github.com/apache/ant/pull/89#issuecomment-475891809
Can one of the admins verify this patch?
This is an automated message from the
twogee opened a new pull request #85: Incorrect HTML
URL: https://github.com/apache/ant/pull/85
javadoc fails nightly builds...
This is an automated message from the Apache Git Service.
To respond to the message, please log
asfgit commented on issue #85: Incorrect HTML
URL: https://github.com/apache/ant/pull/85#issuecomment-468828666
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Linux/108/
asfgit commented on issue #85: Incorrect HTML
URL: https://github.com/apache/ant/pull/85#issuecomment-468822545
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Linux/107/
asfgit commented on issue #85: Incorrect HTML
URL: https://github.com/apache/ant/pull/85#issuecomment-468822780
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Windows/113/
asfgit commented on issue #85: Incorrect HTML
URL: https://github.com/apache/ant/pull/85#issuecomment-468829867
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Windows/114/
asfgit commented on issue #85: Incorrect HTML
URL: https://github.com/apache/ant/pull/85#issuecomment-468947269
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Linux/109/
asfgit commented on issue #85: Incorrect HTML
URL: https://github.com/apache/ant/pull/85#issuecomment-468947532
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Windows/115/
asfgit merged pull request #85: Incorrect HTML
URL: https://github.com/apache/ant/pull/85
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to
asfgit commented on issue #83: Add HTML validation check
URL: https://github.com/apache/ant/pull/83#issuecomment-470197718
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Linux/110/
asfgit commented on issue #83: Add HTML validation check
URL: https://github.com/apache/ant/pull/83#issuecomment-470197699
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Windows/116/
asfgit merged pull request #86: htmlcheck: correct tool name, consistent report
placement
URL: https://github.com/apache/ant/pull/86
This is an automated message from the Apache Git Service.
To respond to the message,
janmaterne commented on issue #83: Add HTML validation check
URL: https://github.com/apache/ant/pull/83#issuecomment-470423575
I like the idea of checking the html files.
But why are these changes required? Only because a tool sais that is not
enough for me. Do you have pointers to some
janmaterne commented on issue #83: Add HTML validation check
URL: https://github.com/apache/ant/pull/83#issuecomment-470422494
Summerize the changes:
- Add new tool "Nu HTML Validator" for checking manual and site
check.xml: new target 'vnu'
- add attribute: -->
-
twogee commented on issue #83: Add HTML validation check
URL: https://github.com/apache/ant/pull/83#issuecomment-470478712
The goal is to follow HTML 5 spec (except for index.html where we still have
frames).
What is missing is doctype and lang attribute instead of meta tags.
scope
janmaterne edited a comment on issue #83: Add HTML validation check
URL: https://github.com/apache/ant/pull/83#issuecomment-470422494
Summerize the changes:
- Add new tool "Nu HTML Validator" for checking manual and site
check.xml: new target 'vnu'
- add attribute: html> --
janmaterne commented on issue #83: Add HTML validation check
URL: https://github.com/apache/ant/pull/83#issuecomment-470939094
out|err-props could also work. Catch into prop, write prop to file and
print. Give it a try.
This
twogee commented on issue #83: Add HTML validation check
URL: https://github.com/apache/ant/pull/83#issuecomment-470576630
Recommendations re [doctype](https://www.w3.org/QA/Tips/Doctype), [language
declaration](https://www.w3.org/International/questions/qa-html-language-declarations),
twogee commented on issue #83: Add HTML validation check
URL: https://github.com/apache/ant/pull/83#issuecomment-470920527
Thanks for reviewing and excellent summaries.
These tests have failed multiple times previously; it looks like redirection
tests cannot connect to external
asfgit merged pull request #83: Add HTML validation check
URL: https://github.com/apache/ant/pull/83
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
janmaterne commented on issue #83: Add HTML validation check
URL: https://github.com/apache/ant/pull/83#issuecomment-471177878
Added the redirection to file and did a few renamings. Merged into master.
This is an automated
asfgit commented on issue #88: HTML5-compliant javadoc
URL: https://github.com/apache/ant/pull/88#issuecomment-474616741
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Windows/119/
asfgit commented on issue #88: HTML5-compliant javadoc
URL: https://github.com/apache/ant/pull/88#issuecomment-474616822
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Linux/113/
twogee closed pull request #87: CSS contains non-ASCII quotes
URL: https://github.com/apache/ant/pull/87
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
twogee opened a new pull request #88: HTML5-compliant javadoc
URL: https://github.com/apache/ant/pull/88
We should also convert XSLT to produce HTML5...
This is an automated message from the Apache Git Service.
To respond to
asfgit closed pull request #88: HTML5-compliant javadoc
URL: https://github.com/apache/ant/pull/88
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
jaikiran commented on issue #88: HTML5-compliant javadoc
URL: https://github.com/apache/ant/pull/88#issuecomment-474683056
Thank you, this is now merged.
This is an automated message from the Apache Git Service.
To respond to
twogee opened a new pull request #87: CSS contains non-ASCII quotes
URL: https://github.com/apache/ant/pull/87
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
asfgit commented on issue #87: CSS contains non-ASCII quotes
URL: https://github.com/apache/ant/pull/87#issuecomment-474221694
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Linux/112/
asfgit commented on issue #87: CSS contains non-ASCII quotes
URL: https://github.com/apache/ant/pull/87#issuecomment-474221620
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Windows/118/
janmaterne commented on issue #83: Add HTML validation check
URL: https://github.com/apache/ant/pull/83#issuecomment-470832663
Tool "Copyright (c) 2013-2018 Mozilla Foundation"
(https://github.com/validator/validator/blob/master/src/nu/validator/client/SimpleCommandLineValidator.java),
so
janmaterne commented on issue #83: Add HTML validation check
URL: https://github.com/apache/ant/pull/83#issuecomment-470836739
This PR can't be responsible for the failing tests as the sources or test
files are not touched.
Failing tests are:
asfgit commented on issue #86: htmlcheck: correct tool name, consistent report
placement
URL: https://github.com/apache/ant/pull/86#issuecomment-471250804
Refer to this link for build results (access rights to CI server needed):
twogee opened a new pull request #86: htmlcheck: correct tool name, consistent
report placement
URL: https://github.com/apache/ant/pull/86
Sorry about tardiness...
This is an automated message from the Apache Git Service.
To
asfgit commented on issue #86: htmlcheck: correct tool name, consistent report
placement
URL: https://github.com/apache/ant/pull/86#issuecomment-471250862
Refer to this link for build results (access rights to CI server needed):
janmaterne commented on issue #83: Add HTML validation check
URL: https://github.com/apache/ant/pull/83#issuecomment-470824474
Thanks for the links, sounds reasonable.
Two things I found:
1. maybe you should add a short paragraph in the description> section
like for the other tools
janmaterne commented on issue #83: Add HTML validation check
URL: https://github.com/apache/ant/pull/83#issuecomment-470825471
Just summarize/quote the link contents:
- **doctype**: that declaration was used in HTML5 to distinguish between a
standards-compliant parsing mode and a
asfgit commented on issue #91: Update ScpFromMessage.java
URL: https://github.com/apache/ant/pull/91#issuecomment-481542452
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Linux/116/
linux1fans commented on issue #91: Update ScpFromMessage.java
URL: https://github.com/apache/ant/pull/91#issuecomment-481622348
> This change looks fine to me. Is there a specific name you want us to use
to credit you for this change in our contributors list
asfgit commented on issue #91: Update ScpFromMessage.java
URL: https://github.com/apache/ant/pull/91#issuecomment-481578372
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Windows/122/
jaikiran commented on issue #91: Update ScpFromMessage.java
URL: https://github.com/apache/ant/pull/91#issuecomment-481125990
this is ok to test
This is an automated message from the Apache Git Service.
To respond to the
asfgit commented on issue #91: Update ScpFromMessage.java
URL: https://github.com/apache/ant/pull/91#issuecomment-481115349
Can one of the admins verify this patch?
This is an automated message from the Apache Git Service.
To
asfgit commented on issue #91: Update ScpFromMessage.java
URL: https://github.com/apache/ant/pull/91#issuecomment-481115346
Can one of the admins verify this patch?
This is an automated message from the Apache Git Service.
To
asfgit commented on issue #91: Update ScpFromMessage.java
URL: https://github.com/apache/ant/pull/91#issuecomment-481115348
Can one of the admins verify this patch?
This is an automated message from the Apache Git Service.
To
asfgit commented on issue #91: Update ScpFromMessage.java
URL: https://github.com/apache/ant/pull/91#issuecomment-481115345
Can one of the admins verify this patch?
This is an automated message from the Apache Git Service.
To
asfgit commented on issue #92: Wrap FileWriter with BufferedWriter for better
performance
URL: https://github.com/apache/ant/pull/92#issuecomment-482920253
Can one of the admins verify this patch?
This is an automated
asfgit commented on issue #92: Wrap FileWriter with BufferedWriter for better
performance
URL: https://github.com/apache/ant/pull/92#issuecomment-482920252
Can one of the admins verify this patch?
This is an automated
bd2019us opened a new pull request #92: Wrap FileWriter with BufferedWriter for
better performance
URL: https://github.com/apache/ant/pull/92
When the FileWriter.write() method is invoked in a loop, it has a bad impact
on the performance of program. Therefore, the BufferedWriter is
asfgit commented on issue #54: Let’s use Ivy (properly!) and drop Maven Ant
tasks + Commons OpenPGP
URL: https://github.com/apache/ant/pull/54#issuecomment-495981386
Refer to this link for build results (access rights to CI server needed):
asfgit commented on issue #54: Let’s use Ivy (properly!) and drop Maven Ant
tasks + Commons OpenPGP
URL: https://github.com/apache/ant/pull/54#issuecomment-495981411
Refer to this link for build results (access rights to CI server needed):
asfgit commented on issue #97: Retrofit Ant with CharSet
URL: https://github.com/apache/ant/pull/97#issuecomment-498067649
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Windows/132/
asfgit commented on issue #97: Retrofit Ant with CharSet
URL: https://github.com/apache/ant/pull/97#issuecomment-498067653
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Linux/126/
dimwael opened a new pull request #94: Run single test method
URL: https://github.com/apache/ant/pull/94
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
asfgit commented on issue #94: Run single test method
URL: https://github.com/apache/ant/pull/94#issuecomment-497888171
Can one of the admins verify this patch?
This is an automated message from the Apache Git Service.
To
asfgit commented on issue #94: Run single test method
URL: https://github.com/apache/ant/pull/94#issuecomment-497888172
Can one of the admins verify this patch?
This is an automated message from the Apache Git Service.
To
jaikiran commented on issue #94: Run single test method
URL: https://github.com/apache/ant/pull/94#issuecomment-497901176
I'm not sure what this PR is about. @dimwael Can you explain?
This is an automated message from the
jmkf commented on issue #94: Run single test method
URL: https://github.com/apache/ant/pull/94#issuecomment-497943700
This has been solved already since ant 1.8, seems someone found a stale
branch from 2010
This is an
twogee opened a new pull request #95: Refer to Java 8 documentation for
consistency
URL: https://github.com/apache/ant/pull/95
http to https changes in references also introduced Java 5 to 8 changes, so
let's be consistent
asfgit commented on issue #95: Refer to Java 8 documentation for consistency
URL: https://github.com/apache/ant/pull/95#issuecomment-497997900
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Linux/121/
asfgit commented on issue #95: Refer to Java 8 documentation for consistency
URL: https://github.com/apache/ant/pull/95#issuecomment-497997906
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Windows/127/
twogee opened a new pull request #97: Retrofit Ant with CharSet
URL: https://github.com/apache/ant/pull/97
CharSet is like a character encoding except
- it's never null
- it's always valid
- it understands the equivalence of aliases
asfgit commented on issue #97: Retrofit Ant with CharSet
URL: https://github.com/apache/ant/pull/97#issuecomment-498056873
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Windows/131/
asfgit commented on issue #97: Retrofit Ant with CharSet
URL: https://github.com/apache/ant/pull/97#issuecomment-498056745
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Linux/125/
asfgit commented on issue #96: Port runant.py to Python 3
URL: https://github.com/apache/ant/pull/96#issuecomment-498001078
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ant%20Github-PR-Windows/128/
1 - 100 of 706 matches
Mail list logo