Overview

Request 843137 accepted

- Name the installed binary `delta`, as upstream calls it, as
renaming causes various breakages (e.g. bash completions) and
inconsitencies with upstream documentation and usage.
Add Conflict with sccs, which also provides a binary called
`delta` instead.

Loading...

Martin Sirringhaus's avatar
author source maintainer target maintainer

@jubalh I'd like to have a second pair of eyes on this one, could you please have a look?


Michael Vetter's avatar

Sure! I think "Provides: delta" needs to be removed. I don't think thats needed/right. The conflicts is correct. Maybe it would be good to notify upstream about this and ask them to rename the binary? Maybe "gdelta"?


Martin Sirringhaus's avatar
author source maintainer target maintainer

Thanks. A colleague suggested Provides: delta to be able to do zypper in delta as well. Not sure about it myself. Upstream is aware of this name clash, and are not open for renaming, apparently, but likes to go this route (https://github.com/dandavison/delta/issues/26)


Michael Vetter's avatar

Thanks! I commented there.


Martin Sirringhaus's avatar
author source maintainer target maintainer

@jubalh So, judging from the upstreams response, we merge this SR?


Michael Vetter's avatar

Yep! I'm still not sure about the provides though but don't strong opinion (since I'm not sure :))


Martin Sirringhaus's avatar
author source maintainer target maintainer

I'll just remove it.


Request History
Martin Sirringhaus's avatar

MSirringhaus created request

- Name the installed binary `delta`, as upstream calls it, as
renaming causes various breakages (e.g. bash completions) and
inconsitencies with upstream documentation and usage.
Add Conflict with sccs, which also provides a binary called
`delta` instead.


Martin Sirringhaus's avatar

MSirringhaus accepted request

openSUSE Build Service is sponsored by