Skip to content

Jak and Daxter: Lock PME requirement to 1.6.0. - #6345

Closed
massimilianodelliubaldini wants to merge 2 commits into
ArchipelagoMW:mainfrom
ArchipelaGOAL:jak1-lock-reqs
Closed

Jak and Daxter: Lock PME requirement to 1.6.0.#6345
massimilianodelliubaldini wants to merge 2 commits into
ArchipelagoMW:mainfrom
ArchipelaGOAL:jak1-lock-reqs

Conversation

@massimilianodelliubaldini

Copy link
Copy Markdown
Contributor

What is this fixing or adding?

PyMemoryEditor 2.0.x is incompatible with 1.6.x. Jak's requirements file specifies >=1.6.0 so this will accidentally grab the later version, which won't work with the way Jak's client is currently written. We'll lock the version to =1.6.0 until a future update where we can move the whole client to 2.0.x.

The reason we're not doing that now is there's an active beta world for Jak and I don't want to have multiple versions of the world that require different versions of the same library.

How was this tested?

Not at all.

If this makes graphical changes, please attach screenshots.

N/A.

@github-actions github-actions Bot added the waiting-on: peer-review Issue/PR has not been reviewed by enough people yet. label Jul 19, 2026
Comment thread worlds/jakanddaxter/requirements.txt Outdated
@duckboycool duckboycool added is: maintenance Regular updates to requirements and utilities that do not fix bugs or change/add features. waiting-on: author Issue/PR is waiting for feedback or changes from its author. and removed waiting-on: peer-review Issue/PR has not been reviewed by enough people yet. labels Jul 19, 2026
Co-authored-by: Duck <31627079+duckboycool@users.noreply.github.com>
@duckboycool duckboycool added waiting-on: peer-review Issue/PR has not been reviewed by enough people yet. and removed waiting-on: author Issue/PR is waiting for feedback or changes from its author. labels Jul 19, 2026

@duckboycool duckboycool left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Confirmed at least being able to open the client with this version installed. Probably somebody should test actually playing the game though.

@massimilianodelliubaldini

Copy link
Copy Markdown
Contributor Author

1.6.0 is the version of the library that I have tested on for a little while now, so my only concern would be like, a brand new install of AP with this requirements file to see if it pulls 1.6.0 instead of 2.0.0.

@nicholassaylor nicholassaylor left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spun up a VM installing with this PR and it correctly installs 1.6.0

@massimilianodelliubaldini

Copy link
Copy Markdown
Contributor Author

Nick the GOAT, thank you!

@duckboycool duckboycool added waiting-on: core-review Issue/PR has been peer-reviewed and is ready to be merged or needs input from a core maintainer. and removed waiting-on: peer-review Issue/PR has not been reviewed by enough people yet. labels Jul 21, 2026
Soggy-Pancake added a commit to Soggy-Pancake/ArchipelaGOAL3-Linux that referenced this pull request Aug 2, 2026
@duckboycool

Copy link
Copy Markdown
Collaborator

Indeed is superceded.

@github-actions github-actions Bot removed the waiting-on: core-review Issue/PR has been peer-reviewed and is ready to be merged or needs input from a core maintainer. label Aug 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

is: maintenance Regular updates to requirements and utilities that do not fix bugs or change/add features.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants