linux-kernel - Re: [PATCH] perf tools: Use $KBUILD_BUILD

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <ZFuVRK4cKapgaRwD@kernel.org>
Date:   Wed, 10 May 2023 09:59:48 -0300
From:   Arnaldo Carvalho de Melo <acme@...nel.org>
To:     Salvatore Bonaccorso <carnil@...ian.org>
Cc:     Ben Hutchings <ben@...adent.org.uk>,
        Peter Zijlstra <peterz@...radead.org>,
        Ingo Molnar <mingo@...hat.com>,
        Alexander Shishkin <alexander.shishkin@...ux.intel.com>,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH] perf tools: Use $KBUILD_BUILD_TIMESTAMP as man page date

Em Tue, May 09, 2023 at 09:03:28PM +0200, Salvatore Bonaccorso escreveu:
> Hi Peter, Ingo, Arnaldo,
> 
> On Tue, May 17, 2016 at 02:28:09PM +0100, Ben Hutchings wrote:
> > This allows man pages to be built reproducibly.
> > 
> > Signed-off-by: Ben Hutchings <ben@...adent.org.uk>
> > ---
> > --- a/tools/perf/Documentation/Makefile
> > +++ b/tools/perf/Documentation/Makefile
> > @@ -120,6 +120,9 @@ endif
> >  ifdef DOCBOOK_SUPPRESS_SP
> >  XMLTO_EXTRA += -m manpage-suppress-sp.xsl
> >  endif
> > +ifdef KBUILD_BUILD_TIMESTAMP
> > +ASCIIDOC_EXTRA += -a revdate=$(shell date -u -d '$(KBUILD_BUILD_TIMESTAMP)' +%Y-%m-%d)
> > +endif
> >  
> >  SHELL_PATH ?= $(SHELL)
> >  # Shell quote;
> > --- a/tools/perf/Documentation/asciidoc.conf
> > +++ b/tools/perf/Documentation/asciidoc.conf
> > @@ -71,6 +71,9 @@ ifdef::backend-docbook[]
> >  [header]
> >  template::[header-declarations]
> >  <refentry>
> > +<refentryinfo>
> > +template::[docinfo]
> > +</refentryinfo>
> >  <refmeta>
> >  <refentrytitle>{mantitle}</refentrytitle>
> >  <manvolnum>{manvolnum}</manvolnum>
> 
> As we do apply the change in Downstream in Debian, I was wondering if
> it just felt trough the cracks and could still be applied. Is there
> anything missing?

Isn't this fixed in:

commit d586ac10ce56b2381b8e1d8ed74660c1b2b8ab0d
Author: Ian Rogers <irogers@...gle.com>
Date:   Fri Sep 20 21:13:27 2019 -0700

    perf docs: Allow man page date to be specified

    With this change if a perf_date parameter is provided to asciidoc then
    it will override the default date written to the man page metadata.

    Without this change, or if the perf_date isn't specified, then the
    current date is written to the metadata.

    Having this parameter allows the metadata to be constant if builds
    happen on different dates.

    The name of the parameter is intended to be consistent with the existing
    perf_version parameter.

    Signed-off-by: Ian Rogers <irogers@...gle.com>
    Cc: Alexander Shishkin <alexander.shishkin@...ux.intel.com>
    Cc: Andi Kleen <ak@...ux.intel.com>
    Cc: Jiri Olsa <jolsa@...hat.com>
    Cc: Namhyung Kim <namhyung@...nel.org>
    Cc: Peter Zijlstra <peterz@...radead.org>
    Cc: Stephane Eranian <eranian@...gle.com>
    Link: http://lore.kernel.org/lkml/20190921041327.155054-1-irogers@google.com
    Signed-off-by: Arnaldo Carvalho de Melo <acme@...hat.com>