Hacer las excepciones al comienzo más amigables - popen2 por subprocess

[Federico Moreira]

Estaba pensando en un

try:
    import pygame
catch:
   "No tienes pygame instalado"

y un

if version == 3.0:
    "Tenes que usar 2.6 sino te va a dar errores"

Por supuesto algo más lindo, la idea es que si no tenes pygame instalado o estás usando python 3.0 te de una advertencia amigable y te diga como poder hacer andar el chess.
Si les gusta la idea me pongo a trabajar en eso.

Otra cosa, el módulo popen2 está descontinuado[0] desde 2.6, se que las XO tienen 2.5 y no es algo taaaaan trivial.
¿Podríamos plantear como un issue sustituirlo por subprocess?

Saludos.

[0] - http://docs.python.org/library/popen2.html

[Alejandro Segovia]

Hola,

2009/3/1 Federico Moreira <fefom...@gmail.com>

Estaba pensando en un

try:
    import pygame
catch:
   "No tienes pygame instalado"

y un

if version == 3.0:
    "Tenes que usar 2.6 sino te va a dar errores"

Por supuesto algo más lindo, la idea es que si no tenes pygame instalado o estás usando python 3.0 te de una advertencia amigable y te diga como poder hacer andar el chess.
Si les gusta la idea me pongo a trabajar en eso.

Me parece excelente! adelante! :D
 

Otra cosa, el módulo popen2 está descontinuado[0] desde 2.6, se que las XO tienen 2.5 y no es algo taaaaan trivial.
¿Podríamos plantear como un issue sustituirlo por subprocess?

No tenía idea que popen2 estaba descontinuado. ¿Te animás a cambiarlo también?

Committea directo al repo, no seas tímido ;)
 
Saludos!

Saludos.

[0] - http://docs.python.org/library/popen2.html

--
Varrojo@Linux
http://varrojo.linuxuruguay.org

[Federico Moreira]

Pronta la parte del try/catch, en el correr del día agrego más cambios.

[Alejandro Segovia]

2009/3/16 Federico Moreira <fefom...@gmail.com>

Pronta la parte del try/catch, en el correr del día agrego más cambios.

d+ Fefo. Me ayudas pila. Cree un branch del trunk en r158, el cual se encuentra en esta URL: https://ceibal-chess.googlecode.com/svn/branches/compat

La idea es que la próxima beta salga de compat y no del trunk, ya que éste de por sí no es más compatible con la XO por lo que les comentaba en el otro mail.

Por otro lado, el código que estás agregando me gustaría que llegara al próximo beta, ya que realmente ayudaría a un usuario a entender por qué la aplicación no funciona si te falta Pygame. Entonces, mi duda es, te animarías a hacer tus cambios sobre compat en vez de sobre trunk?

Simplemente hacete un checkout limpio de la URL de compat y rehacé tus cambios sobre esa. Una vez pronto podés commitear directamente. Esto nos permitirá que las mejoras que vayas agregando lleguen a los usuarios lo antes posible.

Si te embola y querés seguir en el trunk, todo bien, pero tené en cuenta que hasta que reescriba todo el movimiento de piezas esos cambios no llegarán a los releases :-/

Abrazo,
Ale.-

 

--
Varrojo@Linux
http://varrojo.linuxuruguay.org

[Federico Moreira]

El 16 de marzo de 2009 14:31, Alejandro Segovia <ase...@gmail.com> escribió:

Si te embola y querés seguir en el trunk, todo bien, pero tené en cuenta que hasta que reescriba todo el movimiento de piezas esos cambios no llegarán a los releases :-/

Claro, mi idea es que nadie use nunca jamás mis modificaciones (risa malvada)

Lo subo a compat.

Saludos.