Setup.py referencia archivos locales

[pepeleproso]

Siguiendo los pasos para generar el instalador de windows, me tope con que el script referencia un directorio que no se menciona en la configuracion del entorno.
Si se copian los archivos esperados funciona sin problemas. Mi duda es si lo mejor es mencionar esto en las instrucciones o si subir esos archivos al repo para simplificar el proceso. Por cambios en windows esos archivos no son faciles de conseguir y en otro proyecto optamos por incluir esa dll y manifiesto en una carpeta en el repo directamente.

if sys.version_info > (2, 7):
    # add "Microsoft Visual C++ 2008 Redistributable Package (x86)"
    if os.path.exists(r"c:\Program Files\Mercurial"):
        data_files += [(
            ".", glob.glob(r'c:\Program Files\Mercurial\msvc*.dll') +
                 glob.glob(r'c:\Program Files\Mercurial\Microsoft.VC90.CRT.manifest'),

)]

[reingart]

Esos archivos son los Redistribuibles de Microsoft (como esta indicado en el comentario), y hay todo un tema de licenciamiento (no son software libre).

Para evitar problemas y complicaciones no los subiría al repositorio (y de hecho, son binarios y pueden variar). Lo mismo parecen hacer otros proyectos, como Mercurial (hg).

Posiblemente haya que aclararlo en la documentación, en general se asume que utiliza mercurial para clonar en Windows (como esta ejemplificado en el intructivo), por lo que esos archivos deberían existir y no habría problemas.

Lamentablemente no veo que se pueda hacer un PR a la Wiki, pueden clonarla, hacer el cambio, subirlo a su repo y pasarmelo para revisarlo / hacer el merge manualmente.
Otra opción sería que adjunten el diff o mencionen los cambios a hacer en este ticket.

Supongo que están hablando de la página InstalacionCodigoFuente
El repo de la wiki está en https://github.com/reingart/pyafipws.wiki.git

Para más info sobre los redist ver:

• https://wiki.python.org/moin/WindowsCompilers
• http://www.py2exe.org/index.cgi/Tutorial#Step522
• https://www.mercurial-scm.org/wiki/BuildingWindowsInstaller