flathub / org.xonotic.Xonotic

https://flathub.org/apps/details/org.xonotic.Xonotic
6 stars 5 forks source link

Build `d0_blind_id` as shared library and push key to game basedir #16

Closed 000exploit closed 1 year ago

000exploit commented 1 year ago

Fixes crypto support in game and closes #4. Already tested on servers.

flathubbot commented 1 year ago

Started test build 23843

flathubbot commented 1 year ago

Build 23843 successful To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/6488/org.xonotic.Xonotic.flatpakref
AsciiWolf commented 1 year ago

Thank you a lot for working on this! I will merge it after testing, however I am not entirely sure about the LD_LIBRARY_PATH, XDG_DATA_DIRS variables (these should already be set by Flatpak) and other small changes.

000exploit commented 1 year ago

these should already be set by Flatpak

I thought so, but then I've checked it in shell and saw some manifests on flathub.. Well, they're not defined at all!

000exploit commented 1 year ago

@AsciiWolf so, have you tested it?

AsciiWolf commented 1 year ago

Not yet, sorry. I am too busy with other tasks at the moment. Give me another week or two.

AsciiWolf commented 1 year ago

I am still not sure about setting the LD_LIBRARY_PATH and XDG_DATA_DIRS. The /app/lib/xonotic dir is already used by Xonotic and works fine without setting it manually. And /app/share does not seem to be used for Xonotic data (/app/lib/xonotic/data is).

000exploit commented 1 year ago

The /app/lib/xonotic dir is already used by Xonotic and works fine without setting it manually.

It didn't work before for me, I'll check it now.

000exploit commented 1 year ago

Yes, they're not required anymore

flathubbot commented 1 year ago

Started test build 24701

flathubbot commented 1 year ago

Build 24701 successful To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/7348/org.xonotic.Xonotic.flatpakref
flathubbot commented 1 year ago

Started test build 24704

flathubbot commented 1 year ago

Build 24704 successful To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/7351/org.xonotic.Xonotic.flatpakref
flathubbot commented 1 year ago

Started test build 24706

flathubbot commented 1 year ago

Build 24706 successful To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/7353/org.xonotic.Xonotic.flatpakref
flathubbot commented 1 year ago

Started test build 24709

flathubbot commented 1 year ago

Build 24709 successful To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/7357/org.xonotic.Xonotic.flatpakref
flathubbot commented 1 year ago

Started test build 24710

flathubbot commented 1 year ago

Build 24710 successful To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/7358/org.xonotic.Xonotic.flatpakref
AsciiWolf commented 1 year ago

Tested and here is how big noob I am. :-D So I think it works and is ready to be merged. :-)

000exploit commented 1 year ago

xd

At least you've tested it and it works, what's pretty important.