prompt: You will be provided with a partial code base and an issue statement explaining a problem to resolve.
  <issue>
  About option of win_package
  I found two issue.
  
  I think Product-ID parameter is not "product_id" , is it "productid"?
  Also , it seems the required columns is "yes".
  
  ```
  fatal: [10.1.1.6]: FAILED! => {"changed": false, "failed": true, "msg": "Missing required argument: productid"
  ```
  
  Therefore , it take a mistake about "ProductId" below an example on document of win_package:
  
  ```
  # Playbook example
    - name: Install the vc thingy
      win_package:
        name="Microsoft Visual C thingy"
        path="http://download.microsoft.com/download/1/6/B/16B06F60-3B20-4FF2-B699-5E9B7962F9AE/VSU_4/vcredist_x64.exe"
        ProductId="{CF2BEA3C-26EA-32F8-AA9B-331F7E34BA97}"
        Arguments="/install /passive /norestart"
  ```
  
  
  </issue>
  <code>
  [start of windows/win_package.py]
  1 #!/usr/bin/python
  2 # -*- coding: utf-8 -*-
  3 
  4 # (c) 2014, Trond Hindenes <trond@hindenes.com>, and others
  5 #
  6 # This file is part of Ansible
  7 #
  8 # Ansible is free software: you can redistribute it and/or modify
  9 # it under the terms of the GNU General Public License as published by
  10 # the Free Software Foundation, either version 3 of the License, or
  11 # (at your option) any later version.
  12 #
  13 # Ansible is distributed in the hope that it will be useful,
  14 # but WITHOUT ANY WARRANTY; without even the implied warranty of
  15 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  16 # GNU General Public License for more details.
  17 #
  18 # You should have received a copy of the GNU General Public License
  19 # along with Ansible.  If not, see <http://www.gnu.org/licenses/>.
  20 
  21 # this is a windows documentation stub.  actual code lives in the .ps1
  22 # file of the same name
  23 
  24 DOCUMENTATION = '''
  25 ---
  26 module: win_package
  27 version_added: "1.7"
  28 short_description: Installs/Uninstalls a installable package, either from local file system or url
  29 description:
  30      - Installs or uninstalls a package
  31 options:
  32   path:
  33     description:
  34       - Location of the package to be installed (either on file system, network share or url)
  35     required: true
  36     default: null
  37     aliases: []
  38   name:
  39     description:
  40       - name of the package. Just for logging reasons, will use the value of path if name isn't specified
  41     required: false
  42     default: null
  43     aliases: []
  44   product_id:
  45     description:
  46       - product id of the installed package (used for checking if already installed)
  47     required: false
  48     default: null
  49     aliases: []
  50   arguments:
  51     description:
  52       - Any arguments the installer needs
  53     default: null
  54     aliases: []
  55   state:
  56     description:
  57       - Install or Uninstall
  58     choices:
  59       - present
  60       - absent
  61     default: present
  62     aliases: [ensure]
  63   user_name:
  64     description:
  65       - Username of an account with access to the package if its located on a file share. Only needed if the winrm user doesn't have access to the package. Also specify user_password for this to function properly.
  66     default: null
  67     aliases: []
  68   user_password:
  69     description:
  70       - Password of an account with access to the package if its located on a file share. Only needed if the winrm user doesn't have access to the package. Also specify user_name for this to function properly.
  71     default: null
  72     aliases: []
  73 author: Trond Hindenes
  74 '''
  75 
  76 EXAMPLES = '''
  77 # Playbook example
  78   - name: Install the vc thingy
  79     win_package:
  80       name="Microsoft Visual C thingy"
  81       path="http://download.microsoft.com/download/1/6/B/16B06F60-3B20-4FF2-B699-5E9B7962F9AE/VSU_4/vcredist_x64.exe"
  82       ProductId="{CF2BEA3C-26EA-32F8-AA9B-331F7E34BA97}"
  83       Arguments="/install /passive /norestart"
  84 
  85 
  86 '''
  87 
  88
  [end of windows/win_package.py]
  </code>
  
  I need you to solve this issue by generating a single patch file that I can apply directly to this repository using git apply. Please respond with a single patch file in the following format.
  <patch>
  diff --git a/file.py b/file.py
  --- a/file.py
  +++ b/file.py
  @@ -1,27 +1,35 @@
   def euclidean(a, b):
  -    while b:
  -        a, b = b, a % b
  -    return a
  +    if b == 0:
  +        return a
  +    return euclidean(b, a % b)
   
   
   def bresenham(x0, y0, x1, y1):
       points = []
       dx = abs(x1 - x0)
       dy = abs(y1 - y0)
  -    sx = 1 if x0 < x1 else -1
  -    sy = 1 if y0 < y1 else -1
  -    err = dx - dy
  +    x, y = x0, y0
  +    sx = -1 if x0 > x1 else 1
  +    sy = -1 if y0 > y1 else 1
   
  -    while True:
  -        points.append((x0, y0))
  -        if x0 == x1 and y0 == y1:
  -            break
  -        e2 = 2 * err
  -        if e2 > -dy:
  -            err -= dy
  -            x0 += sx
  -        if e2 < dx:
  -            err += dx
  -            y0 += sy
  +    if dx > dy:
  +        err = dx / 2.0
  +        while x != x1:
  +            points.append((x, y))
  +            err -= dy
  +            if err < 0:
  +                y += sy
  +                err += dx
  +            x += sx
  +    else:
  +        err = dy / 2.0
  +        while y != y1:
  +            points.append((x, y))
  +            err -= dx
  +            if err < 0:
  +                x += sx
  +                err += dy
  +            y += sy
  +
  +    points.append((x, y))
       return points
  
  </patch>
golden_diff: diff --git a/windows/win_package.py b/windows/win_package.py
  --- a/windows/win_package.py
  +++ b/windows/win_package.py
  @@ -25,6 +25,7 @@
   ---
   module: win_package
   version_added: "1.7"
  +author: Trond Hindenes
   short_description: Installs/Uninstalls a installable package, either from local file system or url
   description:
        - Installs or uninstalls a package
  @@ -44,9 +45,9 @@
     product_id:
       description:
         - product id of the installed package (used for checking if already installed)
  -    required: false
  +    required: true
       default: null
  -    aliases: []
  +    aliases: [productid]
     arguments:
       description:
         - Any arguments the installer needs
  @@ -79,7 +80,7 @@
       win_package:
         name="Microsoft Visual C thingy"
         path="http://download.microsoft.com/download/1/6/B/16B06F60-3B20-4FF2-B699-5E9B7962F9AE/VSU_4/vcredist_x64.exe"
  -      ProductId="{CF2BEA3C-26EA-32F8-AA9B-331F7E34BA97}"
  +      Product_Id="{CF2BEA3C-26EA-32F8-AA9B-331F7E34BA97}"
         Arguments="/install /passive /norestart"
  
verification_info: {"golden_diff": "diff --git a/windows/win_package.py b/windows/win_package.py\n--- a/windows/win_package.py\n+++ b/windows/win_package.py\n@@ -25,6 +25,7 @@\n ---\n module: win_package\n version_added: \"1.7\"\n+author: Trond Hindenes\n short_description: Installs/Uninstalls a installable package, either from local file system or url\n description:\n      - Installs or uninstalls a package\n@@ -44,9 +45,9 @@\n   product_id:\n     description:\n       - product id of the installed package (used for checking if already installed)\n-    required: false\n+    required: true\n     default: null\n-    aliases: []\n+    aliases: [productid]\n   arguments:\n     description:\n       - Any arguments the installer needs\n@@ -79,7 +80,7 @@\n     win_package:\n       name=\"Microsoft Visual C thingy\"\n       path=\"http://download.microsoft.com/download/1/6/B/16B06F60-3B20-4FF2-B699-5E9B7962F9AE/VSU_4/vcredist_x64.exe\"\n-      ProductId=\"{CF2BEA3C-26EA-32F8-AA9B-331F7E34BA97}\"\n+      Product_Id=\"{CF2BEA3C-26EA-32F8-AA9B-331F7E34BA97}\"\n       Arguments=\"/install /passive /norestart\"\n", "issue": "About option of win_package\nI found two issue.\n\nI think Product-ID parameter is not \"product_id\" , is it \"productid\"?\nAlso , it seems the required columns is \"yes\".\n\n```\nfatal: [10.1.1.6]: FAILED! => {\"changed\": false, \"failed\": true, \"msg\": \"Missing required argument: productid\"\n```\n\nTherefore , it take a mistake about \"ProductId\" below an example on document of win_package:\n\n```\n# Playbook example\n  - name: Install the vc thingy\n    win_package:\n      name=\"Microsoft Visual C thingy\"\n      path=\"http://download.microsoft.com/download/1/6/B/16B06F60-3B20-4FF2-B699-5E9B7962F9AE/VSU_4/vcredist_x64.exe\"\n      ProductId=\"{CF2BEA3C-26EA-32F8-AA9B-331F7E34BA97}\"\n      Arguments=\"/install /passive /norestart\"\n```\n\n", "before_files": [{"content": "#!/usr/bin/python\n# -*- coding: utf-8 -*-\n\n# (c) 2014, Trond Hindenes <trond@hindenes.com>, and others\n#\n# This file is part of Ansible\n#\n# Ansible is free software: you can redistribute it and/or modify\n# it under the terms of the GNU General Public License as published by\n# the Free Software Foundation, either version 3 of the License, or\n# (at your option) any later version.\n#\n# Ansible is distributed in the hope that it will be useful,\n# but WITHOUT ANY WARRANTY; without even the implied warranty of\n# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the\n# GNU General Public License for more details.\n#\n# You should have received a copy of the GNU General Public License\n# along with Ansible.  If not, see <http://www.gnu.org/licenses/>.\n\n# this is a windows documentation stub.  actual code lives in the .ps1\n# file of the same name\n\nDOCUMENTATION = '''\n---\nmodule: win_package\nversion_added: \"1.7\"\nshort_description: Installs/Uninstalls a installable package, either from local file system or url\ndescription:\n     - Installs or uninstalls a package\noptions:\n  path:\n    description:\n      - Location of the package to be installed (either on file system, network share or url)\n    required: true\n    default: null\n    aliases: []\n  name:\n    description:\n      - name of the package. Just for logging reasons, will use the value of path if name isn't specified\n    required: false\n    default: null\n    aliases: []\n  product_id:\n    description:\n      - product id of the installed package (used for checking if already installed)\n    required: false\n    default: null\n    aliases: []\n  arguments:\n    description:\n      - Any arguments the installer needs\n    default: null\n    aliases: []\n  state:\n    description:\n      - Install or Uninstall\n    choices:\n      - present\n      - absent\n    default: present\n    aliases: [ensure]\n  user_name:\n    description:\n      - Username of an account with access to the package if its located on a file share. Only needed if the winrm user doesn't have access to the package. Also specify user_password for this to function properly.\n    default: null\n    aliases: []\n  user_password:\n    description:\n      - Password of an account with access to the package if its located on a file share. Only needed if the winrm user doesn't have access to the package. Also specify user_name for this to function properly.\n    default: null\n    aliases: []\nauthor: Trond Hindenes\n'''\n\nEXAMPLES = '''\n# Playbook example\n  - name: Install the vc thingy\n    win_package:\n      name=\"Microsoft Visual C thingy\"\n      path=\"http://download.microsoft.com/download/1/6/B/16B06F60-3B20-4FF2-B699-5E9B7962F9AE/VSU_4/vcredist_x64.exe\"\n      ProductId=\"{CF2BEA3C-26EA-32F8-AA9B-331F7E34BA97}\"\n      Arguments=\"/install /passive /norestart\"\n\n\n'''\n\n", "path": "windows/win_package.py"}]}