search

openSUSE / brp-check-suse

Build root policy check scripts
GNU General Public License v2.0
5 stars 21 forks source link

Fix the issue from #41 in a testable way #45

Closed coolo closed 3 years ago

e4t commented 3 years ago

@coolo - I do agree that it looks like there was a typo in the changelog. lib/64 should probably read lib64/ But what was the issue with the fix?

coolo commented 3 years ago

I spent huge efforts to get rid of bash logic and have test cases - for some reason I didn't notice your PR earlier

e4t commented 3 years ago

Ok, understood, makes sense :) For very selfish reasons I prefer the bash logic - it gives me the opportunity to make changes and test them in situ without having to deploy the cavalry to compile stuff.