Kozova1 / parsec-fedora

Parsec port to fedora
39 stars 6 forks source link

RPM cleanup for Fedora and some changes to stability of the script #2

Closed magikmw closed 5 years ago

magikmw commented 5 years ago

Hi,

Thanks for creating this script, I never considered converting .deb packages to rpm directly. Since I dislike piping wget/curl output directly to bash, I've run the commands one by one and discovered there's some improvements I could make.
Here's a list of changes:

I haven't touched the Opensuse part of the script, since I don't have that system on hand for testing. Let me know if you'd like to do anything different.

Kozova1 commented 5 years ago

Was this tested? if yes I'll be happy to merge. thank you for contributing?

magikmw commented 5 years ago

Well, I ran it on my laptop :) I'll setup a clean Fedora in a VM tomorrow and try it out to make sure nothing breaks in a controlled environment. Nonetheless I've ran those commands one by one and that's how it's setup on my laptop.

Kozova1 commented 5 years ago

great, I will merge it into a new branch until either you or I confirm it is working on a clean machine, and then I'll move it to the main one. good job!

magikmw commented 5 years ago

So I did a test, and indeed I messed up the conditional check for OpenSUSE. It's fixed up over at my fork, but since you already merged this pull request I cannot create a new one with all changes or amend this one. Not sure what to do. Commit in question is here: https://github.com/magikmw/parsec-fedora/commit/ed69d2d2af2e9d894680b26ef4a3fea1e70ad3ae

Otherwise, it works!