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

dxx-rebirth game data #3922

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

s1eve-mcdichae1
Copy link
Contributor

Don't download D2 shareware data if registered version files exist.

Ignore case - don't download D1/D2 shareware whether user-provided data files in ALL CAPS (DOS) or lowercase (Steam).

Shareware and add-on files no longer available from dxx-rebirth.com - switch to self-hosted files at $__archive_url (I don't know how to actually put them there, though. Someone else will have to do that part, or show me how.)

@joolswills
Copy link
Member

I have some dxx rebirth changes in progress including the game data locations. I've already uploaded them to our server I think. I'll try and finish it this week.

In 77bb4c1 the code was locked to an older commit due to incompatibilities with Raspberry Pi OS Buster, but this version doesn't build with newer GCC.

Switch to using the latest code by default but lock to older versions for Buster and GCC 10 and lower

Tested to build correctly on Raspberry Pi OS Buster, Bullseye and Bookworm.
Newer dxx-rebirth uses a build/ subfolder.
The official site no longer hosts the files.
@joolswills
Copy link
Member

joolswills commented May 21, 2024

Please see #3923

I am happy to include your changes minus the shareware data part. I can cherry-pick your commits into my branch if you like or you can open a new PR with them.

One thing I would change in your code is to not use 2 find calls and instead do

[[ -z "$(find "$dest_d2" -maxdepth 1 \( -iname D2DEMO.HOG -o -iname DESCENT2.HOG \))" ]]

Thanks for the PR :-)

@s1eve-mcdichae1
Copy link
Contributor Author

I can cherry-pick your commits into my branch if you like or you can open a new PR with them.

So I would open a pull against your branch, then, you mean?

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