rleigh-codelibre commented on pull request #2:
URL: https://github.com/apache/xerces-c/pull/2#issuecomment-647028400
If all the work is individual and there is no corporate ownership of any of
it, then if you both submit the needed ICLA that should be sufficient. Thanks.
rleigh-codelibre commented on a change in pull request #2:
URL: https://github.com/apache/xerces-c/pull/2#discussion_r443149185
##
File path: CMakeLists.txt
##
@@ -177,8 +177,14 @@ install(
# Process subdirectories
add_subdirectory(doc)
add_subdirectory(src)
-add_subdirector
viniul edited a comment on pull request #2:
URL: https://github.com/apache/xerces-c/pull/2#issuecomment-646989210
@rleigh-codelibre We have not yet submitted the Apache licensing agreement -
this is individual work by me and @bshastry, so I think if we both submit the
ICLA this should be s
viniul commented on pull request #2:
URL: https://github.com/apache/xerces-c/pull/2#issuecomment-646989210
@rleigh-codelibre We have not yet submitted the Apache licensing agreement -
this is individual work by me and @bshastry, so I think if we both submit the
ICLA this should be sufficie
viniul commented on a change in pull request #2:
URL: https://github.com/apache/xerces-c/pull/2#discussion_r443127661
##
File path: CMakeLists.txt
##
@@ -177,8 +177,14 @@ install(
# Process subdirectories
add_subdirectory(doc)
add_subdirectory(src)
-add_subdirectory(tests)
-
viniul commented on a change in pull request #2:
URL: https://github.com/apache/xerces-c/pull/2#discussion_r443127695
##
File path: CMakeLists.txt
##
@@ -177,8 +177,14 @@ install(
# Process subdirectories
add_subdirectory(doc)
add_subdirectory(src)
-add_subdirectory(tests)
-
viniul commented on a change in pull request #2:
URL: https://github.com/apache/xerces-c/pull/2#discussion_r443127487
##
File path: CMakeLists.txt
##
@@ -177,8 +177,14 @@ install(
# Process subdirectories
add_subdirectory(doc)
add_subdirectory(src)
-add_subdirectory(tests)
-
viniul commented on a change in pull request #2:
URL: https://github.com/apache/xerces-c/pull/2#discussion_r443127356
##
File path: build_fuzzer.sh
##
@@ -0,0 +1,8 @@
+#!/bin/bash
Review comment:
Not necessarily needed, but I thought it might be helpful if someone
does
rleigh-codelibre commented on pull request #2:
URL: https://github.com/apache/xerces-c/pull/2#issuecomment-646988406
@viniul I have done a preliminary review, please see the above comments.
Please could you also answer the question regarding the licensing. I see
Google copyrights on
viniul commented on pull request #2:
URL: https://github.com/apache/xerces-c/pull/2#issuecomment-646988283
@rleigh-codelibre Thank you very much for reviewing this PR. I will
implement the suggested changes and open a new PR based on the current master.
--
rleigh-codelibre commented on a change in pull request #2:
URL: https://github.com/apache/xerces-c/pull/2#discussion_r443126284
##
File path: .gitignore
##
@@ -105,3 +105,6 @@ tests/scripts/*.log
tests/scripts/*.trs
tests/test-suite.log
Testing/
+
+# Vincent
+build/
Review
viniul commented on pull request #2:
URL: https://github.com/apache/xerces-c/pull/2#issuecomment-646984925
@rleigh-codelibre I updated the PR to also includes the changes for CMake.
Is this sufficient to get the PR merge dfor now? I f so, let me create a new
clean PR based on the currnet
viniul edited a comment on pull request #2:
URL: https://github.com/apache/xerces-c/pull/2#issuecomment-646984925
@rleigh-codelibre I updated the PR to also includes the changes for CMake.
Is this sufficient to get the PR merge dfor now? I f so, let me create a new
clean PR based on the c
rleigh-codelibre commented on pull request #2:
URL: https://github.com/apache/xerces-c/pull/2#issuecomment-646975989
@viniul The addition of a test looks fairly low-risk in terms of wider
impact on the project.
Has the necessary Apache ICLA and/or company CLA been done?
On the
viniul commented on pull request #2:
URL: https://github.com/apache/xerces-c/pull/2#issuecomment-646973122
CC @rleigh-codelibre As described above, this PR adds some fuzzing harnesses
for xerces, which we (@bshastry and me) think would be nice to have in the
master branch, because it allow
I moved all the issues using version 3.3.0 to version 4.0.0.
I don't have Jira admin access, so I can't delete the 3.3.0 release
version, which is no longer in use.
Roger
[
https://issues.apache.org/jira/browse/XERCESC-2141?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Roger Leigh updated XERCESC-2141:
-
Fix Version/s: (was: 3.3.0)
4.0.0
> Enable C++11/14/17 with autoconf bui
[
https://issues.apache.org/jira/browse/XERCESC-2202?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Roger Leigh updated XERCESC-2202:
-
Fix Version/s: (was: 3.3.0)
4.0.0
> Update version of master branch to 3
[
https://issues.apache.org/jira/browse/XERCESC-2201?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Roger Leigh updated XERCESC-2201:
-
Fix Version/s: (was: 3.3.0)
4.0.0
> Update Travis-CI image in use
>
[
https://issues.apache.org/jira/browse/XERCESC-2200?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Roger Leigh updated XERCESC-2200:
-
Fix Version/s: (was: 3.3.0)
4.0.0
> Update AppVeyor image in use
> -
[
https://issues.apache.org/jira/browse/XERCESC-2203?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Roger Leigh updated XERCESC-2203:
-
Fix Version/s: (was: 3.3.0)
4.0.0
> MingGW time functions are broken
> -
[
https://issues.apache.org/jira/browse/XERCESC-2206?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Roger Leigh updated XERCESC-2206:
-
Fix Version/s: (was: 3.3.0)
4.0.0
> Use char16_t and unicode literals to
[
https://issues.apache.org/jira/browse/XERCESC-2204?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Roger Leigh updated XERCESC-2204:
-
Fix Version/s: (was: 3.3.0)
4.0.0
> Remove message loader
>
[
https://issues.apache.org/jira/browse/XERCESC-2209?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Roger Leigh updated XERCESC-2209:
-
Fix Version/s: (was: 3.3.0)
4.0.0
> Remove HAVE_LSTRING
> --
[
https://issues.apache.org/jira/browse/XERCESC-2210?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Roger Leigh updated XERCESC-2210:
-
Fix Version/s: (was: 3.3.0)
4.0.0
> Remove XERCES_NO_MATCHING_DELETE_OPE
[
https://issues.apache.org/jira/browse/XERCESC-2208?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Roger Leigh updated XERCESC-2208:
-
Fix Version/s: (was: 3.3.0)
4.0.0
> Rationalise XercesIntTypes
> ---
[
https://issues.apache.org/jira/browse/XERCESC-2205?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Roger Leigh updated XERCESC-2205:
-
Fix Version/s: (was: 3.3.0)
4.0.0
> Require std::mutex and std::thread
>
[
https://issues.apache.org/jira/browse/XERCESC-2207?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Roger Leigh updated XERCESC-2207:
-
Fix Version/s: (was: 3.3.0)
4.0.0
> Rationalise network accessors
>
[
https://issues.apache.org/jira/browse/XERCESC-2138?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Roger Leigh updated XERCESC-2138:
-
Fix Version/s: (was: 3.3.0)
4.0.0
> Xerces-C++ should use C++98 features
[
https://issues.apache.org/jira/browse/XERCESC-2176?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Roger Leigh updated XERCESC-2176:
-
Fix Version/s: (was: 3.3.0)
4.0.0
> Incorrect symbolic links created for
30 matches
Mail list logo
