Hi,

On Solaris, it will be "0Ssf", the 'h' is use only on freebsd.
PARAM_HONOR_NODUMP is an emtpy string on solaris.
PARAM_HONOR_NODUMP is 'h' on freebsd.

Jean-Louis

Chris Hoogendyk wrote:
NOTE: I was just writing a wrapper script for ufsdump on Solaris 9 to incorporate fssnap snapshots and a couple of other things (under 2.5.1p3). I noticed that my script was being called with the options "0SSsf". ufsdump did not seem to mind, and worked without any hangups. I also tried manually doing it with "0Ssf" and with "0SSSsf" and it made no difference.

However, if this is an indication that amanda wasn't accounting for Solaris and ufsdump, and that it would substitute "0Shsf" when patched to work "correctly", then it will break on Solaris, which has no "h" option. I hope somone can look at the code and see that it is handling the various OS differences correctly. The double "SS" and your note below makes it sound like amanda might not be accounting for Solaris properly.

Thanks,


---------------

Chris Hoogendyk

-
  O__  ---- Systems Administrator
 c/ /'_ --- Biology & Geology Departments
(*) \(*) -- 140 Morrill Science Center
~~~~~~~~~~ - University of Massachusetts, Amherst
<[EMAIL PROTECTED]>

---------------
Erdös 4



Jean-Louis Martineau wrote:
Chris,

The attached patch fix the problem.
The problem is the two 'S', one of them should be 'h'.

The patch is already in the CVS.

Jean-Louis

Chris Stenton wrote:
Not sure if this is to do with the FreeBSD port but upgrading to 2.5.1p3 from p2 and the dump estimate has stopped working. Here is the debug output from client sendsize.

sendsize[43898]: time 1.025: calculating for device /dev/da0s1a with ufs
sendsize[43898]: time 1.025: running "/sbin/dump 1SSsf 0 1048576 -
/dev/da0s1a"
sendsize[43898]: time 1.026: running /usr/local/libexec/amanda/killpgrp
sendsize[43898]: time 1.030: dump: tape size must be between 1 and 0
sendsize[43898]: time 1.030: .....


not sure why the extra argument "0" has been added. Any ideas?

Thanks


Chris


------------------------------------------------------------------------

diff -u -r --show-c-function --new-file --exclude-from=/home/martinea/src.orig/amanda.diff --ignore-matching-lines='$Id:' amanda-2.5.1p3.new/client-src/sendsize.c amanda-2.5.1p3.new.krb4/client-src/sendsize.c --- amanda-2.5.1p3.new/client-src/sendsize.c 2006-12-22 09:42:42.000000000 -0500 +++ amanda-2.5.1p3.new.krb4/client-src/sendsize.c 2007-02-08 15:13:57.000000000 -0500
@@ -1190,7 +1190,7 @@ getsize_dump(
 #  endif
     dumpkeys = vstralloc(level_str,
                  PARAM_DUMP_ESTIMATE,
-                 PARAM_DUMP_ESTIMATE,
+                 PARAM_HONOR_NODUMP,
                  "s", "f", NULL);
# ifdef HAVE_DUMP_ESTIMATE

Reply via email to