Skip to content

feat: update-rpcs3 no descarga si ya esta la ultima version - #67

Merged
Xalcker merged 1 commit into
mainfrom
feat/update-skip-same-version
Oct 2, 2026
Merged

Xalcker merged 1 commit into
mainfrom
feat/update-skip-same-version

Conversation

@Xalcker

@Xalcker Xalcker commented Oct 2, 2026

Copy link
Copy Markdown
Owner

Problema

La opcion 7 del menu (update-rpcs3) descargaba (~300 MB) y reinstalaba RPCS3 siempre, sin comparar con la version instalada.

Cambio

  • El updater guarda la URL del AppImage instalado en /opt/RPCS3/.rpcs3-url (el nombre lleva la version). La marca se escribe dentro de la instalacion nueva antes de moverla, asi solo existe si el reemplazo se completo.
  • Si la URL del ultimo release coincide, avisa y sale sin descargar. update-rpcs3 --force reinstala igual.
  • install_rpcs3 deja la marca en la instalacion inicial. Una instalacion sin marca descarga una vez.
  • El menu dice "Listo." en lugar de "Actualizacion completada." (tambien aplica cuando no habia nada que actualizar).
  • README y 2 tests nuevos.

Verificacion

  • Ejecute el updater renderizado con un curl y un AppImage falsos (misma tecnica que los tests): sin marca descarga; con la misma version no descarga ni reemplaza; --force reinstala; con marca de otra version descarga y deja la marca nueva.
  • Sintaxis de lib y tests validada. No probado: los tests bats no se corrieron (sin bats en el equipo), ni contra GitHub real.
  • No probado el updater con una ejecucion real en el kiosko.

🤖 Generated with Claude Code

…reinstala)

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@Xalcker
Xalcker merged commit dc854e0 into main Oct 2, 2026
1 of 2 checks passed
@Xalcker
Xalcker deleted the feat/update-skip-same-version branch October 3, 2026 03:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant