Linux-Modules Archive on lore.kernel.org
 help / Atom feed
* Build fails when Xsltproc not present
@ 2016-12-20  0:50 Bryan Henderson
  2017-02-23 22:16 ` Lucas De Marchi
  0 siblings, 1 reply; 7+ messages in thread
From: Bryan Henderson @ 2016-12-20  0:50 UTC (permalink / raw)
  To: linux-modules

I found in building kmod 23 that because my system does not have
Xsltproc installed, the build fails when it tries to use a null string
as the Xsltproc program name to create the man pages.

I believe in that case, it should just skip building the man pages.

Or make 'configure' fail and suggest --disable-manpages.

Or make the make fail gracefully, telling you you don't have the tools
necessary to create man pages and suggesting --disable-manpages.

-- 
Bryan Henderson                                   San Jose, California

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: Build fails when Xsltproc not present
  2016-12-20  0:50 Build fails when Xsltproc not present Bryan Henderson
@ 2017-02-23 22:16 ` Lucas De Marchi
  2017-02-24  3:13   ` Lucas De Marchi
  0 siblings, 1 reply; 7+ messages in thread
From: Lucas De Marchi @ 2017-02-23 22:16 UTC (permalink / raw)
  To: Bryan Henderson; +Cc: linux-modules

On Mon, Dec 19, 2016 at 4:50 PM, Bryan Henderson
<bryanh@giraffe-data.com> wrote:
>
> I believe in that case, it should just skip building the man pages.

We don't do auto-enabling/disabling.

> Or make 'configure' fail and suggest --disable-manpages.
>
> Or make the make fail gracefully, telling you you don't have the tools
> necessary to create man pages and suggesting --disable-manpages.

Failing on configure phase would be the desired solution. I'll take a
look on this for next release, but if  you have a patch ready it would
be appreciated.


thanks
Lucas De Marchi

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: Build fails when Xsltproc not present
  2017-02-23 22:16 ` Lucas De Marchi
@ 2017-02-24  3:13   ` Lucas De Marchi
  2017-02-24  3:17     ` Bryan Henderson
  0 siblings, 1 reply; 7+ messages in thread
From: Lucas De Marchi @ 2017-02-24  3:13 UTC (permalink / raw)
  To: Bryan Henderson; +Cc: linux-modules

On Thu, Feb 23, 2017 at 2:16 PM, Lucas De Marchi
<lucas.de.marchi@gmail.com> wrote:
> On Mon, Dec 19, 2016 at 4:50 PM, Bryan Henderson
> <bryanh@giraffe-data.com> wrote:
>>
>> I believe in that case, it should just skip building the man pages.
>
> We don't do auto-enabling/disabling.
>
>> Or make 'configure' fail and suggest --disable-manpages.
>>
>> Or make the make fail gracefully, telling you you don't have the tools
>> necessary to create man pages and suggesting --disable-manpages.
>
> Failing on configure phase would be the desired solution. I'll take a
> look on this for next release, but if  you have a patch ready it would
> be appreciated.

Actually I did it on make because man pages are  with the distributed tarball

Lucas De Marchi

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: Build fails when Xsltproc not present
  2017-02-24  3:13   ` Lucas De Marchi
@ 2017-02-24  3:17     ` Bryan Henderson
  2017-02-24  4:03       ` Lucas De Marchi
  0 siblings, 1 reply; 7+ messages in thread
From: Bryan Henderson @ 2017-02-24  3:17 UTC (permalink / raw)
  To: Lucas De Marchi; +Cc: linux-modules

>Actually I did it on make because man pages are  with the distributed tarball

I'm not sure I understand this.  Are you saying a person doesn't need Xsltproc
to do a default build because the man pages are already built?  Then I wonder
why I encountered this problem.  I had no reason to want to rebuild something
that was distributed.

-- 
Bryan Henderson                                   San Jose, California

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: Build fails when Xsltproc not present
  2017-02-24  3:17     ` Bryan Henderson
@ 2017-02-24  4:03       ` Lucas De Marchi
  2017-02-24 16:32         ` Bryan Henderson
  0 siblings, 1 reply; 7+ messages in thread
From: Lucas De Marchi @ 2017-02-24  4:03 UTC (permalink / raw)
  To: Bryan Henderson; +Cc: linux-modules

On Thu, Feb 23, 2017 at 7:17 PM, Bryan Henderson
<bryanh@giraffe-data.com> wrote:
>>Actually I did it on make because man pages are  with the distributed tarball
>
> I'm not sure I understand this.  Are you saying a person doesn't need Xsltproc
> to do a default build because the man pages are already built?  Then I wonder
> why I encountered this problem.  I had no reason to want to rebuild something
> that was distributed.

They are already built on the distributed tarball, those coming from
https://www.kernel.org/pub/linux/utils/kernel/kmod/
If you are building from the git repository then yes, you need
xsltproc. I pushed this patch:

https://git.kernel.org/cgit/utils/kernel/kmod/kmod.git/commit/?id=72167ba113c00eea6eb73cd91ce173e5c734564b

So now when building from git it will say xsltproc is missing

Lucas De Marchi

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: Build fails when Xsltproc not present
  2017-02-24  4:03       ` Lucas De Marchi
@ 2017-02-24 16:32         ` Bryan Henderson
  2017-02-26 23:09           ` Bryan Henderson
  0 siblings, 1 reply; 7+ messages in thread
From: Bryan Henderson @ 2017-02-24 16:32 UTC (permalink / raw)
  To: Lucas De Marchi; +Cc: linux-modules

>They are already built on the distributed tarball, those coming from
>https://www.kernel.org/pub/linux/utils/kernel/kmod/
>If you are building from the git repository then yes, you need
>xsltproc. I pushed this patch:

Well, I built from the tarball, which I still have and see contains
e.g. man/kmod.8, and yet the build failed on me.  I will try to reproduce it
and see what happened, but looking at my local patch to the package, I see
that I 1) used --disable-manpages and 2) modified libkmod/docs/Makefile.in to
prevent that directory from building.  There are no man pages in libkmod/docs/
.

Your change is still right, of course.  The make files should be able to
tolerate these null values.

-- 
Bryan Henderson                                   San Jose, California

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: Build fails when Xsltproc not present
  2017-02-24 16:32         ` Bryan Henderson
@ 2017-02-26 23:09           ` Bryan Henderson
  0 siblings, 0 replies; 7+ messages in thread
From: Bryan Henderson @ 2017-02-26 23:09 UTC (permalink / raw)
  To: lucas.de.marchi; +Cc: linux-modules

>Well, I built from the tarball, which I still have and see contains
>e.g. man/kmod.8, and yet the build failed on me.  I will try to reproduce it
>and see what happened

OK, I figured it out.

'make clean' deletes the distributed man page files.  Neither 'make clean' nor
'make distclean' should delete distributed files, but 'make maintainer-clean'
should.

And while we're looking at clean, 'make clean' does _not_ delete the built
files 'rmmod', etc. from the tools/ directory, and it should.

-- 
Bryan Henderson                                   San Jose, California

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, back to index

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-12-20  0:50 Build fails when Xsltproc not present Bryan Henderson
2017-02-23 22:16 ` Lucas De Marchi
2017-02-24  3:13   ` Lucas De Marchi
2017-02-24  3:17     ` Bryan Henderson
2017-02-24  4:03       ` Lucas De Marchi
2017-02-24 16:32         ` Bryan Henderson
2017-02-26 23:09           ` Bryan Henderson

Linux-Modules Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-modules/0 linux-modules/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-modules linux-modules/ https://lore.kernel.org/linux-modules \
		linux-modules@vger.kernel.org linux-modules@archiver.kernel.org
	public-inbox-index linux-modules


Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-modules


AGPL code for this site: git clone https://public-inbox.org/ public-inbox