You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@brooklyn.apache.org by GitBox <gi...@apache.org> on 2020/11/20 15:53:12 UTC
[GitHub] [brooklyn-dist] duncangrant opened a new pull request #166: Fixes soap linkage error
duncangrant opened a new pull request #166:
URL: https://github.com/apache/brooklyn-dist/pull/166
When soap returns some errors Brooklyn doesn't manage to deserialise
them and instead shows a linkage error.
This is documented in this issue
here:https://github.com/cloudsoft/winrm4j/issues/129
----------------------------------------------------------------
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 go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [brooklyn-dist] ahgittin commented on pull request #166: Fixes soap linkage error
Posted by GitBox <gi...@apache.org>.
ahgittin commented on pull request #166:
URL: https://github.com/apache/brooklyn-dist/pull/166#issuecomment-733608611
LGTM - merging
----------------------------------------------------------------
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 go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [brooklyn-dist] asfgit merged pull request #166: Fixes soap linkage error
Posted by GitBox <gi...@apache.org>.
asfgit merged pull request #166:
URL: https://github.com/apache/brooklyn-dist/pull/166
----------------------------------------------------------------
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 go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [brooklyn-dist] asfgit closed pull request #166: Fixes soap linkage error
Posted by GitBox <gi...@apache.org>.
asfgit closed pull request #166:
URL: https://github.com/apache/brooklyn-dist/pull/166
----------------------------------------------------------------
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 go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [brooklyn-dist] ahgittin commented on pull request #166: Fixes soap linkage error
Posted by GitBox <gi...@apache.org>.
ahgittin commented on pull request #166:
URL: https://github.com/apache/brooklyn-dist/pull/166#issuecomment-731268111
could you add a comment saying what is removed, and/or do this as two commits one which copies the stock jre.properties value and another which removes the relevant ones?
also this will fix it only for jre 1.8. i wonder whether we'd be better off overwriting the original `jre.properties`, removing the packages we provide explicitly for all JREs. if not at least a comment to say this overwrites the value from `jre.properties`, and similar might be needed for other jre versions.
so glad it's fixed though!!!
----------------------------------------------------------------
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 go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [brooklyn-dist] duncangrant commented on pull request #166: Fixes soap linkage error
Posted by GitBox <gi...@apache.org>.
duncangrant commented on pull request #166:
URL: https://github.com/apache/brooklyn-dist/pull/166#issuecomment-731282855
@ahgittin I've split this into 2 commits to make it clear what has been removed.
Although this only fixes the issue for java 8, the equivalent packages are included in java 9 + so I don't think it should be an issue there.
----------------------------------------------------------------
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 go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org