wmclockmon: update change-log

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmclockmon: bump version to 1.0.0

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmclockmon: update maintainer address

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmclockmon: add wmaker-dev to AUTHORS and Thomas Nemeth to THANKS

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmclockmon: remove old BUGS file and add a note to the INSTALL file about the requirement for the GTK development files

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmclockmon: update debug flags to reflect what I've been using

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: update change-log

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmclockmon: replace deprecated `gtk_misc_set_alignment`

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: replace deprecated `GtkTable`

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: replace deprecated `gtk_dialog_get_action_area`

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: replace deprecated `gtk_[hv]box_new`

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: bump GTK dependency to 3.0

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: remove obsolete `GTK_CALENDAR_WEEK_START_MONDAY`

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: replace deprecated `GtkCombo`

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: replace deprecated `GtkCList`

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: replace deprecated `gtk_signal_*`

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: replace deprecated `gtk_calendar_display_options`

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: replace deprecated `gtk_container_border_width`

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: replace deprecated `gtk_widget_set_usize`

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: replace deprecated `gtk_label_get`

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: replace deprecated `GTK_WIDGET_SET_FLAGS`

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: replace direct member access with getter calls

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: update change-log

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmclockmon: add missing `const` qualifiers

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: remove `bzero` call

The function was superseded by `memset` aeons ago, and the call is not
necessary.

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: handle fgets NULL returns and zero-length lines

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: fix pre-ANSI void-parameter function declarations

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: fix function-pointer <-> void-pointer conversions

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: fix unused parameter warnings

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: drop `getuid()` error checks

It is always successful.

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: add missing initializers

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: fix buffer overrun

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: properly fix allocating and freeing of `command`, `light_color` and the other GtkEntry strings

Instead of sometimes allocating memory and sometimes storing the values return
by `gtk_entry_get_text()`, always storing allocated memory and unconditionally
free it.

Do this for all the entry strings.

Fixes: 7f042fb7959f ("wmclockmon: check whether memory has been allocated before freeing.")
Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: fix mismatched signedness

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: fix assignment

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: fix conditional

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: update `getbool` not to modify its argument

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: remove unused code

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: update change-log

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmclockmon: use `AC_CONFIG_AUX_DIR`

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmclockmon: include sample config-files in dist tar-ball

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: include styles in dist tar-ball and install in `$(pkgdatadir)`

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: change compression for dist tar-ball

Ignore generated tar-balls.

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: simplify `INSTALL_PROGRAM` definitions

`AM_INSTALL_PROGRAM_FLAGS` is obsolete.

Define `INSTALL_PROGRAM` once.

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: use `AS_IF` and `AS_CASE`

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: use `AC_MSG_ERROR`

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: remove obsolete autoconf header checks

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: remove unused `AC_CANONINCAL_TARGET`

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: update automake macros

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: quote autoconf macro arguments

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: use `LDADD` for `gtk_LIBS`

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: use `LDADD` for all app's

wmclockmon-cal and wmclockmon-config use the wrong prefixes, but no prefixes
are necessary, so remove them all.

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: remove unused `IGNORE_*` variables

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: remove unused `OS` variable

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: remove unnecessary `AC_SUBST` of `LIBS`

autoconf does this automatically.

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: remove `--with-incs-from` and `--with-libs-from`

They are superfluous, duplicating the use of `LDFLAGS` and `CPPFLAGS`.

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: use `AS_HELP_STRING` for `--enable-debug` and fix typo'

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: rename `DFLAGS` `debug_CFLAGS`

`DFLAGS` is used by autoconf for the D programming language.

Add it to `AM_CFLAGS`, not `CPPFLAGS`, because none of the flags is for
cpp, and `CPPFLAGS` should be reserved for the user anyway.

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: use pkg-config for X libraries

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: add autogen script

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmclockmon: ignore build and editor artefacts

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: update change-log

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmclockmon: fix memory-leaks reported by Clang's static analyser

wmclockmon: fix global variables causing linkage failure

A number of global variables are declared in headers with no explicit linkage.
This results in there being definitions of them in multiple object files and
causes linking to fail under gcc-10.

Add `extern` to the header declarations and separate declarations with no
linkage in files where they are assigned.

wmclockmon: check whether memory has been allocated before freeing.

Two strings, command and light_color, were always freed even though they
were only conditionally allocated.  This could cause "munmap_chunk(): invalid
pointer" errors.  We fix this by keeping track of whether they have been
allocated and only freeing if they have.

wmclockmon: fix typos pointed out by Lintian.

In particular, we had spelling-error-in-binary and spelling-error-in-manpage
warnings.

wmclockmon: build against GTK+ 2.0.

Link: https://bugs.debian.org/437442
Link: https://bugs.debian.org/641335
Co-developed-by: Doug Torrance <dtorrance@piedmont.edu>

wmclockmon: update autotools files

Define the DATADIR macro and make sure that the auxiliary
programs wmclockmon-cal and wmclockmon-config are built properly.

wmclockmon: remove Debian packaging

The upstream source includes some ancient Debian packaging.
Get rid of it.

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: remove generated autotools files

Signed-off-by: Jeremy Sowden <azazel@debian.org>

wmclockmon: import version 0.8.1

wmclockmon: import version 0.8.0

wmbiff: bump to 0.4.37

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmbiff: fix generation of XPM's from separate build-directory

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmbiff: fix `print_info` prototypes in test sources

They should match the real `print_info` in gnutls-common.c.

Make sure all stubbed functions are declared in headers and those headers
are included where the functions are definitions.

Reported-by: Eli Schwartz <eschwartz93@gmail.com>
Link: https://bugs.gentoo.org/864855
Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmbiff: `noinst_PROGRAMS` -> `check_PROGRAMS`

test_wmbiff and test_tlscomm are intended to be run by `make check`.

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmbiff: update gnutls pkg-config configuration

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmbiff: use pkg-config to find libgcrypt

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmbiff: improve formatting of Makefile.am

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

Make AddMouseRegion's index unsigned

This was previously already done for wmbiff/wmgeneral: the index
argument is used as an index into an array, and is always only tested to
be below the upper bound. Passing in a negative value would pass that
test, and would result in an out-of-bounds access. All dockapps appear
to get that right, but libdockapp could see this violated by a (future)
user.

Temperature.app: fix temperature data URL

Hello All,

I am new to Window Maker. I wanted to install the Temperature.app dockapp
and found out that the ftp address that retrieves temperature data is
invalid. Investigating further I found that noaa has changed the address. I
have created a patch and am attaching it to this email.

I did not create a pull request, It wanted me to create a fork off of the
dockapps repository.

Maybe someone might find the dockapp useful and might use it.

Regards
Riza

Fix building with clang 16

From Clang 16 (and GCC 13) -Wimplicit-function-declaration has been
turned on by default. This resutls in warnings such as

song_hash.c: In function ‘insert_song’:
song_hash.c:22:39: warning: implicit declaration of function ‘malloc’ [-Wimplicit-function-declaration]
   22 |         new_elt = (struct hash_elt *) malloc(sizeof(struct hash_elt));
      |                                       ^~~~~~

This patch should fix these warnings. The warnings can be reproceded by
both recent version of Clang and GCC.

First discovered on Gentoo Linux: https://bugs.gentoo.org/882321

Signed-off-by: Brahmajit Das <brahmajit.xyz@gmail.com>

wmusic: bump version to 2.1.1 and update change-log

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmusic: fix overflow when calculating remaing track time

Track position and length are 64-bit integer quantities scaled in
microseconds, so use appropriate types.

Link: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1038264
Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmusic: divide time once in `DrawTime`.

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmusic: use appropriate variable names for track positions

Currently track posiions are called "time".

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmusic: use appropriate variables and function names for track numbers

Currently track numbers are referred to as "position".

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmressel: fix refresh rate check and add interlace mode detection

Interlace video mode was not supported and the refresh rate check using
only dotclock was inaccurate, which was selecting multiple current modes
and freezing the app.

Fixed build error on missing libiconv

ld error.
Error discovered while building in Freebsd 13.1. Uncertain if change
will cause problems on other platforms.

Updated deps error info

information to user updated with working link

Makefile cleanup - wmSMPmon

Disregard previous patch - forgot to update comment in makefile.
Related to issue #43.
Next on the list.

Changed path for man-file installation

Fixes problem with washerDryer / make install - broken path for man-pages
Changed path to what seems to make more sense.
Perhaps another approach might be to programmaticly examine manpath.
At the same time, if multiple paths are available a new problem presents itself.

Fixed make install/uninstall

washerDryer dockapp
related to issue #43
Next on the list created by d-torrance

Update README - Download link out of date.

Changed to current

wmcalclockkbd: Move variable definitions from xutil.h -> xutil.c

Otherwise we get "multiple definition" linking errors when compiling
with "-fno-common", which has been the default since GCC 10.

wmarchup: Add X11 linker flags

Otherwise, we get an "undefined reference" error during linking.

wmarchup: Fix path to libdockapp header

buttonmaker: Link against libdockapp instead of vendored wmgeneral

cputnik: Drop "file_types" variable name for enum.

It was causing a "multiple definition" linking error.  This could be
fixed using "static", but then we get an "unused variable" warning.
The F_* macros it defines are used, however.

cputnik: Update "make install" to support prefix/DESTDIR variables

Also other variables as recommended by GNU make docs.