Use TS_RAW_US_MAX_DIGITS in parse_time().

Cirrus CI: Bump FreeBSD up to 13.5.

Autoconf: Define PACKAGE_URL.

Same as in libpcap and tcpdump.

Add a GitHub issue template

Create the .github/ISSUE_TEMPLATE directory and add in it:
- the issue_template.md file with some keywords (name, about, ...),
- the config.yml file containing "blank_issues_enabled: false".

(Same as in libpcap and tcpdump)

[skip ci]

Add *.orig and *.rej to .gitignore. [skip ci]

Cirrus CI: Update the macOS task image

Based on https://cirrus-ci.org/guide/macOS/.

This should avoid the warning:
Only ghcr.io/cirruslabs/macos-runner:sonoma is allowed. Automatically
upgraded.

Add tags file build by ctags command via "make extags" to .gitignore

build_matrix.sh: Add a "| cat" at the end of a git command

This can avoid a display pause problem with "less -S" as a git pager.

This is a folow-up to 99a008a7dbc855bbba28401214ecae723fa416e8.

Revert "Cirrus CI: Add a task for haiku-amd64." [skip ci]

That was an unfinished work in progress that I put into the master
branch too early.

build_matrix.sh: Run "git show --oneline -s"

It is used to identify the git HEAD.

Add vim swap files to .gitignore. [skip ci]

Cirrus CI: Add a task for haiku-amd64.

Cirrus CI: Bump FreeBSD up to 13.4.

Start tcpslice 1.9-PRE-GIT.

[skip ci]

This is tcpslice 1.8.

Cirrus CI: Set LANG in coverity_task same as in linux_task

To avoid these warnings:

perl: warning: Setting locale failed.
perl: warning: Please check that your locale settings:
LANGUAGE = (unset),
LC_ALL = (unset),
LANG = "en_US.UTF-8"
    are supported and installed on your system.
perl: warning: Falling back to the standard locale ("C").

autogen.sh: Delete all trailing blank lines at end of configure.

autogen.sh: Allow to configure Autoconf warnings.

In autogen.sh translate AUTOCONF_WARNINGS to Autoconf WARNINGS
environment variable if set.  During "make releasetar" instead of
autoreconf run autogen.sh to act upon the same environment variables as
the latter.  Since all current Cirrus CI tasks run in VMs with Autoconf
2.71 or later, set the new variable there to suppress obsolete construct
warnings.

Cirrus CI: Bump Ubuntu up to 24.04.

While at it, avoid the irrelevant library warnings from the configure
script and set DEBIAN_FRONTEND better.

CHANGES: Add some changes

mkdep: Exit with a non-zero status if a command fails

This could help detect errors such as in the previous commit.

Makefile.in: Fix the depend target

Fix an error on Solaris 10 like:
./mkdep: /opt/solarisstudio12.3/bin/cc -D_STDC_C99=: not found

When configure get some compiler option like:
checking for /opt/solarisstudio12.3/bin/cc option to accept ISO C99...
-D_STDC_C99=
Makefile will contain:
CC = /opt/solarisstudio12.3/bin/cc -D_STDC_C99=

And if we use '-c "$(CC)"' mkdep will set and try to run:
CC="/opt/solarisstudio12.3/bin/cc -D_STDC_C99=", which is incorrect.

Remove the quotes to allow mkdep to set CC with the compiler name and
set flags with the option.

Makefile.in: Update the .c.o build rule

Don't use the hacks for the rather old SunOS 4.

It's helps to do VPATH builds, e.g. 32-bit, 64-bit in two directories.
(https://www.gnu.org/software/automake/manual/html_node/VPATH-Builds.html)

Update the error message when checking for pcap_loop()

Add a first error case when trying to build a 32-bit tcpslice with a
64-bit libpcap or vice versa.

[skip ci]

Fixup the previous commit.

Makefile.in: Add instrumentation configuration in releasecheck target

Same as in tcpdump.

Cirrus CI: Add the binutils-dev package, Linux task.
It is needed to build with --enable-instrument-functions.

Cirrus CI: Add another missing package.

make releasecheck
/bin/sh: 10: git: not found
/bin/sh: 15: git: not found
tar: This does not look like a tar archive
tar: Exiting with failure status due to previous errors
make: *** [Makefile:184: releasetar] Error 2

Fixup the previous commit.

./configure needs libpcap, use a distribution package.

Cirrus CI: Reduce the footprint a little bit.

Use parallel build for macOS only, for FreeBSD and Coverity use a single
CPU core, for Linux use a minimal configuration and remove the build
matrix from the task.

Cirrus CI: Do not test on FreeBSD 14.0. [skip ci]

Buildbot does it now.

Include <config.h> unconditionally

Builds using Autotools or CMake generate config.h, thus remove the
'#ifdef HAVE_CONFIG_H'/'#endif'.

Use <fcntl.h>, <stdint.h> and INT32_MAX directly.

They all are in POSIX.1-2001.

Forgot to include CHANGES in the previous commit.

[skip ci]

Clarify the man page further.

Mention the imperfections from GH issue #23, fix a formatting
inconsistency for the time example.

[skip ci]

Cirrus CI: Try FreeBSD 13.3.

Address a few warnings from Sun C.

cc: Studio 12.6 Sun C 5.15 SunOS_i386 2017/05/30

"./seek-tell.c", line 57: warning: implicit function declaration: fseeko
"./seek-tell.c", line 63: warning: implicit function declaration: ftello
"./tcpslice.c", line 183: warning: implicit function declaration: getopt
"./tcpslice.c", line 205: warning: statement not reached
"./tcpslice.c", line 302: warning: implicit function declaration: fileno

Fix a shellcheck warning in the previous commit.

CI: Use a workaround for OpenIndiana Clang.

Same as in libpcap.

Update -h (help) option to print 32/64-bit build and time_t size

With an Autoconf update: Get the size of a void * and a time_t.

With this change we know:
whether this is a 32-bit or 64-bit build ;
whether the time_t size is 32-bit or 64-bit.

Fix a typo

Found by the codespell command.

[skip ci]

Add a warning flag (-Wundef)

Fixup the previous commit.

Cirrus CI: Try newer compilers on FreeBSD.

CI: Use -errtags=yes for Sun C. [skip ci]

CI: Improve Haiku identification. [skip ci]

Before: Haiku-hrev57609 Feb 28 2024 07:02:35
After: Haiku-hrev57609

Before: Haiku-hrev56578
After: Haiku-hrev56578+95

Extend "make shellcheck" onto mkdep too.

Same as in tcpdump.

mkdep: Use TMPDIR if it is set and not null

CI: Copy recent improvements from libpcap.

Start tcpslice 1.8-PRE-GIT.

[skip ci]

This is tcpslice 1.7.

Autoconf: Set PACKAGE_BUGREPORT in AC_INIT().

autoconf: Update config.{guess,sub}, timestamps 2024-01-01

In INSTALL.md say "missing" is not really there.

(since commit 07d50c9)

[skip ci]

Fix a typo

Found by the codespell command.

[skip ci]

Spell INSTALL.md in aclocal.m4.

I forgot to make these changes in the earlier commit 017ee31, which
converted the file.  Make the error messages the same as in tcpdump
aclocal.m4.

Extend "make shellcheck" onto autogen.sh too.

Address a warning from shellcheck 0.9.0:

autogen.sh:11:39: warning: Prefer [ p ] && [ q ] as [ p -a q ] is not
  well defined. [SC2166]

Remove remaining props for IRIX and OSF/1.

Add more entries to the 1.7 section in CHANGES.

Condense the old sections for consistency.

Add recent contributors to CREDITS.

Cirrus CI: Drop FreeBSD 12.4 and add 14.0.

Same as in tcpdump.

CI: Copy build_common.sh from libpcap.

build_matrix.sh: Fix a shellcheck note

The note was:
Double quote to prevent globbing and word splitting. [SC2086]

Look for the VERSION file in the source directory.

This matches libpcap and tcpdump.

It used to be necessary for out-of-tree builds.  It's no longer needed
for that, but at least it means that attempts to build a release tarball
will fail with "you're not in a git clone directory" rather than "I
can't find the VERSION file".

Now that the configure script is no longer in the repository, but is
generated by autogen.sh, and given that configure.ac gets the package
version from the VERSION file, we should probably just use
PACKAGE_VERSION in Makefile.in rather than `cat $(srcdir)/VERSION`.

instrument functions: Use two loop indexes with reduced scopes

The first is a long, comparison with the long 'symcount'.
The second is a int, comparison with the int 'profile_func_level'.

Moreover:
Add a cast to silence the following warning:
./instrument-functions.c:143:31: warning: implicit conversion changes
signedness: 'long' to 'unsigned long' [-Wsign-conversion]
        symtab = (asymbol **)malloc(symsize);
                             ~~~~~~ ^~~~~~~
(symsize is a long as return type of bfd_get_symtab_upper_bound().)

(Same as in tcpdump.)

[skip ci]

Add some warning flags for clang 13 or newer

-Wnull-pointer-subtraction
-Wunused-but-set-parameter
-Wunused-but-set-variable

Prefer calloc over malloc

malloc + memset can be better expressed as calloc.

Merge pull request #22 from jsoref/spelling

spelling: subsequent

spelling: subsequent

Signed-off-by: Josh Soref <[email protected]>

Makefile.in: Update the whitespacecheck target

install-sh has no longer a tab at the end of one line.

This is a follow-up to e2c80762d208f5b6fc3911b002da2df1d1c180c5.

Update install-sh script to the latest available version

The version of install-sh we have was made over 2 decades ago,
and last edited 9 years ago.

I updated the script by running autoreconf -fiv on autoconf 2.71,
and autoreconf automatically updated the script.

Cirrus CI: Back GCC version off on Linux.

Unfortunately, a simple workaround did not work, so let's leave the GCC
part of Linux task as it was for now.  Set LANG same as in tcpdump.

Cirrus CI: Try to fix things after the previous commit.

Apparently, Clang 16 is not available on FreeBSD 12.4, let's see if
Clang 15 is.  In the Linux task specify CC so Autoconf can generate the
Makefile (steps that do not involve any C compliling should not depend
on a C compiler presence, maybe in future this will have a better
solution).

Cirrus CI: Bump a few versions up.

FreeBSD 13.2 was released almost a month ago, let's see if it is
available in this infrastructure.  Try switching to Clang 16 in the
same go.  Whilst at it, switch from default to the most recent compiler
versions on Ubuntu 22.04, which already works fine in Buildbot.

instrument functions: Reduce the scope of a variable

Fix the cppcheck warning:
instrument-functions.c:77:15: warning: The scope of the variable
'instrument_type' can be reduced. [variableScope]

[skip ci]

Update config.{guess,sub}, timestamps 2023-01-01,2023-01-21

From repository git://git.savannah.gnu.org/config.git

Check for errors before calling posix_fadvise().

These error conditions are unlikely, but let's not count on that.

Autoconf: Synchronize with tcpdump aclocal.m4.

Minimize the diff and update the change log.

configure: don't use egrep, use $EGREP.

(ported from tcpdump commit 7659491)

On Solaris, for 64-bit builds, use the 64-bit pcap-config.

(ported from tcpdump commit c39d40a)

configure: special-case macOS /usr/bin/pcap-config as we do in CMake.

(ported from tcpdump commit fc240c6)

autoconf: Find a local libpcap even with rcX directory suffix

(ported from tcpdump commit f91ef58)

Remove a trailing tab

(ported from tcpdump commit a4e0f7d)

autoconf: replace --with-system-libpcap with --disable-local-libpcap.

(ported from tcpdump commit daff285)

Fix --static-pcap-only test on Solaris 10.

(Same as in tcpdump.)

Before:

checking for pcap-config... ../libpcap/pcap-config
grep: illegal option -- q
Usage: grep -hblcnsviw pattern file . . .
checking for pcap_loop... no
configure: error: This is a bug, please follow...

After:

checking for pcap-config... ../libpcap/pcap-config
checking for pcap_loop... yes

Fixup the previous commit. [skip ci]

In fact, Haiku has posix_fadvise().

Use posix_fadvise() on input files if available.

Let's confirm this works as expected before trying the same elsewhere.

Autoconf: Lose unused tcpslice_telloseeko.

This variable is unused since commit e1467ce.

Cirrus CI: Configure with --enable-instrument-functions, Coverity task

Add the required binutils-dev package.

[skip ci]

instrument functions: Use an environment variable instead of config files

If the environment variable INSTRUMENT is
- unset or set to an empty string, print nothing, like with no
  instrumentation
- set to "all" or "a", print all the functions names
- set to "global" or "g", print only the global functions names

The configuration with --enable-instrument-functions remains.

Note that before this change, the default was to print all functions.
Now it is to print nothing.

This allows to run:
$ INSTRUMENT=a ./tcpslice ...
$ INSTRUMENT=g ./tcpslice ...
$ INSTRUMENT= ./tcpslice ...
or
$ export INSTRUMENT=global
$ ./tcpslice ...

This also allows to run the statically compiled binary on another host
after copying it.

It is no longer necessary to modify the configuration with:
$ make instrument_all
$ make instrument_global
$ make instrument_off
(Targets removed.)

Update .gitignore and Makefile.in accordingly.

Moreover:
Reduce the scope of a variable.
Rename a variable.
Remove '\n' in the perror() call.
Remove 2 spaces in function calls (style).

instrument functions: Add a NULL check

This should fix Coverity CID 310986:
Null pointer dereferences  (FORWARD_NULL)
Passing null pointer "func" to "strncmp", which dereferences it.

instrument functions: Add a length check

We need space for the '\0'.

This should fix Coverity CID 310985.

Moreover:
Use '\0' for the null character.

Get the patterns right in .gitignore.

Almost all files there are meant to be in the source tree root.

[skip ci]

Makefile.in: Add instrument-functions.o in CLEANFILES

It is not in OBJ because it is optionally built.

[skip ci]

tcpslice(1): Use bold font more consistently. [skip ci]

Pay attention to the surrounding punctuation, use bold for file names.

CI: Introduce TEST_RELEASETAR. [skip ci]

The default behaviour is the same as before, setting it to "no" works
around broken build matrix on AIX 7.1 and 7.2.

State there are currently no OS dependent defines and prototypes

The last header was removed by commit
55f6b22a08f36b6127ad05a44e498d576b639ff9.

instrument functions: Minimize stat() calls

This change decreases the execution time when the file indicating that
instrumentation is "off" exists.

[skip ci]

Make a comment more readable

Moreover:
Replace the deprecated 'egrep' variant by 'grep -E'.

[skip ci]

Makefile.in: Update whitespacecheck target to show spaces before tabs

Avoid a space before a tab

This change avoids a false positive when looking for bad indentations.