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

Allow for multiple users #23

Merged
merged 3 commits into from
Nov 19, 2020

Conversation

bryceml
Copy link
Collaborator

@bryceml bryceml commented Jun 16, 2020

This simplifies finding the home directory so we can support non /home/*
locations.

It also installs all desktop files, executables, and icons in the users
account so they don't interfere with any other users, this allows
multiple users to potentially install on the same machine.

This simplifies finding the home directory so we can support non /home/*
locations.

It also installs all desktop files, executables, and icons in the users
account so they don't interfere with any other users, this allows
multiple users to potentially install on the same machine.
@bryceml bryceml changed the title Allow for multiple users WIP: Allow for multiple users Jun 16, 2020
@bryceml bryceml marked this pull request as draft June 16, 2020 16:15
@brandleesee
Copy link
Owner

I'll be happy to merge for Manjaro. At one point I even wanted to add for Arch but because I am as yet unfamiliar with that refrained from attempting.

When we get to include Manjaro, the script's name will have to change !!

@brandleesee brandleesee marked this pull request as ready for review November 9, 2020 11:38
@brandleesee
Copy link
Owner

Is this ready for merging, mate?

@bryceml
Copy link
Collaborator Author

bryceml commented Nov 17, 2020

I tested it on both ubuntu desktop 20.04 and xubuntu 20.04 and it seems to work fine, so we can probably merge this.

Also, I updated https://github.com/bryceml/epsxe-snap with a message that it looks like https://github.com/ahimta/epsxe-tools is probably the best future for a snap version of epsxe.

@bryceml bryceml changed the title WIP: Allow for multiple users Allow for multiple users Nov 18, 2020
@brandleesee brandleesee merged commit 1976469 into brandleesee:master Nov 19, 2020
@brandleesee
Copy link
Owner

Thank you very much Bry! Appreciated. :)

Re snap, no problem, shall refer a particular user to the other repository.

@bryceml bryceml deleted the allow_multiple_users branch December 5, 2020 05:45
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.

2 participants