Re: [Rpm-maint] [PATCH] Add debugsource recommends to debuginfo packages.

2017-09-21 Thread Panu Matilainen
On 09/21/2017 05:55 PM, Mark Wielaard wrote: On Thu, 2017-09-21 at 10:41 +0300, Panu Matilainen wrote: Moving an entire function while also changing it is a bit of a no- no, because it's makes it hard to see what actually changed. Moving things around also obfuscates git history, which is why I

Re: [Rpm-maint] [PATCH] Add debugsource recommends to debuginfo packages.

2017-09-21 Thread Mark Wielaard
On Thu, 2017-09-21 at 10:41 +0300, Panu Matilainen wrote: > Moving an entire function while also changing it is a bit of a no- > no, because it's makes it hard to see what actually changed. Moving > things around also obfuscates git history, which is why I prefer > adding a prototype to the top of

Re: [Rpm-maint] [PATCH] Add debugsource recommends to debuginfo packages.

2017-09-21 Thread Panu Matilainen
On 09/18/2017 06:56 PM, Mark Wielaard wrote: Debuginfo packages are useful without debugsource files. But it is often useful to also have the debugsourc files. So make debuginfo packages that don't contain sources recommend the debugsource package (or the main debuginfo one if the sources are

[Rpm-maint] [PATCH] Add debugsource recommends to debuginfo packages.

2017-09-18 Thread Mark Wielaard
Debuginfo packages are useful without debugsource files. But it is often useful to also have the debugsourc files. So make debuginfo packages that don't contain sources recommend the debugsource package (or the main debuginfo one if the sources are not in a separate debugsource package). Rename