You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by GitBox <gi...@apache.org> on 2020/09/15 15:15:15 UTC

[GitHub] [camel-quarkus] ppalaga opened a new issue #1796: Find a proper place for registering SAXMessages native resource bundle

ppalaga opened a new issue #1796:
URL: https://github.com/apache/camel-quarkus/issues/1796


   In https://github.com/apache/camel-quarkus/pull/1793/files#diff-a2ef5a4c98491936f1d708b2ecd89661R83 we register `com.sun.org.apache.xerces.internal.impl.msg.SAXMessages` in our FOP extension. It does not feel like the right place. I guess it should go to our or Quarkus JAXP extension.


----------------------------------------------------------------
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] [camel-quarkus] ppalaga closed issue #1796: Find a proper place for registering SAXMessages native resource bundle

Posted by GitBox <gi...@apache.org>.
ppalaga closed issue #1796:
URL: https://github.com/apache/camel-quarkus/issues/1796


   


----------------------------------------------------------------
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] [camel-quarkus] JiriOndrusek commented on issue #1796: Find a proper place for registering SAXMessages native resource bundle

Posted by GitBox <gi...@apache.org>.
JiriOndrusek commented on issue #1796:
URL: https://github.com/apache/camel-quarkus/issues/1796#issuecomment-693201222


   Please, assign to me.


----------------------------------------------------------------
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] [camel-quarkus] JiriOndrusek commented on issue #1796: Find a proper place for registering SAXMessages native resource bundle

Posted by GitBox <gi...@apache.org>.
JiriOndrusek commented on issue #1796:
URL: https://github.com/apache/camel-quarkus/issues/1796#issuecomment-693240330


   As a part of Fop extension, there was registered bundle `com.sun.org.apache.xerces.internal.impl.msg.SAXMessages`. Bundle is not necessary for proper functionality but it helps in case there is an exception. 
   Example: Real problem is missing reflection registration for some classes, but developer will see only that bundle can not be loaded.
   I'd like to add this bundle into camel-quarkus jaxp component, because it can help in future.
   
   > @BuildStep
       NativeImageResourceBundleBuildItem initBundles() {
           return new NativeImageResourceBundleBuildItem(
                   "com.sun.org.apache.xerces.internal.impl.msg.SAXMessages");
       }


----------------------------------------------------------------
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] [camel-quarkus] ppalaga commented on issue #1796: Find a proper place for registering SAXMessages native resource bundle

Posted by GitBox <gi...@apache.org>.
ppalaga commented on issue #1796:
URL: https://github.com/apache/camel-quarkus/issues/1796#issuecomment-693288785


   See also https://github.com/quarkusio/quarkus/issues/12129


----------------------------------------------------------------
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] [camel-quarkus] JiriOndrusek edited a comment on issue #1796: Find a proper place for registering SAXMessages native resource bundle

Posted by GitBox <gi...@apache.org>.
JiriOndrusek edited a comment on issue #1796:
URL: https://github.com/apache/camel-quarkus/issues/1796#issuecomment-693240330


   As a part of Fop extension, there was registered bundle `com.sun.org.apache.xerces.internal.impl.msg.SAXMessages`. Bundle is not necessary for proper functionality but it helps in case there is an exception. 
   Example: Real problem is missing reflection registration for some classes, but developer will see also an error that bundle can not be loaded, which could lead to confusion
   I'd like to add this bundle into camel-quarkus jaxp component, because it can help in future.
   
   > @BuildStep
       NativeImageResourceBundleBuildItem initBundles() {
           return new NativeImageResourceBundleBuildItem(
                   "com.sun.org.apache.xerces.internal.impl.msg.SAXMessages");
       }


----------------------------------------------------------------
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] [camel-quarkus] ppalaga commented on issue #1796: Find a proper place for registering SAXMessages native resource bundle

Posted by GitBox <gi...@apache.org>.
ppalaga commented on issue #1796:
URL: https://github.com/apache/camel-quarkus/issues/1796#issuecomment-694675467


   Solved via https://github.com/quarkusio/quarkus/pull/12144 , esp. https://github.com/quarkusio/quarkus/pull/12144/files#diff-d7d3c24be2a44a5a8077b76e7b3290c6R36 
   I have filed https://github.com/apache/camel-quarkus/issues/1806 for the stuff we should do after Quarkus 1.9


----------------------------------------------------------------
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