search

Katello / katello-packaging

[DEPRECATED] Packaging for Katello
7 stars 33 forks source link

FIxes #21341 - check for /usr/sbin path of the tools when loading #558

Closed evgeni closed 6 years ago

theforeman-bot commented 6 years ago

Issues: #21341

theforeman-bot commented 6 years ago

Issues: #21341

johnpmitsch commented 6 years ago

@evgeni I didn't realize you had already opened this (much appreciated) when I opened up this https://github.com/Katello/katello-packaging/pull/559

johnpmitsch commented 6 years ago

@evgeni why not just check if the file path includes sbin? That seems unique enough to test if its production or not. The only edge case is having some tool with a substring of "sbin", which seems unlikely and even if it happens, we are still defaulting to production working (and dev setup breaking)

evgeni commented 6 years ago

@johnpmitsch yeah, either works. closing as yours is merged.