[Fusioninventory-commit] [fusinv/fusioninventory-agent] 65474f: import Win32::Job based runCommand()

[Gonéri Le Bouder]

  Branch: refs/heads/2.2.x+unicode-fixes
  Home:   https://github.com/fusinv/fusioninventory-agent
  Commit: 65474fc5d5383202a1b0d17e4aceabb552902196
      https://github.com/fusinv/fusioninventory-agent/commit/65474fc5d5383202a1b0d17e4aceabb552902196
  Author: Gonéri Le Bouder <goneri at rulezlan.org>
  Date:   2012-05-10 (Thu, 10 May 2012)

  Changed paths:
    M lib/FusionInventory/Agent/Tools/Win32.pm
    A t/tools/win32.t

  Log Message:
  -----------
  import Win32::Job based runCommand()


  Commit: a6269c878ddb1c0784f1fb779ae6caa33c73f0e1
      https://github.com/fusinv/fusioninventory-agent/commit/a6269c878ddb1c0784f1fb779ae6caa33c73f0e1
  Author: Gonéri Le Bouder <goneri at rulezlan.org>
  Date:   2012-05-10 (Thu, 10 May 2012)

  Changed paths:
    A t/fake/windows/Win32/Job.pm
    M t/tools/win32.t

  Log Message:
  -----------
  ignore on the test on non-Win32 OS


  Commit: 4c6be1e47f6998633f1ac27761cdcee4208b47cf
      https://github.com/fusinv/fusioninventory-agent/commit/4c6be1e47f6998633f1ac27761cdcee4208b47cf
  Author: Gonéri Le Bouder <goneri at rulezlan.org>
  Date:   2012-05-10 (Thu, 10 May 2012)

  Changed paths:
    M t/tools/win32.t

  Log Message:
  -----------
  fix the number of test


  Commit: cb263dd7c7641643f3c5c3b0ee29dc92ef535cdc
      https://github.com/fusinv/fusioninventory-agent/commit/cb263dd7c7641643f3c5c3b0ee29dc92ef535cdc
  Author: Gonéri Le Bouder <goneri at rulezlan.org>
  Date:   2012-05-11 (Fri, 11 May 2012)

  Changed paths:
    M lib/FusionInventory/Agent/HTTP/Client/Fusion.pm

  Log Message:
  -----------
  use uri_escape_utf8 instead of uri_escape

since string can be in UTF8, we have to use uri_escape_utf8()


  Commit: 7f27e58addfb22cf98b375f01755d5c823a83d16
      https://github.com/fusinv/fusioninventory-agent/commit/7f27e58addfb22cf98b375f01755d5c823a83d16
  Author: Gonéri Le Bouder <goneri at rulezlan.org>
  Date:   2012-05-11 (Fri, 11 May 2012)

  Changed paths:
    M lib/FusionInventory/Agent/Task/Inventory.pm

  Log Message:
  -----------
  W32: encode local filename unicode if possible

if Win32::Unicode::File::open() is available, we use it to create
the file name. This is useful for the rare cases when the computer
name if in a local encoding.

closes: #272


  Commit: b8d9015d42195a58de6215f7c5ee39f092a053d9
      https://github.com/fusinv/fusioninventory-agent/commit/b8d9015d42195a58de6215f7c5ee39f092a053d9
  Author: Gonéri Le Bouder <goneri at rulezlan.org>
  Date:   2012-05-11 (Fri, 11 May 2012)

  Changed paths:
    M lib/FusionInventory/Agent/Tools.pm

  Log Message:
  -----------
  don't reencode if perl flagged string as utf8

If Perl flagged a string as UTF8 internally, there is no reason
to reencode it. A best, this will hide a bug somewhere else.

In my opinion, we should no try to encode string at all.

To know if a string is flagged as UTF8, one can use Devel::Peek.


  Commit: ee4e8a8c9003e16b85e49aed5a759d5a563ed43d
      https://github.com/fusinv/fusioninventory-agent/commit/ee4e8a8c9003e16b85e49aed5a759d5a563ed43d
  Author: Gonéri Le Bouder <goneri at rulezlan.org>
  Date:   2012-05-11 (Fri, 11 May 2012)

  Changed paths:
    M lib/FusionInventory/Agent/Tools/Hostname.pm

  Log Message:
  -----------
  use the UTF8 internal struct of Perl

the decode("UCS-2le", $buffer) return a string in Perl internal
format.
If the string is in UTF8, it will be correctly flagged. There is
no reason to try to be cleaver, espcially now getSanitizedString()
correctly deal with such string.
see commit:b8d9015d42195a58de6215f7c5ee39f092a053d9


  Commit: 1d3f864b78ae33218b9a4b79ab52d5e446d31e85
      https://github.com/fusinv/fusioninventory-agent/commit/1d3f864b78ae33218b9a4b79ab52d5e446d31e85
  Author: Gonéri Le Bouder <goneri at rulezlan.org>
  Date:   2012-05-11 (Fri, 11 May 2012)

  Changed paths:
    M lib/FusionInventory/Agent/Task/Inventory/Input/Win32/OS.pm

  Log Message:
  -----------
  do not overwrite hostname returned by getHostname

Tools::Win32::getHostname() returns a valid unicode hostname
unlike WMI.


Compare: https://github.com/fusinv/fusioninventory-agent/compare/65474fc^...1d3f864