Re: [Refdb-devel] refdbc -C addref & stdin

[Michael S. <sm...@xm...>]

Hi Markus,

You wrote:

> Michael Smith writes:
>  > Will it break behavior on other platforms if I just include the -f stdin
>  > option by default in the set of args that Emacs passes to
>  > refdbc -C addref ? That is, is -f stdin just ignored on platforms that
>  > can already auto detect data on stdin?
>  > 
> 
> It is safe to use this switch on all platforms. The switch isn't
> actually ignored on the other platforms but it sets a variable to 1
> which has the value 1 anyway if data are available on stdin.

OK. I've added it as the default value for variable that sets additional
options for the commands that accept stdin (addref & updateref, for now).

>  > Anyway, when I actually try it on Cygwin, it doesn't seem to work -
>  > 
>  >   $ refdbc -d foo -C addref -f stdin < /tmp/data/alltypes.ris 
>  >   Could not set terminal attributes
>  >   0 dataset(s) added, 0 skipped, 0 failed
>  >   0 dataset(s) sent.
>  > 
> 
> I'm afraid this is my fault. I've just checked that for reasons
> entirely unclear to me addref does not yet support the -f stdin
> switch. It will fail on Cygwin no matter how hard you'll try. The
> appended patch fixes the problem. It is against a current CVS version
> so expect some hunks to fail if you try to patch 0.9.3.

Thanks -- I applied this patch to source in my CVS sandbox and built
from there. (Had a couple of problems building -- will send a separate
message about those.) So, I've now set up to test with binaries built
from the latest source.

  --Mike