Correct the copyright holder named in some files.

For some reason, these files named myself as the copyright holder,
instead of the FSF.

metainfo.its: changed this from a build requirement to a warning

When the build system is not capable to extract the strings from
org.fsf.pspp.metainfo.xml.in file, then this will not be translated.

Fix build error when building --without-gui

Closes bug #59069.

Update version number to 1.4.2.

Release version 1.4.1.

This commit moves the version number *backward*, because we decided
that it was necessary to release 1.4.1 with all of the commits that
were currently on master.

po: Delete temporary files to fix "make distcheck" error.

Makefiles: Use $(PHONY) instead of .PHONY.

Avoids a warning from Automake in Smake:

src/ui/gui/automake.mk:463: warning: .PHONY was already defined in condition TRUE, which includes condition HAVE_GUI ...
Makefile.am:91:   'src/automake.mk' included from here
src/automake.mk:55:   'src/ui/automake.mk' included from here
src/ui/automake.mk:21:   'src/ui/gui/automake.mk' included from here
po/automake.mk:128: ... '.PHONY' previously defined here
Makefile.am:87:   'po/automake.mk' included from here

added appstream as a build requirement

appstream provides the metainfo.its file which is required for
xgettext to extract the translation strings of the
src/ui/gui/org.fsf.pspp.metainfo.xml.in file

The metainfo.its file is located in /usr/share/gettext/its on
debian and on opensuse. It is provided by the packages

debian:     appstream
opensuse:   AppStream-devel

po/automake.mk: changed xgettext glade language to Glade

Revert "po/metainfo.its: New file."

This reverts commit e94442a3b00c1f22952586cc26b7be60617b93e1.

Revert "Work around bug in xgettext 0.19.8.1"

This reverts commit df9ada336f1373c1144b7133a78c97ef5e7afdc3.

Work around bug in xgettext 0.19.8.1

This version crashes on exit when passed a .its file.  So ignore the
exit status.

Use g_ascii_strtoll instead of strtol.

The former is locale sensitive.

po-update rule: use wget rather than rsync

Rsync is considered insecure these days

Remove the --enable-debug configure option.

In practice, it is faster and simpler to make local modifications
to the respective files which debugging rather than reconfiguring
with --enable-debug.   Furthermore this is just another configure
option which confuses users and serves little purpose.

Check that merged po files are well formed.

src/ui/*/automake.mk: Conditionally include the gui automake file

po/automake.mk: Fixed typo in variable names.

This resulted in po/pspp.pot not getting updated when it should have been.
This is a very good demonstration of the reason why carefully desigined
languages should insist on a variable being declared before it is used.

Also make the file depend on Makefile, so that it gets rebuilt when the
PACKAGE VERSION changes.

Deal properly with EOF in the REPL, when built without readline

utitilites/pspp-output.c: Deal properly with plural forms

updated german translation de.po

doc: Describe running the "rc" file at pspp startup.

Bug #54376.
Thanks to Harry Thijssen for reporting this bug.

doc/manual: Remove gitignore file in otherwise empty directory.

I added this by mistake some time ago.  It's not useful.

tests: Add missing host.at tests.

Thanks to Friedrich Beckmann for reporting the problem.

encoding-guesser: Explain UTF-16, UTF-16BE, UTF-16LE distinctions.

u8-istream: Add some more tests for byte-order marks.

make-file: Pass O_BINARY or O_TEXT to open() call in replace_file_start().

It's hard to imagine why this would make a difference on a real system,
because the file descriptor obtained from open() is passed to fdopen(),
which is documented to determine whether the stream is text or binary
based on the mode string.  However, bug #40223 implies that it makes a
difference.  I hope that this commit fixes the problem (if any!).

Thanks to the anonymous submitter of bug #40223.

csv-file-writer: Fix typo in user message.

HOST: Reimplement and add support for TIMELIMIT subcommand.

Fixes bug #25816.
Thanks to John Darrington for reporting this bug.

tests: Add 'slow' tag to more tests.

This reduces the runtime of "make check TESTSUITEFLAGS='-k \!slow -j128'"
from 4.4 to 3.5 seconds on my computer.

spv-light-decoder: Avoid GCC 10.x warning in decode_spvlb_valign().

It seems to be wrong, but GCC can't tell and it's hard to blame it.

value: Suppress uninitialized tmp.s warning in value_resize().

This warning is wrong, but it should not cause a performance problem to
avoid it.  Encountered with GCC 10.x.

Makefile.am: Suppress GCC 10.x warning in gl/careadlinkat.c.

This suppresses an error in gl/careadlinkat.c that otherwise can't be
avoided in GCC 10.x:
  ../../gl/careadlinkat.c: In function ‘careadlinkat’:
  cc1: error: function may return address of local variable

GFUNC_COMPAT_CAST also for dialog-common.c

I missed that one - Thanks!

Work around bug in Automake 1.16.1

moved GFunc cast to macro GFUNC_COMPAT_CAST

I moved the GFunc cast for g_list_foreach to a macro with
some explaining comment. The macro is in helper.h

psppire-spread-sheetmodel.c: modified userdata increment using GPOINTER_TO_INT

po/pl.po - replaced strange character in comment

This results in sed on MacOS to report an error.

dummy.c: Conform to exact prototypes

Apparently certain toolchains complain if the symbols in dummy.c do not
exactly match their declarations.  This should not be a problem unless
the compiler has enabled a lot of warnings, and is treating those warnings
as errors, (which is inadvisable except under very special circumstances).
However, it appears that some people are doing that.

This change ensures that they are identical.

Prefer GTK+ style casts rather than C-style casts.

Replace '(GObjectClass*) class' with 'G_OBJECT_CLASS(class)'

Remove _base_init functions from custom widget definitions.

These served no purpose, and are in Gtk+ only for reasons of
legacy so far as I can make out.   Also use G_TYPE_DEFINE macros
instead of explicit *_get_type function definitions.

INSTALL: Remove reference to --with-gui-tools which no longer exists

switched off unused-paramter warning for lib-essential gui files

I disabled the unused-parameter warning as for the other
gui files. Same as before the introduction of the essential
library.

src/ui/gui/automake.mk: Add missing AM_CFLAGS parameter

Warnings: correct cast for psppire-spread-sheet model

This fixes the compiler warning:
warning: wrong type argument to increment [-Wpointer-arith]

Remove src/ui/gui/memorandum.txt

Almost all of the information in this file is out of date.

Add convenience library libwidgets-essential

This avoids having to compile large numbers of modules twice.

xml-parser-generator - iterate over sorted dict - #58975

I sort the dict before iteration because the normal
iterator over a dict is not reproducible and will generate
different code in each run. This will work but the
build is not reproducible.

Closes: https://savannah.gnu.org/bugs/index.php?58975

src/ui/gui/psppire-conf.c: Use GError instead of strerror.

Apparently the use of strerror was causing problems on certain
platforms.  In this case, since the function being checked provides
a GError argument it makes sense to use that instead.

Closes bug #59034

replace finite with isfinite

finite function is deprecated on MacOS.

Warning: added missing initializer for closure_callback in GSourceFuncs

Warning: "static const" instead of "const static"

Remove warning:
 ‘static’ is not at beginning of declaration

Warning: avoid warning that weekday may be used uninitialized

Warning: adhere to const qualifier for initializations and assignments

This commit avoids the warnings:
warning: initialization discards ‘const’ qualifier from pointer target type
warning: assignment discards ‘const’ qualifier from pointer target type

Warnings: fix missing function prototypes

In most cases I added the static qualifier which was
missing. Some functions are never called so I either
removed them or I put them in #if DEBUG

Warnings: removed fallthrough in case statements

I guess the fallthroughs in the case statements were not intentional.
This avoids the warning:
warning: this statement may fall through

Warnings: added cast for enumaration types

This avoids the warning:
warning: implicit conversion from ‘enum <anonymous>’ to ...

Added tooltip for statistics in dialogs

This removed the warning:

warning: missing initializer for field ‘tooltip’

Warnings: function type cast for g_list_foreach

I added a cast for the functions used in g_list_foreach. Without
the cast, the warning
warning: cast between incompatible function types
is reported.

Warnings: missing initializer for value_tables and function type cast (GObject)

I fixed the warnings:
warning: missing initializer for field ‘value_table’ of ‘GTypeInfo’
warning: cast between incompatible function types
by replacing the handwritten type info code with G_DEFINE_TYPE...
macros (preferred). When this was not possible due to usage of
base init functions I added casts and the initializer.

Warnings: add AM_CFLAGS also for the psppire build

With the new setup the psppire and glade build honours the
AM_CFLAGS setup. This includes the Wextra warnings which
show some more warnings. The two checks which are disabled are

-Wno-sign-compare     : disabled also for core pspp
-Wno-unused-parameter : many callbacks result in unused parameters

Fixed bug which caused a crash when using the Find feature

src/ui/gui/dummy.c: Eliminate warnings when building with gcc-10.2.0

Updated translations from translationproject.org

src/ui/gui/automake.mk: Remove yelp-check rule

psppire-marshaller.c: Include the header file

html.c: Reimplement as HTML5

get_language: New function

Issue error message if failing to change directory

barchart.c: Sort the categories before displaying them.

Partial fix for bug #58968

limit the number of variables in barchart from crosstabs #58968

The barchart was created with more than two variables from
the crosstabs subcommand. I added an assertion and limit
the number of variables to two. This bug resulted in a
regression failure (crash) for test

493: CROSSTABS barchart

on i386 architecture.

see: https://savannah.gnu.org/bugs/?58968

fixed range check in parse_record_placement (32Bit/64Bit issue) #58968

The regression failed for test "lexer crash due to overflow" (403)
on i386 architecture. The reason is that long is 64Bit on amd64 while
it is 32Bit on i386. The code used long as type that can hold larger
values than int which does not work on 32Bit systems.

The change keeps the value as double as long as the range checks are done.

see: https://savannah.gnu.org/bugs/?58968

fixed git distribution check for .gitattributes added in previous commit

added .gitattributes file such that lexer.at shows diff

factor.c: added translation hint

Removed stray newline in acinclude.m4

Reported-by: Michel Boaventura

Fixed error running 'make distcheck'

Fixed some errors in format and orthography of printed strings.

Reported-by: Anonymous https://lists.gnu.org/archive/html/bug-gnu-pspp/2020-08/msg00004.html

pspp-convert.1 - fixed a syntax typo

The debian build checks reported a bug in the man page:

troff: <standard input>:129: '\-' is not allowed in an escape name

The man page can be checked with:

LC_ALL=en_US.UTF-8 MANROFFSEQ='' MANWIDTH=80 \
man --warnings -E UTF-8 -l -Tutf8 -Z ./pspp-convert.1 >/dev/null

Revive psppire's glade compatibility

PsppireSelector: Don't attempt to unref a null pointer

Update GtkBuilder files to mention gtk+ version 3.22

src/language/stats/chart-category.h: Add #include directive

PsppireVarView: Use g_realloc instead of xrealloc

src/ui/gui/automake.mk: Sort the manifest

Rename PsppireButtonbox to PsppireButtonBox

This is more consistent with other names.

src/ui/gui/psppire-conf.c: On error, give a reason for failure to create file

Avoid warnings about ! being passed a non-boolean

PsppireImportAssistant: Proceed to next page when a file is double clicked

Remove/Replace some obsolete gnulib modules

Smake: create build-aux/config.rpath

Update to more recent GNULIB

Bump version to 1.5.0

Release PSPP version 1.4.0.

sys-file-reader: get_text_token did not set delimiter - bug #58943

parse_mrsets used loop local variables which were not initialized.
The regression worked with low compiler optimization levels because
the value for "delimiter" was kept over the loop iterations.
get_text_token did not set the delimiter value when only delimiters
are left in the string. I set the delimiter value to the last value
in the string in that case. I initialized the variables to see the
bug also in -O0.

Closes: https://savannah.gnu.org/bugs/?58943

render-test: Initialize i18n to make tests pass on mingw.

On mingw, filename recoding requires i18n to be initialized.

tests: Tolerate 3-digit exponents in output for chart tick format.

This makes the test pass on mingw.

tests: Ignore Pango warnings when generating charts.

On mingw, Pango complains about font names.  It's harmless.  I don't know
a way to suppress the warning.

placement-parser: Properly handle LONG_MAX != INT_MAX.

Also, delete spurious null bytes from the related test.  The null
bytes spurred a test failure on mingw although I am not sure why.