Updated: [rhinstaller/anaconda/pulls/214 master] Added package logging to anaconda.log.

[dashea]

Minor update, package operations are now logged in main anaconda.log
file.

Signed-off-by: Karel Valek

[rhinstaller/anaconda/pulls/214 master] Added package logging to anaconda.log.

[dashea]

Looks good to me now, thanks!

[rhinstaller/anaconda/pulls/214 master] Added package logging to anaconda.log.

[dashea]

Added label: ACK.

[rhinstaller/anaconda/pulls/214 master] Added package logging to anaconda.log.

[dashea]

> @@ -49,8 +50,11 @@ def progress_report(message):
>      yield
>      progress_step(message)
>
> -def progress_message(message):
> progressQ.send_message(message)
>
> +def progress_message(message, xlate=True):

Check the use of this in pyanaconda/install.py (I'm not totally sure but I think xlate=False is needed for that call) and in pyanaconda/ui/lib/entropy.py where it already passes in a translated message.

There are also a number of calls to progress_report passing in translated strings so maybe progress_report should also set xlate=False.

[rhinstaller/anaconda/pulls/214 master] Added package logging to anaconda.log.

[dashea]

> @@ -49,8 +50,11 @@ def progress_report(message):
>      yield
>      progress_step(message)
>
> -def progress_message(message):
> progressQ.send_message(message)
>
> +def progress_message(message, xlate=True):

I'd rather see all the places to be update to pass an untranslated string because we really want the untranslated string to be logged.

[rhinstaller/anaconda/pulls/214 master] Added package logging to anaconda.log.

[dashea]

Removed label: ACK.

[rhinstaller/anaconda/pulls/214 master] Added package logging to anaconda.log.

[dashea]

Closed.

[rhinstaller/anaconda/pulls/214 master] Added package logging to anaconda.log.

[dashea]

Superseded by the PR #269.