search

sailfishos-chum / main

Documentation and issue tracker for the SailfishOS:Chum community repository
https://build.merproject.org/project/show/sailfishos:chum
MIT License
26 stars 4 forks source link

Cleanup: Delete ssu directory #79

Closed Olf0 closed 2 years ago

Olf0 commented 2 years ago

… because the two files it contains are not used any more (see https://github.com/sailfishos-chum/main/blob/main/rpm/chum.spec), as these .ini files should not be used for adding these two repositories to the global entries of ssu.ini, which was their purpose.

Olf0 commented 2 years ago

Please check thoroughly that I have not committed a thinko.

rinigus commented 2 years ago

Please check thoroughly ...

OK, that would take time. Don't remember much where those .ini's are used, so would have to refresh it in my memory. I would prefer to postpone it for now and return to it later, sorry.

piggz commented 2 years ago

I think its probably safe. Looks like the use of those files was removed in https://github.com/sailfishos-chum/main/commit/92c20ff377f52c677c1389adff78e99d47d6ce10

Olf0 commented 2 years ago

I think its probably safe. Looks like the use of those files was removed in 92c20ff

@piggz, thank you for looking up this commit as part of your PR #34: Now we have the definite proof for what I concluded.

Now I also remember, that I did the same for Storeman Installer two months ago.

@rinigus, all is fine, "happy merging". :smiley:

rinigus commented 2 years ago

Merged. @Olf0 and @piggz, thank you for working on it!