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

Descent 1,2,3: Replace symbolic links with bind mounts. #1052

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

t0b10-r3tr0
Copy link
Contributor

@t0b10-r3tr0 t0b10-r3tr0 commented Dec 15, 2024

To enable exFAT support on certain CFWs, We replaced symbolic links with bind mounts using the bind_directories function included with PortMaster GUI.

The scope of this task was limited to implementing bind_directories and any fixes preventing the port from launching. Each port was tested in Knulli and one other PM supported CFW (ROCKNIX, most times).

The following criteria were used to validate that a port had passed our testing:

  • The port loads without issue, even after rebooting the device.
  • The saves/settings persist, even after rebooting the device.
  • If port was installed previously, the existing saves/settings were preserved when testing the new version.

Copy link
Contributor

Choose a reason for hiding this comment

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

Aren't these in PR 1050 ?

Copy link
Contributor Author

@t0b10-r3tr0 t0b10-r3tr0 Dec 18, 2024

Choose a reason for hiding this comment

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

No. Descent and Marathon. They're different developers, platforms, etc. Descent was on DOS, by Interplay(?) and Marathon was on Mac, by Bungie. They're both oddly packaged by 3. Hahaha. I am so dense sometimes. Fixing this.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's now fixed. I don't know how those two got mixed up but after merging with upstream, it resolved itself and both PRs are only associated the files related to the PR.

@t0b10-r3tr0 t0b10-r3tr0 requested a review from kloptops December 18, 2024 23:49
@@ -57,7 +56,7 @@ else
fi

if [ "$LIBGL_FB" != "" ]; then
export SDL_VIDEO_GL_DRIVER="$GAMEDIR/gl4es.aarch64/libGL.so.1"
Copy link
Collaborator

Choose a reason for hiding this comment

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

What does this change have to do with bind mounts?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This was one of the issues we had to address because it prevented the port from starting. So, it was done in the context of the bind-directories fixes. I figured not to make 2 PRs for the change.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I know but the title of the PR is replace symbolic links and then does other things :P

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.

3 participants