You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@qpid.apache.org by "Ted Ross (Created) (JIRA)" <ji...@apache.org> on 2011/09/27 21:29:45 UTC

[jira] [Created] (QPID-3510) Holes in C++ client configuration

Holes in C++ client configuration
---------------------------------

                 Key: QPID-3510
                 URL: https://issues.apache.org/jira/browse/QPID-3510
             Project: Qpid
          Issue Type: Bug
          Components: C++ Client
    Affects Versions: 0.12
            Reporter: Ted Ross


Configuration of the C++ (and wrapped) clients suffers from the following limitations:

1) Logging cannot be configured from the client configuration file.  The log module is only configurable via environment variables.

2) Client configuration cannot be loaded from an alternative configuration file.  The configuration file defined in the build is the only file that can be loaded for any application using the C++ client libraries.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:dev-subscribe@qpid.apache.org


[jira] [Commented] (QPID-3510) Holes in C++ client configuration

Posted by "jiraposter@reviews.apache.org (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/QPID-3510?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13257482#comment-13257482 ] 

jiraposter@reviews.apache.org commented on QPID-3510:
-----------------------------------------------------


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4802/
-----------------------------------------------------------

Review request for qpid, Alan Conway and Gordon Sim.


Summary
-------

This patch allows client applications to use configuration files other than the installed default.  It also allows logging options to be supplied via the configuration file, not just environment variables.


This addresses bug QPID-3510.
    https://issues.apache.org/jira/browse/QPID-3510


Diffs
-----

  trunk/qpid/cpp/include/qpid/sys/Env.h PRE-CREATION 
  trunk/qpid/cpp/src/CMakeLists.txt 1327943 
  trunk/qpid/cpp/src/Makefile.am 1327943 
  trunk/qpid/cpp/src/qpid/client/ConnectionImpl.cpp 1327943 
  trunk/qpid/cpp/src/qpid/client/LoadPlugins.cpp 1327943 
  trunk/qpid/cpp/src/qpid/client/SslConnector.cpp 1327943 
  trunk/qpid/cpp/src/qpid/sys/posix/Env.cpp PRE-CREATION 
  trunk/qpid/cpp/src/qpid/sys/windows/Env.cpp PRE-CREATION 

Diff: https://reviews.apache.org/r/4802/diff


Testing
-------


Thanks,

Ted


                
> Holes in C++ client configuration
> ---------------------------------
>
>                 Key: QPID-3510
>                 URL: https://issues.apache.org/jira/browse/QPID-3510
>             Project: Qpid
>          Issue Type: Bug
>          Components: C++ Client
>    Affects Versions: 0.12
>            Reporter: Ted Ross
>            Assignee: Ted Ross
>
> Configuration of the C++ (and wrapped) clients suffers from the following limitations:
> 1) Logging cannot be configured from the client configuration file.  The log module is only configurable via environment variables.
> 2) Client configuration cannot be loaded from an alternative configuration file.  The configuration file defined in the build is the only file that can be loaded for any application using the C++ client libraries.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org


[jira] [Assigned] (QPID-3510) Holes in C++ client configuration

Posted by "Ted Ross (Assigned) (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/QPID-3510?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ted Ross reassigned QPID-3510:
------------------------------

    Assignee: Ted Ross
    
> Holes in C++ client configuration
> ---------------------------------
>
>                 Key: QPID-3510
>                 URL: https://issues.apache.org/jira/browse/QPID-3510
>             Project: Qpid
>          Issue Type: Bug
>          Components: C++ Client
>    Affects Versions: 0.12
>            Reporter: Ted Ross
>            Assignee: Ted Ross
>
> Configuration of the C++ (and wrapped) clients suffers from the following limitations:
> 1) Logging cannot be configured from the client configuration file.  The log module is only configurable via environment variables.
> 2) Client configuration cannot be loaded from an alternative configuration file.  The configuration file defined in the build is the only file that can be loaded for any application using the C++ client libraries.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org


[jira] [Commented] (QPID-3510) Holes in C++ client configuration

Posted by "jiraposter@reviews.apache.org (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/QPID-3510?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13257502#comment-13257502 ] 

jiraposter@reviews.apache.org commented on QPID-3510:
-----------------------------------------------------



bq.  On 2012-04-19 14:03:15, Alan Conway wrote:
bq.  > Just need to make sure it doesn't break windows.

Windows is currently stubbed so it should behave as it did before with regard to env variables.  I'll test it prior to commit.


- Ted


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4802/#review7028
-----------------------------------------------------------


On 2012-04-19 13:56:16, Ted Ross wrote:
bq.  
bq.  -----------------------------------------------------------
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/4802/
bq.  -----------------------------------------------------------
bq.  
bq.  (Updated 2012-04-19 13:56:16)
bq.  
bq.  
bq.  Review request for qpid, Alan Conway and Gordon Sim.
bq.  
bq.  
bq.  Summary
bq.  -------
bq.  
bq.  This patch allows client applications to use configuration files other than the installed default.  It also allows logging options to be supplied via the configuration file, not just environment variables.
bq.  
bq.  
bq.  This addresses bug QPID-3510.
bq.      https://issues.apache.org/jira/browse/QPID-3510
bq.  
bq.  
bq.  Diffs
bq.  -----
bq.  
bq.    trunk/qpid/cpp/include/qpid/sys/Env.h PRE-CREATION 
bq.    trunk/qpid/cpp/src/CMakeLists.txt 1327943 
bq.    trunk/qpid/cpp/src/Makefile.am 1327943 
bq.    trunk/qpid/cpp/src/qpid/client/ConnectionImpl.cpp 1327943 
bq.    trunk/qpid/cpp/src/qpid/client/LoadPlugins.cpp 1327943 
bq.    trunk/qpid/cpp/src/qpid/client/SslConnector.cpp 1327943 
bq.    trunk/qpid/cpp/src/qpid/sys/posix/Env.cpp PRE-CREATION 
bq.    trunk/qpid/cpp/src/qpid/sys/windows/Env.cpp PRE-CREATION 
bq.  
bq.  Diff: https://reviews.apache.org/r/4802/diff
bq.  
bq.  
bq.  Testing
bq.  -------
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Ted
bq.  
bq.


                
> Holes in C++ client configuration
> ---------------------------------
>
>                 Key: QPID-3510
>                 URL: https://issues.apache.org/jira/browse/QPID-3510
>             Project: Qpid
>          Issue Type: Bug
>          Components: C++ Client
>    Affects Versions: 0.12
>            Reporter: Ted Ross
>            Assignee: Ted Ross
>
> Configuration of the C++ (and wrapped) clients suffers from the following limitations:
> 1) Logging cannot be configured from the client configuration file.  The log module is only configurable via environment variables.
> 2) Client configuration cannot be loaded from an alternative configuration file.  The configuration file defined in the build is the only file that can be loaded for any application using the C++ client libraries.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org


[jira] [Commented] (QPID-3510) Holes in C++ client configuration

Posted by "jiraposter@reviews.apache.org (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/QPID-3510?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13257503#comment-13257503 ] 

jiraposter@reviews.apache.org commented on QPID-3510:
-----------------------------------------------------



bq.  On 2012-04-19 14:05:36, Gordon Sim wrote:
bq.  > trunk/qpid/cpp/include/qpid/sys/Env.h, line 34
bq.  > <https://reviews.apache.org/r/4802/diff/1/?file=102984#file102984line34>
bq.  >
bq.  >     Why is this in the public API?

Good point, I'll move that to src.


- Ted


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4802/#review7029
-----------------------------------------------------------


On 2012-04-19 13:56:16, Ted Ross wrote:
bq.  
bq.  -----------------------------------------------------------
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/4802/
bq.  -----------------------------------------------------------
bq.  
bq.  (Updated 2012-04-19 13:56:16)
bq.  
bq.  
bq.  Review request for qpid, Alan Conway and Gordon Sim.
bq.  
bq.  
bq.  Summary
bq.  -------
bq.  
bq.  This patch allows client applications to use configuration files other than the installed default.  It also allows logging options to be supplied via the configuration file, not just environment variables.
bq.  
bq.  
bq.  This addresses bug QPID-3510.
bq.      https://issues.apache.org/jira/browse/QPID-3510
bq.  
bq.  
bq.  Diffs
bq.  -----
bq.  
bq.    trunk/qpid/cpp/include/qpid/sys/Env.h PRE-CREATION 
bq.    trunk/qpid/cpp/src/CMakeLists.txt 1327943 
bq.    trunk/qpid/cpp/src/Makefile.am 1327943 
bq.    trunk/qpid/cpp/src/qpid/client/ConnectionImpl.cpp 1327943 
bq.    trunk/qpid/cpp/src/qpid/client/LoadPlugins.cpp 1327943 
bq.    trunk/qpid/cpp/src/qpid/client/SslConnector.cpp 1327943 
bq.    trunk/qpid/cpp/src/qpid/sys/posix/Env.cpp PRE-CREATION 
bq.    trunk/qpid/cpp/src/qpid/sys/windows/Env.cpp PRE-CREATION 
bq.  
bq.  Diff: https://reviews.apache.org/r/4802/diff
bq.  
bq.  
bq.  Testing
bq.  -------
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Ted
bq.  
bq.


                
> Holes in C++ client configuration
> ---------------------------------
>
>                 Key: QPID-3510
>                 URL: https://issues.apache.org/jira/browse/QPID-3510
>             Project: Qpid
>          Issue Type: Bug
>          Components: C++ Client
>    Affects Versions: 0.12
>            Reporter: Ted Ross
>            Assignee: Ted Ross
>
> Configuration of the C++ (and wrapped) clients suffers from the following limitations:
> 1) Logging cannot be configured from the client configuration file.  The log module is only configurable via environment variables.
> 2) Client configuration cannot be loaded from an alternative configuration file.  The configuration file defined in the build is the only file that can be loaded for any application using the C++ client libraries.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org


[jira] [Commented] (QPID-3510) Holes in C++ client configuration

Posted by "jiraposter@reviews.apache.org (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/QPID-3510?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13257510#comment-13257510 ] 

jiraposter@reviews.apache.org commented on QPID-3510:
-----------------------------------------------------



bq.  On 2012-04-19 14:05:36, Gordon Sim wrote:
bq.  > trunk/qpid/cpp/include/qpid/sys/Env.h, line 34
bq.  > <https://reviews.apache.org/r/4802/diff/1/?file=102984#file102984line34>
bq.  >
bq.  >     Why is this in the public API?
bq.  
bq.  Ted Ross wrote:
bq.      Good point, I'll move that to src.

On that basis, ship it!


- Gordon


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4802/#review7029
-----------------------------------------------------------


On 2012-04-19 13:56:16, Ted Ross wrote:
bq.  
bq.  -----------------------------------------------------------
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/4802/
bq.  -----------------------------------------------------------
bq.  
bq.  (Updated 2012-04-19 13:56:16)
bq.  
bq.  
bq.  Review request for qpid, Alan Conway and Gordon Sim.
bq.  
bq.  
bq.  Summary
bq.  -------
bq.  
bq.  This patch allows client applications to use configuration files other than the installed default.  It also allows logging options to be supplied via the configuration file, not just environment variables.
bq.  
bq.  
bq.  This addresses bug QPID-3510.
bq.      https://issues.apache.org/jira/browse/QPID-3510
bq.  
bq.  
bq.  Diffs
bq.  -----
bq.  
bq.    trunk/qpid/cpp/include/qpid/sys/Env.h PRE-CREATION 
bq.    trunk/qpid/cpp/src/CMakeLists.txt 1327943 
bq.    trunk/qpid/cpp/src/Makefile.am 1327943 
bq.    trunk/qpid/cpp/src/qpid/client/ConnectionImpl.cpp 1327943 
bq.    trunk/qpid/cpp/src/qpid/client/LoadPlugins.cpp 1327943 
bq.    trunk/qpid/cpp/src/qpid/client/SslConnector.cpp 1327943 
bq.    trunk/qpid/cpp/src/qpid/sys/posix/Env.cpp PRE-CREATION 
bq.    trunk/qpid/cpp/src/qpid/sys/windows/Env.cpp PRE-CREATION 
bq.  
bq.  Diff: https://reviews.apache.org/r/4802/diff
bq.  
bq.  
bq.  Testing
bq.  -------
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Ted
bq.  
bq.


                
> Holes in C++ client configuration
> ---------------------------------
>
>                 Key: QPID-3510
>                 URL: https://issues.apache.org/jira/browse/QPID-3510
>             Project: Qpid
>          Issue Type: Bug
>          Components: C++ Client
>    Affects Versions: 0.12
>            Reporter: Ted Ross
>            Assignee: Ted Ross
>
> Configuration of the C++ (and wrapped) clients suffers from the following limitations:
> 1) Logging cannot be configured from the client configuration file.  The log module is only configurable via environment variables.
> 2) Client configuration cannot be loaded from an alternative configuration file.  The configuration file defined in the build is the only file that can be loaded for any application using the C++ client libraries.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org


[jira] [Commented] (QPID-3510) Holes in C++ client configuration

Posted by "Ted Ross (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/QPID-3510?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13259588#comment-13259588 ] 

Ted Ross commented on QPID-3510:
--------------------------------

Some notes on this issue:

The issue of not being able to configure logging from the client configuration file remains a problem for which there is no easy solution.  The reason for this is that the cluster module dynamically loads the client library at run time in order to set up an update-peer relationship between cluster nodes.  When this occurs, the client and broker log configurations can conflict with each other.

Since the client can be loaded by the broker, it is necessary to have separate configuration file options for broker and client configurations.  Using the same option "config", leads to conflicts and ambiguities in configuration loading.

The solution is to leave "config" (QPID_CONFIG) as the broker configuration file option and to add "client-config" (QPID_CLIENT_CONFIG) as the client configuration file option (for both pure-client operation and client-in-broker operation).

Allowing log configuration to be configured for pure client operation in the client configuration file will remain un-solved for now.

                
> Holes in C++ client configuration
> ---------------------------------
>
>                 Key: QPID-3510
>                 URL: https://issues.apache.org/jira/browse/QPID-3510
>             Project: Qpid
>          Issue Type: Bug
>          Components: C++ Client
>    Affects Versions: 0.12
>            Reporter: Ted Ross
>            Assignee: Ted Ross
>
> Configuration of the C++ (and wrapped) clients suffers from the following limitations:
> 1) Logging cannot be configured from the client configuration file.  The log module is only configurable via environment variables.
> 2) Client configuration cannot be loaded from an alternative configuration file.  The configuration file defined in the build is the only file that can be loaded for any application using the C++ client libraries.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org


[jira] [Resolved] (QPID-3510) Holes in C++ client configuration

Posted by "Ted Ross (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/QPID-3510?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ted Ross resolved QPID-3510.
----------------------------

       Resolution: Fixed
    Fix Version/s: 0.17
    
> Holes in C++ client configuration
> ---------------------------------
>
>                 Key: QPID-3510
>                 URL: https://issues.apache.org/jira/browse/QPID-3510
>             Project: Qpid
>          Issue Type: Bug
>          Components: C++ Client
>    Affects Versions: 0.12
>            Reporter: Ted Ross
>            Assignee: Ted Ross
>             Fix For: 0.17
>
>
> Configuration of the C++ (and wrapped) clients suffers from the following limitations:
> 1) Logging cannot be configured from the client configuration file.  The log module is only configurable via environment variables.
> 2) Client configuration cannot be loaded from an alternative configuration file.  The configuration file defined in the build is the only file that can be loaded for any application using the C++ client libraries.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org


[jira] [Commented] (QPID-3510) Holes in C++ client configuration

Posted by "jiraposter@reviews.apache.org (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/QPID-3510?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13257497#comment-13257497 ] 

jiraposter@reviews.apache.org commented on QPID-3510:
-----------------------------------------------------


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4802/#review7028
-----------------------------------------------------------

Ship it!


Just need to make sure it doesn't break windows.

- Alan


On 2012-04-19 13:56:16, Ted Ross wrote:
bq.  
bq.  -----------------------------------------------------------
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/4802/
bq.  -----------------------------------------------------------
bq.  
bq.  (Updated 2012-04-19 13:56:16)
bq.  
bq.  
bq.  Review request for qpid, Alan Conway and Gordon Sim.
bq.  
bq.  
bq.  Summary
bq.  -------
bq.  
bq.  This patch allows client applications to use configuration files other than the installed default.  It also allows logging options to be supplied via the configuration file, not just environment variables.
bq.  
bq.  
bq.  This addresses bug QPID-3510.
bq.      https://issues.apache.org/jira/browse/QPID-3510
bq.  
bq.  
bq.  Diffs
bq.  -----
bq.  
bq.    trunk/qpid/cpp/include/qpid/sys/Env.h PRE-CREATION 
bq.    trunk/qpid/cpp/src/CMakeLists.txt 1327943 
bq.    trunk/qpid/cpp/src/Makefile.am 1327943 
bq.    trunk/qpid/cpp/src/qpid/client/ConnectionImpl.cpp 1327943 
bq.    trunk/qpid/cpp/src/qpid/client/LoadPlugins.cpp 1327943 
bq.    trunk/qpid/cpp/src/qpid/client/SslConnector.cpp 1327943 
bq.    trunk/qpid/cpp/src/qpid/sys/posix/Env.cpp PRE-CREATION 
bq.    trunk/qpid/cpp/src/qpid/sys/windows/Env.cpp PRE-CREATION 
bq.  
bq.  Diff: https://reviews.apache.org/r/4802/diff
bq.  
bq.  
bq.  Testing
bq.  -------
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Ted
bq.  
bq.


                
> Holes in C++ client configuration
> ---------------------------------
>
>                 Key: QPID-3510
>                 URL: https://issues.apache.org/jira/browse/QPID-3510
>             Project: Qpid
>          Issue Type: Bug
>          Components: C++ Client
>    Affects Versions: 0.12
>            Reporter: Ted Ross
>            Assignee: Ted Ross
>
> Configuration of the C++ (and wrapped) clients suffers from the following limitations:
> 1) Logging cannot be configured from the client configuration file.  The log module is only configurable via environment variables.
> 2) Client configuration cannot be loaded from an alternative configuration file.  The configuration file defined in the build is the only file that can be loaded for any application using the C++ client libraries.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org


[jira] [Commented] (QPID-3510) Holes in C++ client configuration

Posted by "jiraposter@reviews.apache.org (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/QPID-3510?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13257499#comment-13257499 ] 

jiraposter@reviews.apache.org commented on QPID-3510:
-----------------------------------------------------


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4802/#review7029
-----------------------------------------------------------



trunk/qpid/cpp/include/qpid/sys/Env.h
<https://reviews.apache.org/r/4802/#comment15618>

    Why is this in the public API?


- Gordon


On 2012-04-19 13:56:16, Ted Ross wrote:
bq.  
bq.  -----------------------------------------------------------
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/4802/
bq.  -----------------------------------------------------------
bq.  
bq.  (Updated 2012-04-19 13:56:16)
bq.  
bq.  
bq.  Review request for qpid, Alan Conway and Gordon Sim.
bq.  
bq.  
bq.  Summary
bq.  -------
bq.  
bq.  This patch allows client applications to use configuration files other than the installed default.  It also allows logging options to be supplied via the configuration file, not just environment variables.
bq.  
bq.  
bq.  This addresses bug QPID-3510.
bq.      https://issues.apache.org/jira/browse/QPID-3510
bq.  
bq.  
bq.  Diffs
bq.  -----
bq.  
bq.    trunk/qpid/cpp/include/qpid/sys/Env.h PRE-CREATION 
bq.    trunk/qpid/cpp/src/CMakeLists.txt 1327943 
bq.    trunk/qpid/cpp/src/Makefile.am 1327943 
bq.    trunk/qpid/cpp/src/qpid/client/ConnectionImpl.cpp 1327943 
bq.    trunk/qpid/cpp/src/qpid/client/LoadPlugins.cpp 1327943 
bq.    trunk/qpid/cpp/src/qpid/client/SslConnector.cpp 1327943 
bq.    trunk/qpid/cpp/src/qpid/sys/posix/Env.cpp PRE-CREATION 
bq.    trunk/qpid/cpp/src/qpid/sys/windows/Env.cpp PRE-CREATION 
bq.  
bq.  Diff: https://reviews.apache.org/r/4802/diff
bq.  
bq.  
bq.  Testing
bq.  -------
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Ted
bq.  
bq.


                
> Holes in C++ client configuration
> ---------------------------------
>
>                 Key: QPID-3510
>                 URL: https://issues.apache.org/jira/browse/QPID-3510
>             Project: Qpid
>          Issue Type: Bug
>          Components: C++ Client
>    Affects Versions: 0.12
>            Reporter: Ted Ross
>            Assignee: Ted Ross
>
> Configuration of the C++ (and wrapped) clients suffers from the following limitations:
> 1) Logging cannot be configured from the client configuration file.  The log module is only configurable via environment variables.
> 2) Client configuration cannot be loaded from an alternative configuration file.  The configuration file defined in the build is the only file that can be loaded for any application using the C++ client libraries.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org