chuck-flowers / humble-lumpia

A script for managing installations of Glorious Eggroll's custom proton build
GNU General Public License v3.0
14 stars 2 forks source link

AUR package #5

Closed NINNiT closed 3 years ago

NINNiT commented 3 years ago

Hi,

I've created an AUR package for easy humble-lumpia package management for ArchLinux users: AUR.

Just FYI, whether or not you want to add it to the README is up to you :-) Installation can be done e.g. with an AUR helper like yay using yay -S humble-lumpia-git.

Thanks for the nice software!

NINNiT

chuck-flowers commented 3 years ago

Hey thanks for the contribution. I'll take a look and give you some feedback once I have some time and if I see anything I think should be updated, I'll let you know. Once any issues are taken care of, I'd be happy to add it as an installation option in the README with a mention to you.

lpww commented 3 years ago

I installed from AUR yesterday. It works well! Thanks

EDIT: Does it really only have 3 dependencies? I followed the GE repo readme and installed many more dependencies than are listed in the AUR package. Eg wine-staging, winetricks, a bunch of wine dependencies, and a bunch of vulkan dependencies. I did that before using humble-lumpia, so am not 100% sure if it was required

chuck-flowers commented 3 years ago

@lpww humble-lumpia simply acts as a downloader and manager of multiple versions of Glorious Eggroll's custom version of proton. I don't feel like installing humble-lumpia should necessarily install Proton's dependencies. I'm open to a conversation about that though.

@NINNiT I noticed that curl and jq are not included as a dependency in the AUR package. These should be added since they are used to download and parse the files provided by GitHub.

lpww commented 3 years ago

@chuck-flowers I agree that humble-lumpia should not install Proton's dependencies. It would be difficult or impossible to account for all the different package managers out there.

However, I do think there is a case for the AUR package to include the proton dependencies. This GE package includes the proton dependencies. Most of them are optional with a note about when you might need them. I think this would be a good approach for the humble-lumpia AUR package and would improve the installation experience

NINNiT commented 3 years ago

@chuck-flowers curl and jq are definitely dependencies which should get installed with the AUR package as it's strictly necessary to run the software.

@lpww I do agree that there really is a case to offer them as optional dependencies. I guess the majority of users coming to humble-lumpia already have a working proton (-ge) installation, but it wouldn't hurt to offer them within the AUR package.

Maybe the humble-lumpia README could also refer to the proton-ge-repo or at least tell users that they have to install the proton dependencies prior to using humble-lumpia.

Thanks for the input, i'll update the PKGBUILD as soon as i have time and let you guys know! :+1:

lpww commented 3 years ago

Thanks @NINNiT ! I agree that updating the humble-lumpia README to mention proton dependencies would be a good idea.

NINNiT commented 3 years ago

I've updated the AUR package.

It now contains curl and jq as dependencies, as well as all (optional) dependencies needed for proton-ge-custom. For the latter, I've basically used the same as the AUR package linked above by @lpww.

Works on my side, however it would be nice if @lpww could give it another test run :+1:

lpww commented 3 years ago

The new dependencies look good! It makes sense that the dependencies from the other GE package can be used here. I'll give it a test run later tonight

chuck-flowers commented 3 years ago

I'll also try it out shortly. Just recently switched to Arch and I haven't installed this project form the AUR yet so I'm interested in seeing what the user experience is like.

chuck-flowers commented 3 years ago

@NINNiT I tried installing from the AUR package and it seems to work well. The only question I had was why python and vulkan-icd-loader seem to be marked as required dependencies? Shouldn't it be optional like the rest of the proton dependencies or am I reading something incorrectly?

I've created a PR for updating the README. If either of you have time, feel free to read over the new changes to the README, and give me any feedback on the wording or something additional that should be added.

EDIT: made note of a similar issue with the python dependency.

NINNiT commented 3 years ago

@chuck-flowers Thanks for taking another look! I've added python and vulkan-icd-loader as required dependencies, following the PKGBUILD of proton-ge-custom-bin which seems to imply that those are strictly needed for running GloriousEggroll's proton builds. Contrary to the optional dependencies, which might offer better performance and user experience but are maybe not directly necessary to get a working proton installation (e.g fsync-patched kernels, xbox controller driver,...).

PR looks good! :100:

chuck-flowers commented 3 years ago

Okay thanks for looking at the PR. We can leave the AUR package as is. I'll merge the PR at the end of the day to allow for any other comments to be made.

lpww commented 3 years ago

According to the Arch Wiki, vulkan-icd-loader (and vulkan graphics drivers) are required to run vulkan programs. A fair amount of games are using Vulkan at this point, so I think it makes sense for it to be a required dependency.

python I am not sure sure about but it's probably safe to assume that if it's required for proton-ge-custom-bin, it is probably required for humble-lumpia-git.

I updated my system and saw logs about the optional dependencies. It seems to be working!

NINNiT commented 3 years ago

I've done a bit of digging, seems that the proton start script itself and a few more scripts are using python.

chuck-flowers commented 3 years ago

I think the vast majority of people who are going to use this will likely already have steam installed so we can leave the AUR package as declared for now and revisit the dependencies once the project is stable and we are getting ready to make a humble-lumpia (non git) AUR package available.

chuck-flowers commented 3 years ago

@NINNiT, sorry to necro this issue but I don't have another way to contact you currently. I made an update to the repo that will require a new build dependency in the AUR package. I added man pages to the project and gzip is necessary now. The relevant issue is #11.