Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make it possible to pass rpyc version value as an argument #4572

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Andykl
Copy link
Member

@Andykl Andykl commented Apr 24, 2023

Main use-case for that is to run renpy --compile --rpyc-version int(HEAD_COMMIT_SHA, 16) in Git environment (by git pull/checkout hook or other means), so we have the same rpycs content between members without involving of old-game (I used that approach and it wasn't very convenient).

I don't think it makes old-game obsolete, because for non-techie it is easier to copy and paste folders with rpycs then playing with console.

@Andykl Andykl marked this pull request as draft April 26, 2023 02:10
@Andykl
Copy link
Member Author

Andykl commented Apr 26, 2023

Note for myself. This could cause Name (..., ..., 0) is defined twice after reload because the value is stored globally.

@renpytom
Copy link
Member

renpytom commented Apr 27, 2023

I'm not convinced that this is correct (that is, that it will wind up with the same .rpyc files for different members of the team), at least in the case where someone updates through multiple revisions.

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.

None yet

2 participants