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

Reply via email to