Martin v. Löwis wrote: > Georg Brandl wrote: >> In case we add a Python .desktop file (as proposed in patch #1353344), >> we'll need some PNGs in /usr/share/icons. A patch for Makefile.pre.in >> is attached. > > Independent of whether this should be done at all, I have a comment on > the patch. Instead of invoking sed, I would use autoconf machinery here > >> +Exec=python_VER_ %f > > This would become > > [EMAIL PROTECTED]@@EXEEXT@ %f > >> + sed 's/_VER_/$(VERSION)$(EXE)/' $(srcdir)/Misc/python.desktop.in > >> $(srcdir)/Misc/python.desktop > > This could go away, and configure should grow > > AC_CONFIG_FILES(Makefile.pre Modules/Setup.config Misc/python.desktop)
Ah, thanks. I'm not very fit with autoconf magic. Georg _______________________________________________ Python-Dev mailing list Python-Dev@python.org http://mail.python.org/mailman/listinfo/python-dev Unsubscribe: http://mail.python.org/mailman/options/python-dev/archive%40mail-archive.com
