In particular,
- Drop INCLUDES and LIBINC variables; referenced deprecated X11R6 directories
and INCLUDES unnecessarily included the default path.
- Switch hard-coded gcc for CC variable.
- Move hard-coded -02 and -Wall flags to new CFLAGS variable.
- Switch ${} to $() for consistency.
- Add new INSTALL_DIR variable for creating installation directories.
- Respect CPPFLAGS and LDFLAGS.
To simplify maintenace, all references to the current version number outside
of the appropriate macro in in wmtime.c have been removed. Note that this
macro had been WMMON_VERSION, likely a copy/paste error, and has been
renamed to WMTIME_VERSION.
In particular,
- Remove reference to deprecated X11R6 directory.
- Honor PREFIX and INSTALL variables.
- Change definition of CFLAGS from += to =; perhaps users don't want to
append -O2. Also move -Wall to CFLAGS.
- Use C*FLAGS only when compiling and LDFLAGS only when linking.
- Create directories if needed during install.
- Install manpage.
Previously, wmitime only had support for English, French, and (in Debian
only) Hungarian. In addition, the choice was made at compile time.
This patch adds run-time support for any language using the Latin alphabet.
The locale is determined by the user's environment or may be specified on
the command line with the "-l" option. Note that users whose environment
specifies a non-Latin locale may wish to use, e.g., "-l C", as otherwise
no date will appear.
Note that, for simplicity, the month and day are now displayed as "01 JAN"
as opposed to "JAN 01". (Previously, the former format was used for English
and the latter for French.)
Previously, release history was kept in README. This has been separated out
into a new file to avoid a "no-upstream-changelog" Lintian warning in the
Debian package.
In particular, fix warnings of the form "warning: ignoring return value of
‘char* fgets(char*, int, FILE*)’, declared with attribute warn_unused_result" by
checking return value of fgets.
Previously, compiling wmcdplay caused a large number of warnings of the form
"deprecated conversion from string constant to ‘char*’". This was fixed by
changing "char *" declarations to "const char *", especially in the .xpm files,
and using "const_cast" as needed when an external library expected a non-const.
Changes include:
- The artwork files XPM/*.art are now created during build time and
therefore have been removed from the repository.
- The script which creates them is now installed alongside the art files in
case users would like to design their own artwork.
- The "makeall" script, whose function has been replaced by the Makefile,
has been removed.
- The default artwork can now be chosen at compile time by the --with-art
option to ./configure.
- Changes have been documented in ARTWORK.
Inspired in part by the Debian patch [1].
[1] http://sources.debian.net/src/wmcdplay/1.0beta1-13/debian/patches/install_art_scripts.patch/
Changes include:
- Rename to wmclock.1.
- Use @prefix@ instead of /usr for the location of the language xpm files.
- Hardcode date instead of picking it up from a variable in the Makefile.
This is a feature that was present in the pre-autotools configure script.
Note a syntax change:
./configure --lang foo
is now
./configure --with-lang=foo
Also remove the .xpm files in the root directory, as they are created during
configuration.
