Mi media center - Foro

Me han contestado desde la lista de xbmc-addons (qué rapidez!) y nos piden que hagamos cambios. Os copio el correo que he enviado:

Name: pelisalacarta
addon: plugin.video.pelisalacarta
version: 2.17
url: https://xbmc-tvalacarta.googlecode.com/ ... isalacarta
revision: HEAD

Lo que me han contestado:

hi,

attached are a patch with some cosmetics changes as well as removal of
unnedded files. also please add a proper, english description to
addon.xml - i updated it so yours is properly registered as the
spanish one.

it seems your plugin has some kind of updating ability? please remove
that, we only want updates happening through the central system.
also what's up with the zip file in resources/lib?
finally, it appears you cache data to the library/ subfolder of the
add-on. this is unacceptable, you must store data to
special://profile/addon_data/<addonid>/

spiff

Y mi respuesta:

I have reviewed the changes you propose. I explain this below but i have a question.

This is the first time i apply a diff patch using SVN, but if i applied it ok you have deleted the license.txt file. Is this ok or is there any problem with the file name / license type? I think it is a requisite using GPL or similar license.

My commens for the changes:

- English description in addon.xml: Ok, i'll add this description.
- Updating: I'll remove this feature.
- Zip file in resources/lib: This is a library that cointains .py files. I download the code and used as is, but i'll extract them and remove the zip file.
- The plugin stores files in library, bookmarks and other directories. I'll change to use user_data folder as you request.

Thanks,
Jesús

Básicamente hay dos cosas que tener en cuenta:

- No usar actualizaciones automáticas: Me parece lógico, haré un proceso para que el empaquetado de la versión XBMC-ADDON no lo incluya y a correr.
- No almacenar ficheros en los directorios del plugin: Esto es algo más complicado, aunque tampoco mucho, pero que sí que requerirá tocar código en cosas que habéis hecho los demás.

No me importa hacerlo a mí, aunque desde luego agradeceré la ayuda y la comprensión en caso de que rompa algo. Entiendo que el cambio afecta a:

1) Descargas y favoritos
2) Control de vistos y favoritos de los canales de Lily
3) Biblioteca
4) Subtítulos

El 1) lo tengo controlado, y al 2) y al 4) les echaré un vistazo con el permiso de Lily y Bandavi porque es algo que quería incorporar de una forma más global a todo el plugin.

El 3) creo que ya lo estaba mirando @Jurrabi, si es que no está resuelto.

¿Me dejo algo?

me pongo con mi parte. No creo que tarde mucho... te mantengo al tanto.


PD. Al spiff lo conozco y es un figura. Funciona superápido y superbien.

Estoy buscando la forma de crear archivos/carpetas con el path special://... si lo has encontrado tu ya, te importaría ponerlo por aquí?

Si lo encuentro yo lo pongo también, lógicamente...

Echa un vistazo a favoritos.py... la funcion xbmc.translatePath se encarga de eso y convierte la ruta "special://" a una ruta nativa que puedes utilizar.

si, ya lo tenía... pero no me diste tiempo a postear gracias.

Acabo de subir las modificaciones del directorio library de los ficheros strm. Lo he probado en 9.11 y en Dharma y en ambos funciona bien. No lo he probado en otras plataformas, pero no tendría porqué afectar...

He estado chateando con spiff por el IRC. Me ha comentado que el "cambió" a la vuelta varios archivos. En concreto el final de linea a modo unix en lugar de modo windows.
Me pidió también que nos aseguremos de que todos los archivos del plugin tiene el fin de linea en modo unix y no windows ya que no se lleva muy bien con sus sistemas git...

He aprovechado también para empujar un poquito el tema de los strm... cruzo los dedos a ver si conseguimos añadir películas

Voy a buscar un método de buscar los archivos que tengan el fin de linea windows o de postearlos siempre en modo unix. Se que eso existe en las herramietnas de git, pero no lo recuerdo en tortoise.

PD. Por cierto, hay que aclarar también en el post que el nuevo directorio que se debe añadir a la biblioteca es:

• special://profile/addon_data/plugin.video.pelisalacarta/library

Ahora voy a comer, pero si quieres que me encargue de otros cambios esta tarde estoy libre... tu me dices.

Lo de los saltos de línea se puede solucionar en Linux... es fácil.

Dado que el tema de los branches y el trunk sólo tenía sentido para el SVN repo installer, he pensado que a partir de ahora vamos a trabajar directamente sobre el trunk. Actualiza tu copia de trabajo a ese directorio, que además ya se llama "plugin.video.pelisalacarta".

https://xbmc-tvalacarta.googlecode.com/ ... isalacarta

Cuando publique una nueva versión haré un tag (lo he visto hacer en otros plugins para publicar el addon en xbmc) que es más sencillo que hacer el merge.

Estoy mirando lo de los findelinea... Cada vez me gusta más Eclipse (cuando tenga un rato me grabo un par de videos de pruebitas para que juzgueis).

El caso es que no encotraba nada útil ni con Tortoise ni con Ultraedit así que miré en eclipse. Y a la primera PUM... En el menú File->Convert Line Delimiters to->Unix

Esto se aplica en todo el proyecto y produjo los siguientes resultados:

channels/filmstreaming.py
channels/veranime.py
youtubeAPI/gdata/oauth/CHANGES.txt
bookmarks/vistos/casttv.txt

Supongo que este último no debería estar ahí puesto que vamos a almacenar todo en addon_data pero lo he convertido igualmente (o mejor dicho, eclipse lo hizo por mi) y lo he subido en el parche 330.

jesus escribió:Actualiza tu copia de trabajo a ese directorio, que además ya se llama "plugin.video.pelisalacarta".

Hemos publicado al tiempo y me temo que mis últimos cambios han ido al branch...