libblkid: add debug message for private DM device skip

Add a DBG() trace and set errno = EINVAL when
blkid_new_probe_from_filename() skips a private device-mapper
device, to aid troubleshooting and distinguish from I/O errors.

Addresses: https://github.com/util-linux/util-linux/pull/4120
Signed-off-by: Karel Zak <kzak@redhat.com>

Merge branch 'private-dm' of https://github.com/zkabelac/util-linux

* 'private-dm' of https://github.com/zkabelac/util-linux:
  libblkid: check for private DM device before open

Merge branch 'PR/libsmartcols-width-calc' of https://github.com/karelzak/util-linux-work

* 'PR/libsmartcols-width-calc' of https://github.com/karelzak/util-linux-work:
  tests: update to reflect libsmartcols changes
  libsmartcols: enlarge columns with absolute width hint
  column: Add notes about the role of the column header
  libsmartcols: improve truncation of column width
  tests: add column strictwidth test
  libsmartcols: fix column width calculation
  libsmartcols: fix width= property parsing

Merge branch 'PR/dmesg-delta-notime' of https://github.com/karelzak/util-linux-work

* 'PR/dmesg-delta-notime' of https://github.com/karelzak/util-linux-work:
  dmesg: rename is_time_fmt_set() to is_time_fmt()
  dmesg: warn on boot time failure instead of suppressing timestamps
  dmesg: add replace_time_fmt() and replace_delta_fmt() helpers
  tests: add dmesg -d -t and -t -d timestamp format tests
  dmesg: fix -d -t (--show-delta --notime) output regression
  dmesg: remove delta variable, use time format list consistently

getopt: document whitespace as separator in --longoptions

  The --longoptions option accepts not only commas but also whitespace
  (spaces, tabs, or newlines) as separators between option names.
  Update the man page to reflect this behavior.

  Addresses: https://github.com/util-linux/util-linux/issues/4119

libblkid: check for private DM device before open

blkid_new_probe_from_filename() opens the device before calling
blkid_probe_set_device(), which checks sysfs_devno_is_dm_private()
and sets BLKID_FL_NOSCAN_DEV.  But the open() itself bumps the
kernel open count, so a concurrent DM_DEVICE_REMOVE ioctl sees
EBUSY even though blkid never actually probes the device.

Move the private-device check before open() in
blkid_new_probe_from_filename().  The sysfs UUID is readable
without opening the block device, so this closes the race window
between udev-worker's blkid builtin and device-mapper remove.

Note: blkid_verify() in verify.c already does the check before
open() -- this patch makes the probe path consistent.

Signed-off-by: Zdenek Kabelac <zkabelac@redhat.com>

build: link test_pager against lib_common in meson too

Same fix as the Autotools Makemodule.am change — the Meson build
also needs lib_common for ul_strfconcat().

build: link test_pager against libcommon

pager.c now uses ul_strfconcat() from strutils.c, so the
test_pager binary needs libcommon.la to resolve the symbol.

pager: use ul_strfconcat() instead of fixed-size buffer

Replace char less_val[256] with dynamically allocated ul_strfconcat()
to avoid silent truncation when LESS env is already long.

pager: support --header 0,M for column-only freezing

less accepts --header 0,M to freeze columns without header lines.
Handle header_width > 0 independently of header_lines.

pager: address review feedback for less --header support

- Remove static globals; store header_lines/header_width in
  struct child_process instead
- Invert pager_open/pager_open_header: pager_open() now calls
  pager_open_header(0, 0) as a thin wrapper
- Append --header to existing LESS value when set, instead of
  silently skipping header setup
- Flatten nested if-blocks into single setenv() call
- Use int consistently instead of mixing int and size_t
- Document that header_lines=0 means no header regardless of width

libsmartcols: add scols_table_calculate to docs sections

Add the new scols_table_calculate() symbol to the
libsmartcols-sections.txt documentation file to fix the
checklibdoc CI check.

libsmartcols: add scols_table_calculate(), pager: add less --header support

Add scols_table_calculate() public API to allow column width calculation
without printing. This enables callers to query column widths (via
scols_column_get_width()) before starting output, which is needed to
set up "less --header" for freezing table headers and the first column.

The new function runs __scols_initialize_printing() (which includes
__scols_calculate()) and sets an is_calculated flag so that subsequent
scols_print_table() calls skip redundant recalculation.

Add pager_open_header() to lib/pager.c that sets the LESS environment
variable with "--header N,M" options before spawning the pager process.
This freezes the first N lines (header row) and first M character columns
(first table column) when scrolling in less.

Addresses: https://github.com/util-linux/util-linux/issues/3542

autotools: Remove the need for --disable-liblastlog2

We typically build all in-tree libraries, but liblastlog2 requires an
external dependency on sqlite3. It doesn't make sense to force all
users to use --disable-liblastlog2 to explicitly disable it.

Let's check for sqlite3, and if it's not installed, disable the library.

Addresses: https://github.com/util-linux/util-linux/issues/4112#issuecomment-4042305081
Signed-off-by: Karel Zak <kzak@redhat.com>

dmesg: rename is_time_fmt_set() to is_time_fmt()

Shorten the function name for consistency with the other time format
helpers (include_time_fmt, exclude_time_fmt, replace_time_fmt).
No functional change.

Signed-off-by: Karel Zak <kzak@redhat.com>

dmesg: warn on boot time failure instead of suppressing timestamps

When dmesg_get_boot_time() fails, warn the user instead of silently
adding DMESG_TIMEFTM_NONE to suppress all timestamp output. The user
asked for a specific format and should get it, with a warning that
wall-clock timestamps may be inaccurate.

Also move the boot time check after the time format post-processing
so it operates on the final resolved format list, and add
DMESG_TIMEFTM_CTIME_DELTA to the condition since it also requires
boot time.

Signed-off-by: Karel Zak <kzak@redhat.com>

dmesg: add replace_time_fmt() and replace_delta_fmt() helpers

Introduce replace_time_fmt() to replace one time format with another
in the format list, and replace_delta_fmt() which combines
exclude_time_fmt() + replace_time_fmt() into a single call.

This deduplicates the for-loop pattern repeated in the delta
merging post-processing.

Signed-off-by: Karel Zak <kzak@redhat.com>

tests: add dmesg -d -t and -t -d timestamp format tests

Add two sub-tests to verify that -d -t (delta-notime) and -t -d
(notime-delta) both produce delta-only output regardless of option
order.

Addresses: https://github.com/util-linux/util-linux/issues/4110
Signed-off-by: Karel Zak <kzak@redhat.com>

dmesg: fix -d -t (--show-delta --notime) output regression

When -d and -t are used together, only the time delta should be
printed without the timestamp prefix. This was broken because
DMESG_TIMEFTM_NONE in the format list caused the print_record()
loop to break early before processing the delta format.

Resolve the conflict after option parsing: when NONE coexists with
a delta format (TIME_DELTA or DELTA), replace all formats with just
DELTA. When NONE is used without delta, it suppresses all timestamps
as before.

Fixes: https://github.com/util-linux/util-linux/issues/4110
Signed-off-by: Karel Zak <kzak@redhat.com>

dmesg: remove delta variable, use time format list consistently

Replace the separate 'delta' flag variable with direct use of the
time format list for all timestamp-related options:

 -d (--show-delta) now calls include_time_fmt(DMESG_TIMEFTM_TIME_DELTA)
 -t (--notime) now calls include_time_fmt(DMESG_TIMEFTM_NONE) without
    reset_time_fmts(), same as --time-format notime

This makes -d and -t behave the same way as their --time-format
equivalents, storing all requested formats in the format list
for post-getopt resolution.

The delta merging logic is split into two blocks:
 - TIME_DELTA + CTIME → CTIME_DELTA (for -d -T)
 - DELTA + TIME/CTIME → TIME_DELTA/CTIME_DELTA (for --time-format delta)

Addresses: https://github.com/util-linux/util-linux/issues/4110
Signed-off-by: Karel Zak <kzak@redhat.com>

waitpid: consistently write the abbreviation "PID" in all-uppercase

Also, use %d instead of %u, as all other comparable messages use %d.

CC: Christian Goeschel Ndjomouo <cgoesc2@wgu.edu>
Signed-off-by: Benno Schulenberg <bensberg@telfort.nl>

nsenter: (man) correct the markup and improve the formatting

Remove the mistaken angle brackets that were introduced last month
by commit f2a5997869, add the missing equals sign, and put both it
and the subsequent colon in bold as they are literal characters.

Also, indent the list of paths to make it clearer where it ends,
and so that the subsequent paragraph gets indented correctly.

CC: Xiao Liang <shaw.leon@gmail.com>
CC: Christian Goeschel Ndjomouo <cgoesc2@wgu.edu>
Signed-off-by: Benno Schulenberg <bensberg@telfort.nl>

nsenter: (usage) improve the alignment of the option descriptions

Perfect alignment is not possible, otherwise too little space remains
for the descriptions.

Also, slightly reword one description to fit within 80 columns again,
and align also a description that was missed by commit 986188d1da two
monts ago.

Signed-off-by: Benno Schulenberg <bensberg@telfort.nl>

nsenter: (usage) correct the formatting of eight option arguments

Angle brackets should wrap each placeholder separately: <file> and
<nsid>.  Angle brackets should not enclose metacharacters (|) nor
literal ones (= and :).

(The mistaken formatting was introduced by commit f2a5997869.)

CC: Xiao Liang <shaw.leon@gmail.com>
Signed-off-by: Benno Schulenberg <bensberg@telfort.nl>

lsmem: correct the grammar of an error message

That is: make the message identical to its twin nine lines back.

(The spello was introduced by commit 92d018a105 from five months ago.)

CC: Sumanth Korikkar <sumanthk@linux.ibm.com>
Signed-off-by: Benno Schulenberg <bensberg@telfort.nl>

Merge branch 'improve/4070' of https://github.com/echoechoin/util-linux

* 'improve/4070' of https://github.com/echoechoin/util-linux:
  man: add ENVIRONMENT section for smartcols support in various man pages
  man: hide LIBSMARTCOLS_JSON in man pages that do not support --json

tests: update to reflect libsmartcols changes

The library now reduces the width of columns even when a width hint is
specified. This affects some test outputs, particularly those where
small numbers appear in columns with a width hint wider than the
column header (e.g., SIZE with hint=5).

Signed-off-by: Karel Zak <kzak@redhat.com>

libsmartcols: enlarge columns with absolute width hint

The width hint is strictly followed only if SCOLS_FL_STRICTWIDTH is
specified. The previous patch ensures minimal space usage for all
other columns. This patch enlarges columns with an absolute hint (>
1.0) until the hints are satisfied and there is free space on screen.

Signed-off-by: Karel Zak <kzak@redhat.com>

man: add ENVIRONMENT section for smartcols support in various man pages

This update introduces an ENVIRONMENT section to the man pages
of several utilities. The section includes references to the common
environment variable for smart column output, ensuring consistent
documentation across tools that support the --json output format.

terminal-colors.d: (man) re-apply improvements lost in merge

Commit 3252142521 improved the man page formatting and wording,
but merge commit d708fc7db6 undid all those changes due to a
conflict resolution mess.  Re-apply the improvements:

 - do not show 'type' as an optional part in the synopsis
 - indent the list of file types for clarity
 - correct and improve several wordings
 - fix grammar ("which provide" not "which provides")
 - compact the examples section

Addresses: https://github.com/util-linux/util-linux/commit/3252142521
Signed-off-by: Karel Zak <kzak@redhat.com>

column: Add notes about the role of the column header

Addresses: https://github.com/util-linux/util-linux/issues/4104
Signed-off-by: Karel Zak <kzak@redhat.com>

libsmartcols: improve truncation of column width

The worst case is to aggressively truncate the largest column. The
default is to reduce it by three characters, but if the needed space
is smaller than three characters, it's better to be less aggressive.

Signed-off-by: Karel Zak <kzak@redhat.com>

tests: add column strictwidth test

Signed-off-by: Karel Zak <kzak@redhat.com>

libsmartcols: fix column width calculation

echo -e "a b c\na b c" | ./column --table --table-column "name=X,width=5" -C "name=YY,width=5" -C "name=Z" --output-separator '|'
X    |YY|Z
a    |b |c
a    |b |c

The only difference between the X and YY columns is the column name.
The column header width affects the use of the width property. This
occurs when the minimum and maximum column widths are the same ("YY").

Fixed version:

echo -e "a b c\na b c" | ./column --table --table-column "name=X,width=5" -C "name=YY,width=5" -C "name=Z" --output-separator '|'
X|YY|Z
a|b |c
a|b |c

Note that width= is just a hint and should not be used when the
'strictwidth' property is not specified.

Signed-off-by: Karel Zak <kzak@redhat.com>

libsmartcols: fix width= property parsing

The code uses 'errno' to detect errors, but the variable is not set to
zero before parsing the number. As a result, any previous error can
affect the width number parsing.

Signed-off-by: Karel Zak <kzak@redhat.com>

mkfs.cramfs: Properly check mmap results

The POSIX manual page states that the error return value is MAP_FAILED.
Sync with all other mmap checks throughout util-linux.

Also, test an mmap call which previously was not tested.

Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>

umount: fix --graceful option cosmetics and tests

- fix missing comma in man page option syntax
- fix typo "succesfully" -> "successfully"
- move --graceful before --namespace in help text
- shorten help text to single line
- improve man page description, add root-only note
- add test for non-existent path with --graceful

Addresses: https://github.com/util-linux/util-linux/pull/4059
Signed-off-by: Karel Zak <kzak@redhat.com>

Merge branch 'umount_graceful_option' of https://github.com/cgoesche/util-linux-fork

* 'umount_graceful_option' of https://github.com/cgoesche/util-linux-fork:
  umount: add test for --graceful option
  umount: new --graceful option to exit successfully even if filesystem is unmounted

Merge branch 'PR/meson-libcommon-duplicates' of https://github.com/karelzak/util-linux-work

* 'PR/meson-libcommon-duplicates' of https://github.com/karelzak/util-linux-work:
  lib/meson: move caputils.c from lib_common to per-binary sources
  lib/meson: remove unconditional procfs.c from lib_common_sources
  lib/meson: remove duplicated source files from lib_common_sources

fsck: Fix stack overflow with many options

If fsck is given too many options, a stack overflow can occur because
these options are temporarily stored in a fixed sized array on stack.

As with arguments, check if too many options are supplied and call errx
if this is the case.

Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>

Merge branch 'eject_oob' of https://github.com/stoeckmann/util-linux

* 'eject_oob' of https://github.com/stoeckmann/util-linux:
  eject: Fix OOB access with empty argument

chmem: check ul_path_read_buffer() return value

Reported-by: Coverity Scan
Signed-off-by: Karel Zak <kzak@redhat.com>

Merge branch 'pg_sigchld' of https://github.com/stoeckmann/util-linux

* 'pg_sigchld' of https://github.com/stoeckmann/util-linux:
  pg: Reset SIGCHLD to default

Merge branch 'eject_sigchld' of https://github.com/stoeckmann/util-linux

* 'eject_sigchld' of https://github.com/stoeckmann/util-linux:
  eject: Reset SIGCHLD to default for wait

Merge branch 'bugfix' of https://github.com/cuiweixie/util-linux

* 'bugfix' of https://github.com/cuiweixie/util-linux:
  fincore: should add break here

Merge branch 'master-branch-1' of https://github.com/Leefancy/util-linux

* 'master-branch-1' of https://github.com/Leefancy/util-linux:
  The incorrect order of fcntl() parameters resulted in the inability to clear O_NONBLOCK

Merge branch 'pager_fork_failure' of https://github.com/stoeckmann/util-linux

* 'pager_fork_failure' of https://github.com/stoeckmann/util-linux:
  lib/pager: Translate error message
  lib/pager: Always reset signals before exit
  lib/pager: Properly handle fork error return value

docs: getopt: improve arg example in getopt-example

The option `wow!*\?` in original example is not the argument of
`--c-long`. Only `=` separated in optional option is allowed.

[karelzak@redhat.com: - fix tcsh quote style to use backtick-quote
 matching actual script output, remove orphaned comment about
 exclamation mark escaping]

Based-on-patch-by: Ben Song <bensongsyz@gmail.com>
Addresses: https://github.com/util-linux/util-linux/pull/4088
Signed-off-by: Karel Zak <kzak@redhat.com>

Merge branch 'patch-27' of https://github.com/mariobl/util-linux

* 'patch-27' of https://github.com/mariobl/util-linux:
  Put file types in namei.1.adoc in a table structure
  Convert code blocks into tables
  Remove backticks from table in terminal-colors.d.5.adoc

# Conflicts:
# lib/terminal-colors.d.5.adoc

lslogins: fix broken GROUP column output

The condition on COL_GROUP incorrectly checked `!grp->gr_name`
instead of `!user->group`. This caused the GROUP column to always
be empty because when gr_name is non-NULL (normal case) the
assignment was skipped, and when gr_name is NULL, xstrdup() would
crash.

Fixes: https://github.com/util-linux/util-linux/issues/4097
Fixes: 52a6e45bfe5ddee1fed20f2f4f7542cac6bf13c9
Signed-off-by: Karel Zak <kzak@redhat.com>

pam_lastlog2: add -lpam to Makemodule.am

If we don't add this, we don't actually link with PAM; compare the
before and after of pam_lastlog2.so -- without -lpam it does not list
libpam.so.0 as a NEEDED dependency.

Signed-off-by: Morgan Jones <me@numin.it>
Signed-off-by: Karel Zak <kzak@redhat.com>

terminal-colors.d: (man) do not show 'type' as an optional part

That is: remove the square brackets from around 'type' in the synopsis.

Also, do not give the impression that a leading dot by itself is fine
before the 'type'.  That is: a dot is required only when 'name' and/or
'@term' is present.

Also, do not colorize the square brackets as if they were part of the
placeholders.  (And use ++double plus++ passthroughs for the opening
square brackets, to prevent asciidoctor from misinterpreting them.)

Indent the list of file types, for clarity.

And correct or improve some wordings, and remove an inconvenient
blank line in an example.

Signed-off-by: Benno Schulenberg <bensberg@telfort.nl>

findfs: (man) improve the markup, the layout, and the wording

Mark NAME as a placeholder instead of a literal, and rename it to
TAGNAME to make it clearer what "tag" in the description refers to.

Indent the list of possible tags, to make it clearer where the
description continues.

Drop the angle brackets around placeholders -- those are used
in --help output, but in man pages it is just italics.

Also add a missing section header.

Signed-off-by: Benno Schulenberg <bensberg@telfort.nl>

lib/meson: move caputils.c from lib_common to per-binary sources

caputils.c is GPL-licensed, and libcommon should only contain LGPL
or public domain code (as stated in lib/Makemodule.am). Move it out
of lib_common_sources and add it directly to the sources of binaries
that use it (unshare, nsenter, setpriv), matching autotools.

Addresses: https://github.com/util-linux/util-linux/issues/4085
Signed-off-by: Karel Zak <kzak@redhat.com>

lib/meson: remove unconditional procfs.c from lib_common_sources

procfs.c was listed in the base lib_common_sources and also added
conditionally under HAVE_OPENAT/HAVE_DIRFD (line 88). In autotools,
procfs.c is only added conditionally. Remove it from the base list
to match autotools and avoid duplication.

Addresses: https://github.com/util-linux/util-linux/issues/4085
Signed-off-by: Karel Zak <kzak@redhat.com>

lib/meson: remove duplicated source files from lib_common_sources

The files idcache.c, randutils.c, md5.c, sha1.c, strutils.c, and
strv.c were listed both in the base lib_common_sources string list
and then appended again via files() objects. Remove them from the
base list since the files() append is needed for the named variables
used by other meson.build files.

Addresses: https://github.com/util-linux/util-linux/issues/4085
Signed-off-by: Karel Zak <kzak@redhat.com>

pg: Reset SIGCHLD to default

When pg runs a child process, i.e. evaluating an "!" command, it calls
wait to receive status for the child.

If SIGCHLD is ignored, this results in an endless loop. Fix this by
resetting SIGCHLD to default before the any fork() call is performed.

Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>

The incorrect order of fcntl() parameters resulted in the inability to clear O_NONBLOCK

Signed-off-by: Leefancy <lijian01@kylinos.cn>

eject: Reset SIGCHLD to default for wait

If SIGCHLD is ignored, the wait call which waits for the umount command
to finish could fail with -1 and ECHILD. Since its return value is not
checked, the uninitialized status variable would be read, leading to
undefined behavior.

While at it, removed the additionally printed newline from errx message.

Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>

eject: Fix OOB access with empty argument

The the supplied argument is an empty string, eject triggers an out of
boundary access due to strlen() - 1 calculation.

Verify that the string is not empty before checking its last character.

Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>

fincore: should add break here

Put file types in namei.1.adoc in a table structure

lib/pager: Translate error message

Since the error message is not printed by a signal handler anymore, it
can be translated.

Requested-by: Karel Zak <kzak@redhat.com>
Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>

lib/pager: Always reset signals before exit

Just in case another exit handler relies on these signal handlers being
set to default, or to allow users to press ^C and let the program exit
as expected.

Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>

Merge branch 'mkfs_minix_alarm' of https://github.com/stoeckmann/util-linux

* 'mkfs_minix_alarm' of https://github.com/stoeckmann/util-linux:
  mkfs.minix: Simplify SIGALRM handler

Merge branch 'PR/portability-hurd' of https://github.com/karelzak/util-linux-work

* 'PR/portability-hurd' of https://github.com/karelzak/util-linux-work:
  agetty: simplify USE_SYSTEMD ifdef in output_special_char
  agetty: introduce USE_NETLINK to separate netlink from reload
  agetty: guard netlink-related code with AGETTY_RELOAD
  libblkid: guard loop device code with __linux__ in cache
  build-sys: move shells.c to common (non-Linux) sources
  build-sys: move netlink sources from libcommon to agetty

Merge branch 'PR/lsmem-sysroot-prefix' of https://github.com/karelzak/util-linux-work

* 'PR/lsmem-sysroot-prefix' of https://github.com/karelzak/util-linux-work:
  lsmem: apply --sysroot prefix to all sysfs paths

lsmem: apply --sysroot prefix to all sysfs paths

Tests fail on architectures where /sys/module/memory_hotplug/parameters/
memmap_on_memory does not exist (e.g., loong64), because lsmem reads
from the real /sys instead of the --sysroot directory.

 - Route memmap_on_memory read through ul_path API with a local handler
   in print_summary(), so --sysroot prefix is respected.

 - Apply --sysroot prefix to sysmemconfig (/sys/firmware/memory) before
   the memory0 existence check.

 - Fix sysmemconfig memory leak (missing ul_unref_path).

 - Update expected test output (remove "Memmap on memory parameter"
   lines, not available in test dumps).

Reported-by: Chris Hofstaedtler <zeha@debian.org>
Signed-off-by: Karel Zak <kzak@redhat.com>
Tested-by: Sumanth Korikkar <sumanthk@linux.ibm.com>

Convert code blocks into tables

lib/pager: Properly handle fork error return value

Most of the code checks for pager_process.pid != 0 to see if a pager is
running. If fork fails and returns -1, pager_process.pid is not 0, too.

This in turn leads to an issue if waitpid is eventually called with -1
as pid, which would wait for all children. Since there are no children,
the wait_for_pager function calls err with an EXIT_FAILURE value.

Properly handle a potential fork error.

Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>

mkfs.minix: Simplify SIGALRM handler

Calling printf is not signal safe and in general there are too many
branches in alarm_intr for what it's used for.

Check the signal flag outside of the handler and print messages when
needed. Also, disable the alarm and reset the signal handling when it's
not needed anymore.

While at it, use correct formatter in printf.

Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>

tools/git-grouped-log: add co-author support

Parse Co-authored-by and Co-Author trailers from commit bodies and
display co-author names alongside the primary author.

Signed-off-by: Karel Zak <kzak@redhat.com>

agetty: simplify USE_SYSTEMD ifdef in output_special_char

Remove unnecessary duplicate #ifdef USE_SYSTEMD around the else
branch for utmpx-based user counting. Use the same pattern as
USE_NETLINK where the else and opening brace are split across the
endif boundary.

Signed-off-by: Karel Zak <kzak@redhat.com>

agetty: introduce USE_NETLINK to separate netlink from reload

The netlink functionality (network address display in issue file)
and the reload mechanism (inotify-based) are separate features that
shared the same AGETTY_RELOAD ifdef guard. Additionally, the netlink
code in output_special_char() used a different guard
(#if defined(RTMGRP_IPV4_IFADDR) && defined(RTMGRP_IPV6_IFADDR))
creating inconsistency.

Introduce USE_NETLINK, defined when AGETTY_RELOAD is enabled and
the required RTMGRP constants are available. Use it consistently
for all netlink-specific code (nl field, helper functions, socket
handling, escape sequences \4, \6, \a, \A).

Keep AGETTY_RELOAD for the inotify/reload mechanism only.

Signed-off-by: Karel Zak <kzak@redhat.com>

agetty: guard netlink-related code with AGETTY_RELOAD

The struct issue nl field and related helper functions use types
and headers (netaddrq.h, linux/netlink.h) that are only available
when AGETTY_RELOAD is defined. Guard the .nl.fd initializations
in main() and show_issue(), and wrap the netlink helper functions
(print_iface_best, print_addrq_bestofall, dump_iface_good,
dump_iface_all) with #ifdef AGETTY_RELOAD.

This fixes compilation on systems without inotify/netlink support
(e.g., GNU/Hurd).

Co-authored-by: Samuel Thibault <samuel.thibault@aquilenet.fr>
Signed-off-by: Karel Zak <kzak@redhat.com>

libblkid: guard loop device code with __linux__ in cache

The is_loopdev() and loopdev_has_backing_file() functions from
loopdev.h are Linux-specific. Guard the loop device cache cleanup
in blkid_gc_cache() with __linux__ to allow compilation on
non-Linux systems like GNU/Hurd.

Co-authored-by: Samuel Thibault <samuel.thibault@aquilenet.fr>
Signed-off-by: Karel Zak <kzak@redhat.com>

build-sys: move shells.c to common (non-Linux) sources

shells.c uses POSIX setusershell()/getusershell()/endusershell()
which are available on non-Linux systems (e.g., GNU/Hurd). Move it
from the Linux-only section to common libcommon sources.

The meson build already had shells.c in common sources, so only
the autotools build needed fixing.

Co-authored-by: Samuel Thibault <samuel.thibault@aquilenet.fr>
Signed-off-by: Karel Zak <kzak@redhat.com>

build-sys: move netlink sources from libcommon to agetty

The netlink and netaddrq sources use Linux-specific headers
(linux/netlink.h, linux/rtnetlink.h) and cannot be compiled on
non-Linux systems like GNU/Hurd.

Move them out of libcommon and link directly into agetty (the only
user), following the same pattern as plymouth-ctrl.c. Also move the
corresponding test programs to the Linux-only section.

Co-authored-by: Samuel Thibault <samuel.thibault@aquilenet.fr>
Signed-off-by: Karel Zak <kzak@redhat.com>

Remove backticks from table in terminal-colors.d.5.adoc

Merge branch 'pager_git' of https://github.com/stoeckmann/util-linux

* 'pager_git' of https://github.com/stoeckmann/util-linux:
  lib/pager: Drop less workaround
  lib/pager: Add LV support
  lib/pager: Do not overwrite LESS environment var

Merge branch 'libblkid/parent-offset' of https://github.com/t-8ch/util-linux

* 'libblkid/parent-offset' of https://github.com/t-8ch/util-linux:
  libblkid: (probe) Account for IO bias when retrieving buffer from parent

Merge branch 'hardlink_arithmetic' of https://github.com/stoeckmann/util-linux

* 'hardlink_arithmetic' of https://github.com/stoeckmann/util-linux:
  lib/fileeq: Check arithmetic in ul_fileeq_set_size

build(deps): bump actions/upload-artifact from 6 to 7

Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 6 to 7.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](https://github.com/actions/upload-artifact/compare/v6...v7)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-version: '7'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

libblkid: (probe) Account for IO bias when retrieving buffer from parent

The 'off' variable gets aligned to 'pr->io_size'. However when
delegating to the parent prober, the value for 'off' from the original
function parameter needs to be used.

Undo the bias before calling into the parent prober.

Fixes: 8a534253bc52 ("libblkid: (probe) read data in chunks")
Closes: https://github.com/util-linux/util-linux/issues/4078
Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

man: hide LIBSMARTCOLS_JSON in man pages that do not support --json

PR #4070 consolidated libsmartcols env vars into man-common/env-smartcols.adoc
and had all tools (including fdisk and cfdisk) include it. That documented
LIBSMARTCOLS_JSON in fdisk/cfdisk man pages although those tools do not
support --json output, which is misleading.

Use ifdef::smartcols-json[] in env-smartcols.adoc so the JSON env var
section is only rendered for man pages that set :smartcols-json: (column,
lsblk, findmnt, sfdisk). fdisk and cfdisk include the same file but do not
set the attribute, so they only show the debug-related variables.

Addresses: util-linux#4070

lib/pager: Drop less workaround

This was fixed in less back in 2007. Let's follow git and drop the
workaround.

Taken from https://github.com/git/git/commit/e8320f350f523c3219ff8ec639663193941af57d

Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>

lib/pager: Add LV support

Set LV environment variable to "-c" for lv support.

Taken from https://github.com/git/git/commit/e54c1f2d2533c5406abeb8e3e0cf78c68ca9c21e

Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>

lib/pager: Do not overwrite LESS environment var

If LESS is already set, do not overwrite it.

Taken from https://github.com/git/git/commit/25fc1786ab147daa9ffa43ebcbaf5c1cc6e50d4f

Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>

lib/fileeq: Check arithmetic in ul_fileeq_set_size

Make sure that arithmetics do not overflow data types. Such overflows
could occur with large hardlink options or on 32 bit systems with large
files (due to size_t usage).

If possible, reduce sizes so operations can continue successfully.

Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>

tests/lsfd: skip executable mode tests when not running as root

This test does not necessarily work without root.

Reported-by: Christian Hesse <mail@eworm.de>
Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

meson: define _GNU_SOURCE when testing for 'struct statx'

glibc (at least with 2.43) requires _GNU_SOURCE to be defined to expose
'struct statx' from 'sys/stat.h'.

Define _GNU_SOURCE in the feature test, so 'struct statx' is correctly
detected. The code is being build with _GNU_SOURCE anyways.

With autotools this happens automatically.

Reported-by: Christian Hesse <mail@eworm.de>
Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

build-sys: update release dates

Signed-off-by: Karel Zak <kzak@redhat.com>

po: merge changes

po-man: merge changes

tools/git-tp-sync: fix po/ update

Signed-off-by: Karel Zak <kzak@redhat.com>

tools/checkconfig: strip C comments before macro extraction

Strip C/C++ comments (// line comments, single-line /* */ and
multi-line /* */ blocks) before extracting HAVE_ and ENABLE_ macros.
This avoids false positives from macros mentioned in comments, e.g.
"when HAVE_PIDFD_* ..." in include/pidfd-utils.h.

Signed-off-by: Karel Zak <kzak@redhat.com>

include/path: fix HAVE_STRUCT_STATX use

Signed-off-by: Karel Zak <kzak@redhat.com>

lib/procfs: remove duplicate statfs_magic.h include

Signed-off-by: Karel Zak <kzak@redhat.com>

include/mountutils: remove duplicate inttypes.h include

Signed-off-by: Karel Zak <kzak@redhat.com>

tools: git-version-bump, accept X.Y-devel version on master branch

Allow "X.Y-devel" as a valid version format, restricted to the master
branch only. Release versions (X.Y, X.Y.Z, X.Y-rcN, X.Y.Z-rcN) still
require a stable/vX.Y branch.

Signed-off-by: Karel Zak <kzak@redhat.com>