]> git.sur5r.net Git - bacula/bacula/commitdiff
Fix keepatime bug (bugs database).
authorKern Sibbald <kern@sibbald.com>
Mon, 18 Jul 2005 12:56:31 +0000 (12:56 +0000)
committerKern Sibbald <kern@sibbald.com>
Mon, 18 Jul 2005 12:56:31 +0000 (12:56 +0000)
git-svn-id: https://bacula.svn.sourceforge.net/svnroot/bacula/trunk@2212 91ce42f0-d328-0410-95d8-f526ca767f89

bacula/kernstodo
bacula/kes-1.37
bacula/src/dird/dird.c
bacula/src/dird/scheduler.c
bacula/src/findlib/find_one.c

index 553f37ab926aa00824c94d6ec708f2338c7efb95..bb92b379996bdefab9f3cf22864bdc69aa9e3ba6 100644 (file)
@@ -10,7 +10,6 @@ Version 1.37                Kern (see below)
 Final items for 1.37 before release:
 1. Fix bugs
 -  --without-openssl breaks at least on Solaris.
-
 3. Document all the new features (about half done).
 7. Write a bacula-web document
 8. Take one more try at making DVD writing work.
index 6e7ad8f133fe46b9ae900b8c9bdbb13139bb38b7..b1e9f021fbded8bf4e4e3ffa6de14fd93d812a89 100644 (file)
@@ -5,6 +5,7 @@ General:
 
 Changes to 1.37.31:
 18Jul05
+- Fix keepatime bug (bugs database).
 - Move Python variables from Job to Bacula. They are
   DirName, Version, ConfigFile, and WorkingDir
 - Fix delete of bootstrap to only occur on Bacula created
index fa7c930d1d23960fc4d3c5189c9c4f17eefd25f0..e6f0f57dc40f26a3fbe3fec9c70d937a2f939617 100644 (file)
@@ -238,7 +238,7 @@ int main (int argc, char *argv[])
 
    Dmsg0(200, "wait for next job\n");
    /* Main loop -- call scheduler to get next job to run */
-   while ((jcr = wait_for_next_job(runjob))) {
+   while ( (jcr = wait_for_next_job(runjob)) ) {
       run_job(jcr);                   /* run job */
       free_jcr(jcr);                  /* release jcr */
       if (runjob) {                   /* command line, run a single job? */
index edd9afe853e5ed124ae11da79d0c98fd49e1b5fd..b1361ebfa3813066a408fefe42bb18c235777950 100644 (file)
    Copyright (C) 2000-2005 Kern Sibbald
 
    This program is free software; you can redistribute it and/or
-   modify it under the terms of the GNU General Public License as
-   published by the Free Software Foundation; either version 2 of
-   the License, or (at your option) any later version.
+   modify it under the terms of the GNU General Public License
+   version 2 as amended with additional clauses defined in the
+   file LICENSE in the main source directory.
 
    This program is distributed in the hope that it will be useful,
    but WITHOUT ANY WARRANTY; without even the implied warranty of
-   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
-   General Public License for more details.
-
-   You should have received a copy of the GNU General Public
-   License along with this program; if not, write to the Free
-   Software Foundation, Inc., 59 Temple Place - Suite 330, Boston,
-   MA 02111-1307, USA.
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the 
+   the file LICENSE for additional details.
 
  */
 
@@ -41,11 +36,11 @@ struct job_item {
    JOB *job;
    time_t runtime;
    int Priority;
-   dlink link;                       /* link for list */
+   dlink link;                        /* link for list */
 };
 
 /* List of jobs to be run. They were scheduled in this hour or the next */
-static dlist *jobs_to_run;              /* list of jobs to be run */
+static dlist *jobs_to_run;               /* list of jobs to be run */
 
 /* Time interval in secs to sleep if nothing to be run */
 static int const NEXT_CHECK_SECS = 60;
@@ -62,7 +57,7 @@ static void dump_job(job_item *ji, const char *msg);
 
 /*********************************************************************
  *
- *        Main Bacula Scheduler
+ *         Main Bacula Scheduler
  *
  */
 JCR *wait_for_next_job(char *one_shot_job_to_run)
@@ -79,15 +74,15 @@ JCR *wait_for_next_job(char *one_shot_job_to_run)
       first = false;
       /* Create scheduled jobs list */
       jobs_to_run = New(dlist(next_job, &next_job->link));
-      if (one_shot_job_to_run) {           /* one shot */
-        job = (JOB *)GetResWithName(R_JOB, one_shot_job_to_run);
-        if (!job) {
+      if (one_shot_job_to_run) {            /* one shot */
+         job = (JOB *)GetResWithName(R_JOB, one_shot_job_to_run);
+         if (!job) {
             Emsg1(M_ABORT, 0, _("Job %s not found\n"), one_shot_job_to_run);
-        }
+         }
          Dmsg1(5, "Found one_shot_job_to_run %s\n", one_shot_job_to_run);
-        jcr = new_jcr(sizeof(JCR), dird_free_jcr);
-        set_jcr_defaults(jcr, job);
-        return jcr;
+         jcr = new_jcr(sizeof(JCR), dird_free_jcr);
+         set_jcr_defaults(jcr, job);
+         return jcr;
       }
    }
    /* Wait until we have something in the
@@ -96,12 +91,12 @@ JCR *wait_for_next_job(char *one_shot_job_to_run)
    while (jobs_to_run->empty()) {
       find_runs();
       if (!jobs_to_run->empty()) {
-        break;
+         break;
       }
       bmicrosleep(NEXT_CHECK_SECS, 0); /* recheck once per minute */
    }
 
-#ifdef list_chain
+#ifdef  list_chain
    job_item *je;
    foreach_dlist(je, jobs_to_run) {
       dump_job(je, "Walk queue");
@@ -116,7 +111,7 @@ JCR *wait_for_next_job(char *one_shot_job_to_run)
 
    dump_job(next_job, "Dequeued job");
 
-   if (!next_job) {               /* we really should have something now */
+   if (!next_job) {                /* we really should have something now */
       Emsg0(M_ABORT, 0, _("Scheduler logic error\n"));
    }
 
@@ -125,14 +120,14 @@ JCR *wait_for_next_job(char *one_shot_job_to_run)
       time_t twait;
       now = time(NULL);
       twait = next_job->runtime - now;
-      if (twait <= 0) {              /* time to run it */
-        break;
+      if (twait <= 0) {               /* time to run it */
+         break;
       }
       bmicrosleep(twait, 0);
    }
-   run = next_job->run;              /* pick up needed values */
+   run = next_job->run;               /* pick up needed values */
    job = next_job->job;
-   run->last_run = now;              /* mark as run now */
+   run->last_run = now;               /* mark as run now */
 
    dump_job(next_job, "Run job");
 
@@ -145,7 +140,7 @@ JCR *wait_for_next_job(char *one_shot_job_to_run)
       jcr->JobLevel = run->level;     /* override run level */
    }
    if (run->pool) {
-      jcr->pool = run->pool;         /* override pool */
+      jcr->pool = run->pool;          /* override pool */
    }
    if (run->full_pool) {
       jcr->full_pool = run->full_pool; /* override full pool */
@@ -185,7 +180,7 @@ void term_scheduler()
       job_item *je;
       /* Release all queued job entries to be run */
       foreach_dlist(je, jobs_to_run) {
-        free(je);
+         free(je);
       }
       delete jobs_to_run;
    }
@@ -218,7 +213,7 @@ static void find_runs()
    wday = tm.tm_wday;
    month = tm.tm_mon;
    wom = mday / 7;
-   woy = tm_woy(now);                    /* get week of year */
+   woy = tm_woy(now);                     /* get week of year */
 
    /*
     * Compute values for next hour from now.
@@ -233,78 +228,78 @@ static void find_runs()
    nh_month = tm.tm_mon;
    nh_year  = tm.tm_year;
    nh_wom = nh_mday / 7;
-   nh_woy = tm_woy(now);                    /* get week of year */
+   nh_woy = tm_woy(now);                     /* get week of year */
 
    /* Loop through all jobs */
    LockRes();
    foreach_res(job, R_JOB) {
       sched = job->schedule;
-      if (sched == NULL) {           /* scheduled? */
-        continue;                    /* no, skip this job */
+      if (sched == NULL) {            /* scheduled? */
+         continue;                    /* no, skip this job */
       }
       Dmsg1(1200, "Got job: %s\n", job->hdr.name);
       for (run=sched->run; run; run=run->next) {
-        bool run_now, run_nh;
-        /*
-         * Find runs scheduled between now and the next hour.
-         */
+         bool run_now, run_nh;
+         /*
+          * Find runs scheduled between now and the next hour.
+          */
 #ifdef xxxx
          Dmsg0(000, "\n");
          Dmsg6(000, "run h=%d m=%d md=%d wd=%d wom=%d woy=%d\n",
-           hour, month, mday, wday, wom, woy);
+            hour, month, mday, wday, wom, woy);
          Dmsg6(000, "bitset bsh=%d bsm=%d bsmd=%d bswd=%d bswom=%d bswoy=%d\n",
-           bit_is_set(hour, run->hour),
-           bit_is_set(month, run->month),
-           bit_is_set(mday, run->mday),
-           bit_is_set(wday, run->wday),
-           bit_is_set(wom, run->wom),
-           bit_is_set(woy, run->woy));
+            bit_is_set(hour, run->hour),
+            bit_is_set(month, run->month),
+            bit_is_set(mday, run->mday),
+            bit_is_set(wday, run->wday),
+            bit_is_set(wom, run->wom),
+            bit_is_set(woy, run->woy));
 
          Dmsg6(000, "nh_run h=%d m=%d md=%d wd=%d wom=%d woy=%d\n",
-           nh_hour, nh_month, nh_mday, nh_wday, nh_wom, nh_woy);
+            nh_hour, nh_month, nh_mday, nh_wday, nh_wom, nh_woy);
          Dmsg6(000, "nh_bitset bsh=%d bsm=%d bsmd=%d bswd=%d bswom=%d bswoy=%d\n",
-           bit_is_set(nh_hour, run->hour),
-           bit_is_set(nh_month, run->month),
-           bit_is_set(nh_mday, run->mday),
-           bit_is_set(nh_wday, run->wday),
-           bit_is_set(nh_wom, run->wom),
-           bit_is_set(nh_woy, run->woy));
+            bit_is_set(nh_hour, run->hour),
+            bit_is_set(nh_month, run->month),
+            bit_is_set(nh_mday, run->mday),
+            bit_is_set(nh_wday, run->wday),
+            bit_is_set(nh_wom, run->wom),
+            bit_is_set(nh_woy, run->woy));
 #endif
 
-        run_now = bit_is_set(hour, run->hour) &&
-           bit_is_set(mday, run->mday) &&
-           bit_is_set(wday, run->wday) &&
-           bit_is_set(month, run->month) &&
-           bit_is_set(wom, run->wom) &&
-           bit_is_set(woy, run->woy);
+         run_now = bit_is_set(hour, run->hour) &&
+            bit_is_set(mday, run->mday) &&
+            bit_is_set(wday, run->wday) &&
+            bit_is_set(month, run->month) &&
+            bit_is_set(wom, run->wom) &&
+            bit_is_set(woy, run->woy);
 
-        run_nh = bit_is_set(nh_hour, run->hour) &&
-           bit_is_set(nh_mday, run->mday) &&
-           bit_is_set(nh_wday, run->wday) &&
-           bit_is_set(nh_month, run->month) &&
-           bit_is_set(nh_wom, run->wom) &&
-           bit_is_set(nh_woy, run->woy);
+         run_nh = bit_is_set(nh_hour, run->hour) &&
+            bit_is_set(nh_mday, run->mday) &&
+            bit_is_set(nh_wday, run->wday) &&
+            bit_is_set(nh_month, run->month) &&
+            bit_is_set(nh_wom, run->wom) &&
+            bit_is_set(nh_woy, run->woy);
 
          Dmsg2(1200, "run_now=%d run_nh=%d\n", run_now, run_nh);
 
-        /* find time (time_t) job is to be run */
-        localtime_r(&now, &tm);      /* reset tm structure */
-        tm.tm_min = run->minute;     /* set run minute */
-        tm.tm_sec = 0;               /* zero secs */
-        if (run_now) {
-           runtime = mktime(&tm);
-           add_job(job, run, now, runtime);
-        }
-        /* If job is to be run in the next hour schedule it */
-        if (run_nh) {
-           /* Set correct values */
-           tm.tm_hour = nh_hour;
-           tm.tm_mday = nh_mday + 1; /* fixup because we biased for tests above */
-           tm.tm_mon = nh_month;
-           tm.tm_year = nh_year;
-           runtime = mktime(&tm);
-           add_job(job, run, now, runtime);
-        }
+         /* find time (time_t) job is to be run */
+         localtime_r(&now, &tm);      /* reset tm structure */
+         tm.tm_min = run->minute;     /* set run minute */
+         tm.tm_sec = 0;               /* zero secs */
+         if (run_now) {
+            runtime = mktime(&tm);
+            add_job(job, run, now, runtime);
+         }
+         /* If job is to be run in the next hour schedule it */
+         if (run_nh) {
+            /* Set correct values */
+            tm.tm_hour = nh_hour;
+            tm.tm_mday = nh_mday + 1; /* fixup because we biased for tests above */
+            tm.tm_mon = nh_month;
+            tm.tm_year = nh_year;
+            runtime = mktime(&tm);
+            add_job(job, run, now, runtime);
+         }
       }
    }
    UnlockRes();
@@ -326,7 +321,7 @@ static void add_job(JOB *job, RUN *run, time_t now, time_t runtime)
       bstrftime_nc(dt1, sizeof(dt1), run->last_run);
       bstrftime_nc(dt2, sizeof(dt2), now);
       Dmsg4(000, "Drop: Job=\"%s\" run=%s. last_run=%s. now=%s\n", job->hdr.name,
-           dt, dt1, dt2);
+            dt, dt1, dt2);
       fflush(stdout);
 #endif
       return;
@@ -345,11 +340,11 @@ static void add_job(JOB *job, RUN *run, time_t now, time_t runtime)
    /* Add this job to the wait queue in runtime, priority sorted order */
    foreach_dlist(ji, jobs_to_run) {
       if (ji->runtime > je->runtime ||
-         (ji->runtime == je->runtime && ji->Priority > je->Priority)) {
-        jobs_to_run->insert_before(je, ji);
+          (ji->runtime == je->runtime && ji->Priority > je->Priority)) {
+         jobs_to_run->insert_before(je, ji);
          dump_job(je, "Inserted job");
-        inserted = true;
-        break;
+         inserted = true;
+         break;
       }
    }
    /* If place not found in queue, append it */
index 24c06426b0ee89eb774d29d07be05de814301004..91ac80890be9342bd8ed77de494bb45bc16fd7ef 100755 (executable)
@@ -2,22 +2,21 @@
    Copyright (C) 2000-2005 Kern Sibbald
 
    This program is free software; you can redistribute it and/or
-   modify it under the terms of the GNU General Public License as
-   published by the Free Software Foundation; either version 2 of
-   the License, or (at your option) any later version.
+   modify it under the terms of the GNU General Public License
+   version 2 as amended with additional clauses defined in the
+   file LICENSE in the main source directory.
 
    This program is distributed in the hope that it will be useful,
    but WITHOUT ANY WARRANTY; without even the implied warranty of
-   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
-   General Public License for more details.
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the 
+   the file LICENSE for additional details.
 
-   You should have received a copy of the GNU General Public
-   License along with this program; if not, write to the Free
-   Software Foundation, Inc., 59 Temple Place - Suite 330, Boston,
-   MA 02111-1307, USA.
+ */
+
+/*
 
    This file is based on GNU TAR source code. Except for a few key
-   ideas, it has been rewritten for Bacula.
+   ideas, it has been entirely rewritten for Bacula.
 
       Kern Sibbald, MM
 
@@ -280,6 +279,9 @@ find_one_file(JCR *jcr, FF_PKT *ff_pkt,
       if (ff_pkt->linked) {
          ff_pkt->linked->FileIndex = ff_pkt->FileIndex;
       }
+      if (ff_pkt->flags & FO_KEEPATIME) {
+         utime(fname, &restore_times);
+      }       
       return rtn_stat;
 
 
@@ -350,7 +352,8 @@ find_one_file(JCR *jcr, FF_PKT *ff_pkt,
       ff_pkt->link = link;
       if (ff_pkt->incremental &&
           (ff_pkt->statp.st_mtime < ff_pkt->save_time &&
-           ff_pkt->statp.st_ctime < ff_pkt->save_time)) {
+             ((ff_pkt->flags & FO_MTIMEONLY) ||
+               ff_pkt->statp.st_ctime < ff_pkt->save_time))) {
          /* Incremental option, directory entry not changed */
          ff_pkt->type = FT_DIRNOCHG;
       } else {