I'm sorry, I seem to be doing most of the patch reviewing lately, and I've been really busy. I have the patch on disk here, and plan to look at it in the next week or so (along with half a dozen others). From your description, it sounds like a proper fix.
allan On Mon, Sep 13, 2010 at 10:40 AM, Reinhold Kainhofer <reinhold at kainhofer.com> wrote: > Am Freitag, 3. September 2010, 16:57:14 schrieb Reinhold Kainhofer: >> Attached is a patch for the current sane-backends git, which fixes several >> problems with the handling of the SANE_CONFIG_DIR environment variable. > > Is there anyone here you can take a look at the patch, concerning correctness > etc.? > > BTW, the main problem (wrong pointers to the list of config dirs) does not > appear when you set SANE_CONFIG_DIR only to one directory, but only if you set > it to two or more directories. Here's the debug output of current git master > in that case (notice the path before dll.aliases!): > > reinhold at curie:~/Software/sane-backends$ export \ > SANE_CONFIG_DIR=/home/reinhold/temp/sane.d:/etc/sane.d: > reinhold at curie:~/Software/sane-backends$ scanimage -L > [sanei_debug] Setting debug level of sanei_config to 127. > [sanei_config] sanei_config_open: attempting to open > `/home/reinhold/temp/sane.d/dll.conf' > [sanei_config] sanei_config_open: using file > `/home/reinhold/temp/sane.d/dll.conf' > [sanei_config] sanei_config_open: attempting to open `???/dll.aliases' > [sanei_config] sanei_config_open: could not find config file `dll.aliases' > > > The problem that the dll backend always looks only in /etc/sane.d/dll.d/, even > when a different SANE_CONFIG_DIR is given, is of course present in all cases. > > Cheers, > Reinhold > -- > ------------------------------------------------------------------ > Reinhold Kainhofer, reinhold at kainhofer.com, http://reinhold.kainhofer.com/ > ?* Financial & Actuarial Math., Vienna Univ. of Technology, Austria > ?* http://www.fam.tuwien.ac.at/, DVR: 0005886 > ?* LilyPond, Music typesetting, http://www.lilypond.org > > -- > sane-devel mailing list: sane-devel at lists.alioth.debian.org > http://lists.alioth.debian.org/mailman/listinfo/sane-devel > Unsubscribe: Send mail with subject "unsubscribe your_password" > ? ? ? ? ? ? to sane-devel-request at lists.alioth.debian.org -- "The truth is an offense, but not a sin"
