[PATCH] bin/bitbake: Catch establish connection log messages

[PATCH] bin/bitbake: Catch establish connection log messages

[Richard Purdie]

If for example you try "bitbake -m" with an invalid BBSERVER, error
messages are not dsplayed. This change ensures logging is in place
to catch and display such errors.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
diff --git a/bitbake/bin/bitbake b/bitbake/bin/bitbake
index 222dabd..21a6c81 100755
--- a/bitbake/bin/bitbake
+++ b/bitbake/bin/bitbake
@@ -297,8 +297,6 @@ def main():
         server = servermodule.BitBakeXMLRPCClient(configParams.observe_only)
         server.saveConnectionDetails(configParams.remote_server)
 
-    logger.removeHandler(handler)
-
     if not configParams.server_only:
         # Collect the feature set for the UI
         featureset = getattr(ui_module, "featureSet", [])
@@ -310,6 +308,8 @@ def main():
         for k in cleanedvars:
             os.environ[k] = cleanedvars[k]
 
+        logger.removeHandler(handler)
+
         try:
             return ui_module.main(server_connection.connection, server_connection.events, configParams)
         finally:

Re: [PATCH] bin/bitbake: Catch establish connection log messages

[Martin Jansa]

[-- Attachment #1: Type: text/plain, Size: 1484 bytes --]

On Fri, Oct 04, 2013 at 02:14:52PM +0100, Richard Purdie wrote:
> If for example you try "bitbake -m" with an invalid BBSERVER, error
> messages are not dsplayed. This change ensures logging is in place

small typo         ^
                   displayed

> to catch and display such errors.
> 
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> ---
> diff --git a/bitbake/bin/bitbake b/bitbake/bin/bitbake
> index 222dabd..21a6c81 100755
> --- a/bitbake/bin/bitbake
> +++ b/bitbake/bin/bitbake
> @@ -297,8 +297,6 @@ def main():
>          server = servermodule.BitBakeXMLRPCClient(configParams.observe_only)
>          server.saveConnectionDetails(configParams.remote_server)
>  
> -    logger.removeHandler(handler)
> -
>      if not configParams.server_only:
>          # Collect the feature set for the UI
>          featureset = getattr(ui_module, "featureSet", [])
> @@ -310,6 +308,8 @@ def main():
>          for k in cleanedvars:
>              os.environ[k] = cleanedvars[k]
>  
> +        logger.removeHandler(handler)
> +
>          try:
>              return ui_module.main(server_connection.connection, server_connection.events, configParams)
>          finally:
> 
> 
> _______________________________________________
> bitbake-devel mailing list
> bitbake-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/bitbake-devel

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]