Hi Kern, thanks for the new release!
While looking through the changes since v9.4 I noticed a few issues: These files say they have non-free licenses: bacula/src/plugins/fd/docker/baculatar/Dockerfile regress/scripts/regress-utils.sh These files have "Copyright (C) (...) Bacula Systems SA" and don't mention the BSD 2-Clause license like the other files there: bacula/platforms/rpms/suse/bacula-aligned.spec.in bacula/platforms/rpms/redhat/bacula-cloud.spec.in bacula/platforms/rpms/redhat/bacula-aligned.spec.in The file "bacula/scripts/manual_prune.pl" contradicts itself: the header says (c) Kern Sibbald and BSD 2-Clause, but the included man page says (c) Bacula Systems SA and "only for personal use". The new makefiles don't specify "$(CFLAGS)" in the target for .c.lo. Debian's QA tools are unhappy about that, I attached a patch. Some spelling errors, found by Debian QA tools, I attached another patch. Especially fixing the license issues would help us preparing Bacula 9.6 for Debian. Best regards, Carsten
diff --git a/bacula/src/plugins/dir/Makefile.in b/bacula/src/plugins/dir/Makefile.in index 6ba512138d..c20b204769 100644 --- a/bacula/src/plugins/dir/Makefile.in +++ b/bacula/src/plugins/dir/Makefile.in @@ -16,7 +16,7 @@ LIBDIR=../../lib .SUFFIXES: .c .o .lo .c.lo: - $(LIBTOOL_COMPILE) $(CXX) $(DEFS) $(DEBUG) $(CPPFLAGS) -I${SRCDIR} -I${DIRDIR} -DTEST_PROGRAM -c $< + $(LIBTOOL_COMPILE) $(CXX) $(DEFS) $(DEBUG) $(CPPFLAGS) $(CFLAGS) -I${SRCDIR} -I${DIRDIR} -DTEST_PROGRAM -c $< all: example-plugin-dir.la diff --git a/bacula/src/plugins/fd/Makefile.in b/bacula/src/plugins/fd/Makefile.in index 8ea8f61447..b1098565d2 100644 --- a/bacula/src/plugins/fd/Makefile.in +++ b/bacula/src/plugins/fd/Makefile.in @@ -20,7 +20,7 @@ thisdir = src/plugins/fd .SUFFIXES: .c .lo .c.lo: - $(LIBTOOL_COMPILE) $(CXX) $(DEFS) $(DEBUG) $(CPPFLAGS) -I${SRCDIR} -I${FDDIR} -DTEST_PROGRAM -c $< + $(LIBTOOL_COMPILE) $(CXX) $(DEFS) $(DEBUG) $(CPPFLAGS) $(CFLAGS) -I${SRCDIR} -I${FDDIR} -DTEST_PROGRAM -c $< all: bpipe-fd.la test-plugin-fd.la test-deltaseq-fd.la pluglib.lo diff --git a/bacula/src/plugins/fd/docker/Makefile.in b/bacula/src/plugins/fd/docker/Makefile.in index 6774e7d9d6..0370f08eb0 100644 --- a/bacula/src/plugins/fd/docker/Makefile.in +++ b/bacula/src/plugins/fd/docker/Makefile.in @@ -28,7 +28,7 @@ all: docker-fd.la .c.lo: @echo "Compiling $< ..." - $(NO_ECHO)$(LIBTOOL_COMPILE) $(CXX) $(DEFS) $(DEBUG) $(CPPFLAGS) -I${SRCDIR} -I${FDDIR} -I${FDPLUGDIR} -DWORKDIR=\"$(DESTDIR)$(working_dir)\" -c $< + $(NO_ECHO)$(LIBTOOL_COMPILE) $(CXX) $(DEFS) $(DEBUG) $(CPPFLAGS) $(CFLAGS) -I${SRCDIR} -I${FDDIR} -I${FDPLUGDIR} -DWORKDIR=\"$(DESTDIR)$(working_dir)\" -c $< %.lo: %.c %.h Makefile @echo "Compiling $< ..." diff --git a/bacula/src/plugins/sd/Makefile.in b/bacula/src/plugins/sd/Makefile.in index ef5797d839..09187d9281 100644 --- a/bacula/src/plugins/sd/Makefile.in +++ b/bacula/src/plugins/sd/Makefile.in @@ -16,7 +16,7 @@ LIBDIR=../../lib .SUFFIXES: .c .lo .c.lo: - $(LIBTOOL_COMPILE) $(CXX) $(DEFS) $(DEBUG) $(CPPFLAGS) -I${SRCDIR} -I${SDDIR} -DTEST_PROGRAM -c $< + $(LIBTOOL_COMPILE) $(CXX) $(DEFS) $(DEBUG) $(CPPFLAGS) $(CFLAGS) -I${SRCDIR} -I${SDDIR} -DTEST_PROGRAM -c $< all: example-plugin-sd.la
diff --git a/bacula/po/es_AR.po b/bacula/po/es_AR.po index 4ca7b06f16..64fae79aac 100644 --- a/bacula/po/es_AR.po +++ b/bacula/po/es_AR.po @@ -8658,7 +8658,7 @@ msgstr "" #: src/stored/append.c:265 src/stored/btape.c:1976 #, c-format -msgid "Error writting end session label. ERR=%s\n" +msgid "Error writing end session label. ERR=%s\n" msgstr "" #: src/stored/append.c:278 @@ -9783,7 +9783,7 @@ msgstr "" #: src/stored/btape.c:600 msgid "" -"Test writting larger and larger records.\n" +"Test writing larger and larger records.\n" "This is a torture test for records.\n" "I am going to write\n" "larger and larger records. It will stop when the record size\n" diff --git a/bacula/po/fi.po b/bacula/po/fi.po index 62887e934c..48458015f8 100644 --- a/bacula/po/fi.po +++ b/bacula/po/fi.po @@ -9712,7 +9712,7 @@ msgstr "" #: src/stored/append.c:292 src/stored/btape.c:2013 #, c-format -msgid "Error writting end session label. ERR=%s\n" +msgid "Error writing end session label. ERR=%s\n" msgstr "" #: src/stored/append.c:303 src/stored/mac.c:124 src/stored/mac.c:220 @@ -10907,7 +10907,7 @@ msgstr "" #: src/stored/btape.c:632 msgid "" -"Test writting larger and larger records.\n" +"Test writing larger and larger records.\n" "This is a torture test for records.\n" "I am going to write\n" "larger and larger records. It will stop when the record size\n" diff --git a/bacula/po/it.po b/bacula/po/it.po index a49ad035ca..1de1daf926 100644 --- a/bacula/po/it.po +++ b/bacula/po/it.po @@ -9316,7 +9316,7 @@ msgstr "" #: src/stored/append.c:292 src/stored/btape.c:2013 #, c-format -msgid "Error writting end session label. ERR=%s\n" +msgid "Error writing end session label. ERR=%s\n" msgstr "" #: src/stored/append.c:303 src/stored/mac.c:124 src/stored/mac.c:220 @@ -10511,7 +10511,7 @@ msgstr "" #: src/stored/btape.c:632 msgid "" -"Test writting larger and larger records.\n" +"Test writing larger and larger records.\n" "This is a torture test for records.\n" "I am going to write\n" "larger and larger records. It will stop when the record size\n" diff --git a/bacula/src/lib/bstat.c b/bacula/src/lib/bstat.c index c22e17b2ab..44143b405a 100644 --- a/bacula/src/lib/bstat.c +++ b/bacula/src/lib/bstat.c @@ -194,7 +194,7 @@ const char *bstatmetric::metric_unit_str() case METRIC_UNIT_SEC: return "Seconds"; case METRIC_UNIT_MSEC: - return "miliSeconds"; + return "milliseconds"; case METRIC_UNIT_NUMBER: return "Number"; case METRIC_UNIT_PERCENT: diff --git a/bacula/src/stored/dev.c b/bacula/src/stored/dev.c index 6da1fc5b41..f579c77839 100644 --- a/bacula/src/stored/dev.c +++ b/bacula/src/stored/dev.c @@ -1126,7 +1126,7 @@ void DEVICE::register_metrics(bstatcollect *collector) Mmsg(met, "bacula.storage.%s.device.%s.writetime", me->hdr.name, name()); devstatmetrics.bacula_storage_device_writetime = devstatcollector->registration(met.c_str(), METRIC_INT, METRIC_UNIT_MSEC, - (char*)"Time spent writting to device."); + (char*)"Time spent writing to device."); /* total and free space metrics registration */ Mmsg(met, "bacula.storage.%s.device.%s.freespace", me->hdr.name, name()); devstatmetrics.bacula_storage_device_freespace = diff --git a/gui/baculum/framework/pradosoft/prado/framework/I18N/core/MessageSource.php b/gui/baculum/framework/pradosoft/prado/framework/I18N/core/MessageSource.php index 96ed45216b..b5df56c84e 100644 --- a/gui/baculum/framework/pradosoft/prado/framework/I18N/core/MessageSource.php +++ b/gui/baculum/framework/pradosoft/prado/framework/I18N/core/MessageSource.php @@ -48,7 +48,7 @@ require_once(__DIR__ . '/MessageCache.php'); * $source = MessageSource::factory('MySource', $resource, $classfile); * </code> * - * If you are writting your own message sources, pay attention to the + * If you are writing your own message sources, pay attention to the * loadCatalogue method. It details how the resources are loaded and cached. * See also the existing message source types as examples. * diff --git a/gui/baculum/protected/Web/JavaScript/misc.js b/gui/baculum/protected/Web/JavaScript/misc.js index 020dff3ce3..ba2dc8a49f 100644 --- a/gui/baculum/protected/Web/JavaScript/misc.js +++ b/gui/baculum/protected/Web/JavaScript/misc.js @@ -214,7 +214,7 @@ function set_formatters() { } var Cookies = { - default_exipration_time: 31536000000, // 1 year in miliseconds + default_exipration_time: 31536000000, // 1 year in milliseconds set_cookie: function(name, value, expiration) { var date = new Date(); date.setTime(date.getTime() + this.default_exipration_time);
_______________________________________________ Bacula-devel mailing list Bacula-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/bacula-devel