[freeside] Small Patch

[Kenny Elliott]

> Hmm, I'm not certain about this patch.  *All* of the scp commands in
> svc_acct.export, not just the ones you patched, copy a file
> to a file.

Yes this is quite possible. I was just trying to track down this one
problem.

> This seems to work with both "classic" ssh1 and OpenSSH.
> You are using
> the commercial SSH 2?  Is this behavior documented?  What's
> the error
> message you get when it fails?

Yes, I'm using the Commercial version of SSH 2.

The only mention in the scp2 manpage is..

  This option specifies, that scp2 should copy direc­tories
recursively. Does not follow symbolic links.

The error is..

warning: Destination file is not a directory.
warning: Exiting.


> I think I'd be more comfortable with something that chose
> the appopriate
> flags for your each of scp.  In this case you wouldn't need
> to check the
> version as such; you could probably stat the source file,
> and only turn on
> the scp `-r' if it's a directory (in perl, if `-d
> $filename' is true).

Yes. That would be best. Perhaps though there should be a
configuration option instead of checking all of the possible locations
of ssh2.

> The `-q' flag is a separate issue.
>
> The ssh 1.2.26 scp manpage says:
>
>        -q    Turn off statistics display.
>
> The openssh 1.2.3 scp manpages says:
>
> 	-q      Disables the progress meter.
>
> What does the commercial SSH 2 scp manpage say about -q?
>
> (Hmm, I wonder if -q turns off error reporting in all versions)

It does not mention the -q switch however running scp with -q does run
it in "quite" mode. It return no statistics or errors with the -q
switch.