On 15/09/16 08:53, Gianfranco Costamagna wrote:
Hi,I know this approach works, but is it the "right" way? Does it matter?your way is the best one if you also add a patch to automatically install the file inside the upstream build system. Otherwise you can avoid a patch and add the desktop file inside the debian directory, and ask upstream to integrate it. G.
In my personal experience, upstream usually prefers to be asked about where to put the .desktop file. So writing a patch that blindly places it in the root of the source tree may be rejected as is. Instead you should rather first embed it in the debian/ folder and discuss with upstream where to put it in a subsequent PR. Ghis