sys-file-reader: Don't warn if compression bias field is 0.

Some software writes a value of 0 to the compression bias field.  We
expect that this field has a value of 100 and warn if we see any other
value, but there is no real need to do so (because it is very likely
that we can read the file correctly in any case).  So suppress the
warning if we see a value of 0 here, because it bothers users.

Thanks to Tony Reardon <tony@sir.com.au> for reporting this
unnecessary warning.

por-file-reader: Tolerate short lines when detecting portable files.

When the portable file reader reads a file, it assumes that a line
shorter than 80 bytes should actually be padded out on the right with
spaces, because this is a fairly common problem in practice, perhaps
due to text editors or other software that drops spaces at the end of
a line.

However, pfm_detect, the function that is supposed to detect whether
a given file is an SPSS portable file, did not apply this heuristic to
the data that it read at the beginning of the file, and thus files in
which the first few lines were truncated this way were not detected
properly as portable files.

This commit fixes the problem by making pfm_detect a little bit
smarter.  It would probably be better to actually unify the
file-reading logic, instead of implementing it in two separate places,
but this appears to work adequately too.

Thanks to Tony Reardon <tony@sir.com.au> for reporting the problem
and supplying a file that demonstrated it.

data-parser: Treat " ," the same as ",".

The data parser (used e.g. by DATA LIST) was unintentionally treating
soft separators (e.g. white space) followed by a hard separator (e.g.
a comma) as an empty field, but in fact it should be treated as just a
single separator.  This fixes it and adds a test.

Thanks to pascal barbedor <pbarbedor@gmail.com> for reporting the problem.

Ensure that NUMERIC's format (if any) is a valid output format.

Thanks to Pascal Barbedor for reporting this problem.

Change g_print to g_warning

Use the replacement rename function from gnulib.

Use the gnulib replacement rename function, to
overcome problems on certain systems saving a
file when that filename already exists. Closes bug #25795

Allow users to set the precision of output statistics.

Instead of hard coding the width and decimals of output
numbers, respect the default format in most instances.
Counts are normally displayed with the format of the weight
variable, if any.  Closes patch #6785

Fixed bug where piecharts with many segments crashed.

Make sure that we don't index beyond the bounds of the
data_colour array.

Fixed bug writing portable files.

Don't append slash to sysmis values when writing portable
files.  Fixes bug #26034

Added #include <config.h> to files as appropriate.

Thanks to Michel Boaventura for reporting this problem.

Fix corrupted .sav files on w32 builds

Added the O_BINARY flag to the mode of files
created with open, so that lines aren't appended
with extraneous characters, thus corrupting .sav
files.

Correct typo in BINOMIAL section of user manual.

Thanks to Erik Frebold for pointing out this problem.
Closes bug #25892

Fixed a bug reading empty strings from Postgres databases.

Fixed a bug in the Postgresql import feature, where a segfault
would occur when a reading a database which contained string
fields with only null, or zero length values.

Thanks to Joerg Schmengle for reporting this.

Fix bug 25677

Prevent data sheet from traversing into an invalid column.

Thanks to James R. Van Zandt for reporting this problem.

Relocate path names when searching for files.

When searching for configuration files etc. in a
search path, relocate the filenames before testing
for their existence.  Closes bug #25508

Silently ignore some inrecognised subtypes.

When reading system files, record subtypes of which the developers are
aware but which pspp doesn't yet support are silently ignored.  Subtypes
which the developers are unaware provoke a message requesting the user
to send us a sample.

Added a set of per binary LDFLAGS parameters.

Added two optional variables PSPP_LDFLAGS and PSPPIRE_LDFLAGS for the convenience of package creators who need to be able to pass flags to one program but not the other.

Use gnulib 'close' module, for compatibility with newest Gnulib.

Otherwise at link we get an undefined reference to symbol
_close_used_without_requesting_gnulib_module_close.

Thanks to Michel Boaventura for reporting the problem.

Move definition of OUTPUT_FILE_NAME to before first use.

Problem manifested itself when building with cygwin.
Reported by unknown-1

Marked CORRELATIONS and PEARSON CORRELATIONS as unimplemented.

These commands were defined in command.def, but only the parser
was actually implemented; they didn't actually do anything.
Marking them as unimplemented until they're finished.

Added (source) configurable default output directory.

Some systems expect files to be dumped in the home directory
instead of the current directory.  This change makes that
easier.

Fix crash in tabulating long-string variables in CROSSTABS.

Fixes bug #24557.

Do not assume that canonicalize_file_name() always returns non-null.

Fixes bug #24569.

Make src/libpspp/version.c depend on Makefile.

This ensures that re-running "configure" with different settings causes
version.c to be rebuilt.

Thanks to Ivy Foster <ivy.foster@gmail.com> for reporting the problem.

Update NEWS for 0.6.1.

Fixed problems when building with --enable-relocatable

Added the INSTALLDIR macro to tests/dissect-sysfile.
Relocated the pathname of the splash screen graphic.

Use Gnulib module "regex", to fix linking on Mingw.

The GUI code uses POSIX regular expressions, which Mingw has in a separate
library (-lregex).  We could just search for that library, but by including
our own implementation (via Gnulib) we also work around known bugs in
system regular expression implementations.

Update NEWS for 0.6.1-rc2.

Add ".sav" or ".por" suffix to filename when saving with Save_As

Closes bug #23137

(This is a cross-port of commit 169887f6d6d7bf5c10d5b05786aa3fd252040c93
from the master branch, originally authored and committed by John
Darrington.)

Only use -fgnu89-inline if the compiler actually supports it.

Jason Stover pointed out that we were using -fgnu89-inline even when the
compiler didn't support it.  Obviously that's not good.

Also warn about missing prerequisites as we encounter them (bug #24445).

It can happen that configure bails out in the middle due to some issue
that PSPP cannot directly control; for example, AC_CHECK_SIZEOF exits
unconditionally if it fails.  If this happens then, before, we would fail
to warn about missing required prerequisites that could well have been
the actual cause of the failure, because we only warn about these at
the end of the run.

Now, we warn about missing prerequisites both at time of detection and
in a summary at the end, to head off this problem.

Thanks to Jim Orr for allowing me to notice the problem.

Add note to INSTALL explaining how to use libraries from /usr/local.

Several users have had trouble building PSPP with self-built GSL, etc.
that were installed to /usr/local.  Explain how to do it here to be
more friendly to Unix novices.

Thanks to Jim Orr for making me realize that this was important.

Add pspp-dev.dvi to CLEANFILES.

Automake does not seem to clean this file automatically.  (Is this an
Automake bug?)

Append $(EXEEXT_FOR_BUILD) to output file name when building q2c.

Thanks to "Michel Boaventura" <michel.boaventura@gmail.com> for reporting
this problem and to John Darrington for assisting with the solution.

Update NEWS for 0.6.1 release.

Don't append % to count totals.

There was an erroneous '%' appended to the totals for count
values. This change removes it.  Fixes bug #24003

Don't destroy dictionary on error if we didn't create it.

Avoid assertion failure on bad input.  Closes bug #24031

Fixed bug which caused a crash if "end data." was not left aligned.

Merge branch 'master' of ssh://jmd@git.sv.gnu.org/srv/git/pspp

Work around bug #24033 (appending to output in cygwin).

Cygwin (and perhaps other w32 platforms) cannot write to a file
whilst it is open for reading.  This kludge therefore closes the
output after pasting into the output viewer and re-opens when new
data is appended.  It'll be slow if the output file becomes large.

Start generating ChangeLog automatically from Git commit messages.

Thus, at the same time we rename the existing ChangeLog files to
OChangeLog and add them to EXTRA_DIST to make sure that they get
distributed.

Estimate parameters. Moved some code to re-usable functions.

Pointer DEPVAR is now required to be set correctly in linreg.c

Initial versions of functions to estimate parameters
via the covariance matrix

Initialise label for piechart slices.
(Closes bug #24014

Bump minor version number so that git builds are distinct from released version

Use standard C99 isfinite, isnan, isinf in place of GSL substitutes.

In change a9afcdd22, "Use gsl_isnan instead of isnan, ...," isfinite,
isnan, and isinf were changed to use the GSL substitutes because of
lack of portability of the C99 versions.  Now, gnulib has portable
versions of all of these, so change them back.

This commit changes calls to gsl_finite() to call isfinite() instead
of the equivalent finite().  isfinite() is correct here: both gsl_finite()
and finite() return true for NaNs, which is a surprising result given
that a NaN is definitely not finite, but isfinite() returns false and,
more importantly, behaves as actually wanted in each place it is used
in PSPP code.

This commit requires upgrading gnulib to at least change 5183a0e3c,
"Add missing dependencies on new m4/exponent[fdl].m4 files," or later,
and re-running "make -f Smake".

Constness

Changed the default WORKSPACE value to 64MB.
Corrected documentation of WORKSPACE.
When --testing-mode is set, workspace can now be arbitrarily low.

New function for adding a numbering column to a casereader.

Merge branch 'master' of ssh://jmd@git.sv.gnu.org/srv/git/pspp

Conflicts:

src/math/ChangeLog

Merge with recent change by Jason.

Removed the value_cnt associated with the caseordering
Made sure that sort_execute returns a casereader the same
dimensions as its subject.

Dropped weight argument. Corrected update of entries of the covariance matrix

No longer store entire data set in memory.
Accumulate covariance matrix using covariance_pass_two

Why does git only commit some of the changes if you don't pass the -a flag?

Deleted lib/gsl-extras and updated gsl dependence to 1.8

Recurse to simplify flow of covariance_pass_two(). Move tests for V2 into for loop.

Merge branch 'master' of ssh://jstover@git.sv.gnu.org/srv/git/pspp

Fixed update of covariance matrix when both variables are categorical.
Added function to access counted categories.

Make PSPP_LINK2_IFELSE macro work with Autoconf earlier than 2.62.

Fixed updating of covariance matrix when both variables
are categorical. Reported by John Darrington

Added functions to compute covariance matrix.

Autodetect and fix GSL use of GCC-style "extern inline".

Enable users of dist tarballs to use --enable-gui-tools.

Formerly, we didn't distribute the necessary files.

Remove 'de' from LINGUAS and delete po/de.po.

Suggested by Andreas Streichardt <mop@globalpark.de>, who points out
that the poor German translations annoy German users.

Remove obsolete .gitignore entries.

Remove files from .gitignore that are specific to blp's workflow.

These files to ignore are supposed to be listed in the user's
core.gitexcludes file (see gitignore(5)), not in the repository's
.gitignore files.

Rename .cvsignore files to .gitignore.

Thus, Git will now ignore the same files that CVS did.

Fixed bug 23567. Added accessor functions to get/set means and standard deviations for variables associated with struct coeffs. Fixed corresponding values in tests.

removed src/math/linreg/

moved src/math/linreg.[ch] to src/math

Fix typo in AC_PREREQ command name.

Final changes for 0.6.0 release.

Updated about dialog text.

Updated authors' contributions.

Fixed bugs which manifested themselves as bus errors on solaris.

Added a --enable-anachronistic-dependencies flag to configure, which allows psppire to be built against older versions of GTK and other libraries.

Applied patch #6515 : Allow dlls to be built on w32 platforms

Made all the tests respect the PSPP_TEST_NO_CLEANUP environment variable.

Removed the '--text' argument to diff, which is non-portable and unnecessary.

Added "cd /" to the test scripts which don't already have it. This avoids warnings on platforms which refuse to delete the directory they're using.

Don't issue a warning if the TO and FROM encodings are identical.

Fixed non-portable shell syntax in tests.  Added a dist-hook target to check
for these in the future.

Use gsl_isnan instead of isnan, gsl_isinf instead of isinf, and
gsl_finite instead of finite, as a stopgap measure for portability
until appropriate gnulib modules are available.

* Smake (GNULIB_MODULES): Don't depend on memmem module, because
PSPP does not use memmem any longer.  Use
unilbrk/ulc-width-linebreaks module instead of the linebreak
module, because the latter was split up into multiple modules and
that's the one we actually need.

* automake.mk: Make install-data-hook depend on yelp-check if the
GUI is configured.

* automake.mk: Rename install-data-hook to yelp-check and mark it
phony.  This avoids an automake warning for duplicate
install-data-hook commands in doc/automake.mk and this file
(automake does not understand double-colon rules, since they are
not in POSIX).

Missing quotes.

Debian ships a package which provides ncurses5-config but no header files.
Work around this (buggy) situation.

Fix problems with yelp opening info files.

Fixed problem running make distcheck with gettext 0.17

Update po files.

Don't call gtk_widget_unparent on sheet->button if it's null.

Don't make the user wiggle the mouse to be able to click on the
Forward button twice in quick succession.

Change the mouse pointer to a watch to indicate that a long operation
is ongoing.

Don't use gtk_action_connect_proxy for popup menu items which
have labels with mnemonics

(get_tooltip_location): Prevent crash when a tooltip is being prepared
when the assistant is closed.

(text_data_import_assistant): Allocate the struct import_assistant on
the heap instead of the stack, to make it easier for memory debuggers
such as Valgrind to point out use of its members after this function
returns.

(gtk_sheet_dispose): Set the sheet's entry_container and button
members to NULL after unref'ing them, so that a later call to
gtk_sheet_for_all will not try to dereference a dangling pointer.

Don't append cell contents when there is none.