Sorry, please correct: "As for packaging, the update command won't be invoked if DESTDIR is neither defined nor set.:
should be "As for packaging, the update command won't be invoked if DESTDIR is defined and not an empty string. 2015-11-05 16:14 GMT+09:00 Kuriyama Kazunobu <[email protected]>: > >> Is there a reason this step shouldn't be done by the makefile? > > > > The makefile has no idea whether you're installing into a place that the > > running system actually cares about. You may be installing into a > > temporary directory in order to package up the results for distribution, > > and that shouldn't cause the mimeinfo.cache file of the running system > > to get updated with an irrelevant file. > > My patch is written in such a way that the cache file is created/updated > only when the desktop files are installed, and the update command will be > invoked against just the directory where they are installed. IOW, the > makefile has already known the place. > > As for packaging, the update command won't be invoked if DESTDIR is > neither defined nor set. > > I think I wrote my patch with much care :-) > > Please read my patch and give comments on that. > > Best, > Kazunobu Kuriyama > > 2015-11-05 12:32 GMT+09:00 Tony Mechelynck <[email protected]>: > >> The published .desktop files are nice as far as they go but they miss >> a number of MIME types I may want to edit with Vim. Here is the >> mimeinfo.cache generated manually by running update-desktop-database >> as Kuriyama Kazuniobu said earlier in this thread: >> >> [MIME Cache] >> application/x-shellscript=gvim.desktop;vim.desktop; >> text/english=gvim.desktop;vim.desktop; >> text/plain=gvim.desktop;vim.desktop; >> text/x-c=gvim.desktop;vim.desktop; >> text/x-c++=gvim.desktop;vim.desktop; >> text/x-c++hdr=gvim.desktop;vim.desktop; >> text/x-c++src=gvim.desktop;vim.desktop; >> text/x-chdr=gvim.desktop;vim.desktop; >> text/x-csrc=gvim.desktop;vim.desktop; >> text/x-java=gvim.desktop;vim.desktop; >> text/x-makefile=gvim.desktop;vim.desktop; >> text/x-moc=gvim.desktop;vim.desktop; >> text/x-pascal=gvim.desktop;vim.desktop; >> text/x-tcl=gvim.desktop;vim.desktop; >> text/x-tex=gvim.desktop;vim.desktop; >> >> And I miss: >> >> text/css >> text/html >> text/xml >> application/xhtml+xml >> message/rfc822 >> >> plus whatever (if different from text/plain) is used for RDF >> (application/rdf+xml ?), Javascript, Vim scripts, XUL >> (application/xul+xml ?), diff patches, and certainly some others I >> didn't think of today. >> >> >> Best regards, >> Tony. >> >> -- >> -- >> You received this message from the "vim_dev" maillist. >> Do not top-post! Type your reply below the text you are replying to. >> For more information, visit http://www.vim.org/maillist.php >> >> --- >> You received this message because you are subscribed to the Google Groups >> "vim_dev" group. >> To unsubscribe from this group and stop receiving emails from it, send an >> email to [email protected]. >> For more options, visit https://groups.google.com/d/optout. >> > > -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply below the text you are replying to. For more information, visit http://www.vim.org/maillist.php --- You received this message because you are subscribed to the Google Groups "vim_dev" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. For more options, visit https://groups.google.com/d/optout.
