[Geeqie-devel] patch to remove bashisms

2009-07-24 Thread Greg Troxel
Geeqie has several bashisms which seem unnecessary. I don't use all these features so perhaps one or more of the scripts does actually need bash. If it does, it should be found via configure and listed as an explicit dependency. My experience has been that 95% of scripts that say /bin/bash

Re: [Geeqie-devel] [PATCH] Multithreaded image loading

2009-07-24 Thread Vladimir Nadvornik
Hi Bartosz, Thanks for the patch. On út 7. července 2009, Bartosz Zaborowski wrote: Hi all, I've rewritten some code of image loader to get real performance gain from threads on SMP machine. Threads support exist in geeqie since around rev 1100, but in fact, there are up to 2 simultaneously

Re: [Geeqie-devel] [PATCH] Multithreaded image loading

2009-07-24 Thread Klaus Ethgen
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, Am Fr den 24. Jul 2009 um 23:08 schrieb Vladimir Nadvornik: I'd rather wait with adding the patch after 1.0, since it is a non-trivial change. Wouldn't it be better to add such patches to a branch and merge it later? At least with git (its