ldconfig: trace origin paths with -v
Commit Message
Joseph Myers <joseph@codesourcery.com> writes:
> This looks like it's a message, for human readers rather than for
There were two; the other was just "(from %s:%d)". Marked. Anything
else? I note that the "(hwcap ...)" string is NOT marked up but that
may be... well, more correct than not.
From 7a59c4ddc48db342140e65a80bc5a00560d380d2 Mon Sep 17 00:00:00 2001
From: DJ Delorie <dj@redhat.com>
Date: Wed, 19 Feb 2020 12:31:38 -0500
Subject: ldconfig: trace origin paths with -v
With this patch, -v turns on a "from" trace for each directory
searched, that tells you WHY that directory is being searched -
is it a builtin, from the command line, or from some config file?
Comments
On 2/19/20 3:45 PM, DJ Delorie wrote:
> Joseph Myers <joseph@codesourcery.com> writes:
>> This looks like it's a message, for human readers rather than for
>
> There were two; the other was just "(from %s:%d)". Marked. Anything
> else? I note that the "(hwcap ...)" string is NOT marked up but that
> may be... well, more correct than not.
>
> From 7a59c4ddc48db342140e65a80bc5a00560d380d2 Mon Sep 17 00:00:00 2001
> From: DJ Delorie <dj@redhat.com>
> Date: Wed, 19 Feb 2020 12:31:38 -0500
> Subject: ldconfig: trace origin paths with -v
>
> With this patch, -v turns on a "from" trace for each directory
> searched, that tells you WHY that directory is being searched -
> is it a builtin, from the command line, or from some config file?
>
LTGM. FAOD I do not think <builtin> and <cmdline> need translation.
Reviewed-by: Carlos O'Donell <carlos@redhat.com>
> diff --git a/elf/ldconfig.c b/elf/ldconfig.c
> index 681ed78496..71178d416b 100644
> --- a/elf/ldconfig.c
> +++ b/elf/ldconfig.c
> @@ -79,6 +79,8 @@ struct dir_entry
> int flag;
> ino64_t ino;
> dev_t dev;
> + const char *from_file;
> + int from_line;
OK.
> struct dir_entry *next;
> };
>
> @@ -344,7 +346,12 @@ add_single_dir (struct dir_entry *entry, int verbose)
> if (ptr->ino == entry->ino && ptr->dev == entry->dev)
> {
> if (opt_verbose && verbose)
> - error (0, 0, _("Path `%s' given more than once"), entry->path);
> + {
> + error (0, 0, _("Path `%s' given more than once"), entry->path);
> + fprintf (stderr, _("(from %s:%d and %s:%d)\n"),
> + entry->from_file, entry->from_line,
> + ptr->from_file, ptr->from_line);
OK, use the file and line data for an effective message.
> + }
> /* Use the newer information. */
> ptr->flag = entry->flag;
> free (entry->path);
> @@ -363,12 +370,15 @@ add_single_dir (struct dir_entry *entry, int verbose)
>
> /* Add one directory to the list of directories to process. */
> static void
> -add_dir (const char *line)
> +add_dir_1 (const char *line, const char *from_file, int from_line)
> {
> unsigned int i;
> struct dir_entry *entry = xmalloc (sizeof (struct dir_entry));
> entry->next = NULL;
>
> + entry->from_file = strdup (from_file);
> + entry->from_line = from_line;
OK.
> +
> /* Search for an '=' sign. */
> entry->path = xstrdup (line);
> char *equal_sign = strchr (entry->path, '=');
> @@ -428,6 +438,11 @@ add_dir (const char *line)
> free (path);
> }
>
> +static void
> +add_dir (const char *line)
> +{
> + add_dir_1 (line, "<builtin>", 0);
> +}
My opinion is that this doesn't need translation since it's a technical
reference to a specific type of entry.
>
> static int
> chroot_stat (const char *real_path, const char *path, struct stat64 *st)
> @@ -672,9 +687,10 @@ search_dir (const struct dir_entry *entry)
> if (opt_verbose)
> {
> if (hwcap != 0)
> - printf ("%s: (hwcap: %#.16" PRIx64 ")\n", entry->path, hwcap);
> + printf ("%s: (hwcap: %#.16" PRIx64 ")", entry->path, hwcap);
> else
> - printf ("%s:\n", entry->path);
> + printf ("%s:", entry->path);
> + printf (_(" (from %s:%d)\n"), entry->from_file, entry->from_line);
OK.
> }
>
> char *dir_name;
> @@ -815,6 +831,8 @@ search_dir (const struct dir_entry *entry)
> struct dir_entry *new_entry;
>
> new_entry = xmalloc (sizeof (struct dir_entry));
> + new_entry->from_file = entry->from_file;
> + new_entry->from_line = entry->from_line;
OK.
> new_entry->path = xstrdup (file_name);
> new_entry->flag = entry->flag;
> new_entry->next = NULL;
> @@ -1175,7 +1193,7 @@ Warning: ignoring configuration file that cannot be opened: %s"),
> }
> }
> else
> - add_dir (cp);
> + add_dir_1 (cp, filename, lineno);
OK.
> }
> while (!feof_unlocked (file));
>
> @@ -1283,7 +1301,7 @@ main (int argc, char **argv)
> _("relative path `%s' used to build cache"),
> argv[i]);
> else
> - add_dir (argv[i]);
> + add_dir_1 (argv[i], "<cmdline>", 0);
My opinion is that this doesn't need translation since it's a technical
reference to a specific type of entry.
> }
>
> /* The last entry in hwcap_extra is reserved for the "tls" pseudo-hwcap which
>
@@ -79,6 +79,8 @@ struct dir_entry
int flag;
ino64_t ino;
dev_t dev;
+ const char *from_file;
+ int from_line;
struct dir_entry *next;
};
@@ -344,7 +346,12 @@ add_single_dir (struct dir_entry *entry, int verbose)
if (ptr->ino == entry->ino && ptr->dev == entry->dev)
{
if (opt_verbose && verbose)
- error (0, 0, _("Path `%s' given more than once"), entry->path);
+ {
+ error (0, 0, _("Path `%s' given more than once"), entry->path);
+ fprintf (stderr, _("(from %s:%d and %s:%d)\n"),
+ entry->from_file, entry->from_line,
+ ptr->from_file, ptr->from_line);
+ }
/* Use the newer information. */
ptr->flag = entry->flag;
free (entry->path);
@@ -363,12 +370,15 @@ add_single_dir (struct dir_entry *entry, int verbose)
/* Add one directory to the list of directories to process. */
static void
-add_dir (const char *line)
+add_dir_1 (const char *line, const char *from_file, int from_line)
{
unsigned int i;
struct dir_entry *entry = xmalloc (sizeof (struct dir_entry));
entry->next = NULL;
+ entry->from_file = strdup (from_file);
+ entry->from_line = from_line;
+
/* Search for an '=' sign. */
entry->path = xstrdup (line);
char *equal_sign = strchr (entry->path, '=');
@@ -428,6 +438,11 @@ add_dir (const char *line)
free (path);
}
+static void
+add_dir (const char *line)
+{
+ add_dir_1 (line, "<builtin>", 0);
+}
static int
chroot_stat (const char *real_path, const char *path, struct stat64 *st)
@@ -672,9 +687,10 @@ search_dir (const struct dir_entry *entry)
if (opt_verbose)
{
if (hwcap != 0)
- printf ("%s: (hwcap: %#.16" PRIx64 ")\n", entry->path, hwcap);
+ printf ("%s: (hwcap: %#.16" PRIx64 ")", entry->path, hwcap);
else
- printf ("%s:\n", entry->path);
+ printf ("%s:", entry->path);
+ printf (_(" (from %s:%d)\n"), entry->from_file, entry->from_line);
}
char *dir_name;
@@ -815,6 +831,8 @@ search_dir (const struct dir_entry *entry)
struct dir_entry *new_entry;
new_entry = xmalloc (sizeof (struct dir_entry));
+ new_entry->from_file = entry->from_file;
+ new_entry->from_line = entry->from_line;
new_entry->path = xstrdup (file_name);
new_entry->flag = entry->flag;
new_entry->next = NULL;
@@ -1175,7 +1193,7 @@ Warning: ignoring configuration file that cannot be opened: %s"),
}
}
else
- add_dir (cp);
+ add_dir_1 (cp, filename, lineno);
}
while (!feof_unlocked (file));
@@ -1283,7 +1301,7 @@ main (int argc, char **argv)
_("relative path `%s' used to build cache"),
argv[i]);
else
- add_dir (argv[i]);
+ add_dir_1 (argv[i], "<cmdline>", 0);
}
/* The last entry in hwcap_extra is reserved for the "tls" pseudo-hwcap which