Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Fri, 23 Jul 2021 19:28:12 -0300
User-Agent: K-9 Mail for Android
In-Reply-To: <CAP-5=fWUr91jLgD-BDYK08v0ELQtypFHDop7QW4uFOPscncCQw@mail.gmail.com>
References: <20210715013343.2286699-1-irogers@google.com> <20210715013343.2286699-5-irogers@google.com> <YPsKLa0Qe5/as4Jh@kernel.org> <YPsQGZOXqoy7Tfe6@kernel.org> <YPsQ5En9N7xG1Cya@kernel.org> <CAP-5=fWUr91jLgD-BDYK08v0ELQtypFHDop7QW4uFOPscncCQw@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain;
 charset=utf-8
Content-Transfer-Encoding: quoted-printable
Subject: Re: [PATCH 4/7] perf doc: Add info pages to all target.
To:     Ian Rogers <irogers@google.com>,
        Arnaldo Carvalho de Melo <acme@kernel.org>
CC:     Peter Zijlstra <peterz@infradead.org>,
        Ingo Molnar <mingo@redhat.com>,
        Mark Rutland <mark.rutland@arm.com>,
        Alexander Shishkin <alexander.shishkin@linux.intel.com>,
        Jiri Olsa <jolsa@redhat.com>,
        Namhyung Kim <namhyung@kernel.org>,
        linux-perf-users@vger.kernel.org, linux-kernel@vger.kernel.org,
        eranian@google.com
From:   Arnaldo <arnaldo.melo@gmail.com>
Message-ID: <956E2E1D-4F91-4C92-B06B-21FC8DE376C0@gmail.com>
Precedence: bulk


On July 23, 2021 6:03:16 PM GMT-03:00, Ian Rogers <irogers@google=2Ecom> w=
rote:
>On Fri, Jul 23, 2021 at 11:56 AM Arnaldo Carvalho de Melo
><acme@kernel=2Eorg> wrote:
>>
>> Em Fri, Jul 23, 2021 at 03:53:14PM -0300, Arnaldo Carvalho de Melo
>escreveu:
>> > Em Fri, Jul 23, 2021 at 03:27:57PM -0300, Arnaldo Carvalho de Melo
>escreveu:
>> > > Em Wed, Jul 14, 2021 at 06:33:40PM -0700, Ian Rogers escreveu:
>> > > > Enabled to ensure that info pages build=2E
>> > >
>> > > How did you test this? I tried installing texinfo, that is where
>> > > makeinfo belongs:
>> > >
>> > > =E2=AC=A2[acme@toolbox perf]$ rpm -qf /usr/bin/makeinfo
>> > > texinfo-6=2E7-10=2Efc34=2Ex86_64
>> > > =E2=AC=A2[acme@toolbox perf]$ rpm -q texinfo
>> > > texinfo-6=2E7-10=2Efc34=2Ex86_64
>> > > =E2=AC=A2[acme@toolbox perf]$
>> > >
>> > > And then:
>> > >
>> > > =E2=AC=A2[acme@toolbox perf]$ make -C tools/perf O=3D/tmp/build/per=
f
>install-doc
>> > > make: Entering directory '/var/home/acme/git/perf/tools/perf'
>> > >   BUILD:   Doing 'make -j24' parallel build
>> > >   INSTALL Documentation-man
>> > > make: Leaving directory '/var/home/acme/git/perf/tools/perf'
>> > > =E2=AC=A2[acme@toolbox perf]$
>> >
>> > Also:
>>
>> We get closer if we do:
>>
>> =E2=AC=A2[acme@toolbox perf]$ make -C tools/perf/Documentation/ install=
-info
>> make: Entering directory
>'/var/home/acme/git/perf/tools/perf/Documentation'
>>   ASCIIDOC perf=2Exml
>>   DB2TEXI  perf=2Etexi
>> /bin/sh: line 2: docbook2x-texi: command not found
>> make: *** [Makefile:289: perf=2Etexi] Error 127
>> make: Leaving directory
>'/var/home/acme/git/perf/tools/perf/Documentation'
>> =E2=AC=A2[acme@toolbox perf]$
>
>Do you need to install docbook2x?

Sure, but it seems the name of the binaries in the version in fedora are d=
ifferent=2E

>
>$ dpkg -S /usr/bin/docbook2x-texi
>docbook2x: /usr/bin/docbook2x-texi
>
>Thanks,
>Ian
>
>> - Arnaldo

--=20
Sent from my Android device with K-9 Mail=2E Please excuse my brevity=2E