[kamailio/kamailio] debian pkg missing dependency on docbook2x (#1624)

List overview All Threads
Download

newer

older

git:master:0d2dd5b0: uac: remove...

[kamailio/kamailio] Module...

juha-h

23 Aug 2018 23 Aug '18

9:01 a.m.

Kamailio master build on Debian Stretch fails due to missing dependency on docbook2x. See https://lists.kamailio.org//pipermail/sr-dev/2018-August/047549.html for more info.

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/1624

Attachments:

attachment.html (text/html — 3.4 KB)

Show replies by date

Victor Seva

23 Aug 23 Aug

10:11 a.m.

We have no problems with the nightly builds, example: https://kamailio.sipwise.com/job/kamailiodev-nightly-binaries/architecture=a...

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/1624#issuecomment-415364350

juha-h

12:10 p.m.

Victor Seva writes:

...

We have no problems with the nightly builds, example: https://kamailio.sipwise.com/job/kamailiodev-nightly-binaries/architecture=a...

For sure you have. Exactly the same problem that I described:

mkdir -p /tmp/buildd/kamailio-5.2.0~dev6+0~20180823005931.1186+stretch/debian/kamailio/usr/share/man//man5 s ../../../doc/stylesheets/serdoc2man.xsl auth.xml make[3]: s: Command not found ../../Makefile.modules:283: recipe for target 'auth.7' failed make[3]: [auth.7] Error 127 (ignored) s ../../../doc/stylesheets/serdoc2man.xsl avp.xml make[3]: s: Command not found ../../Makefile.modules:283: recipe for target 'avp.7' failed make[3]: [avp.7] Error 127 (ignored)

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/1624#issuecomment-415392440

juha-h

31 Aug 31 Aug

8:01 a.m.

This clearly is a bug. I don't understand why bug label was removed. Also, I my opinion this kind of error should not be ignored, but build should stop at it.

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/1624#issuecomment-417585090

Daniel-Constantin Mierla

9:51 a.m.

Overall this is not actually deb build specific issue, but something that happens during install, as I could spot during building and installing using git repo inside a docker container (for kamailio unit tests). I guess that was more an optional step, attempting to build man pages from docbooks for modules readmes.

I think this should be made optional properly. I do not think anyone was using manpage-style docs so far for modules and starting now to install/package them all makes no sense.

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/1624#issuecomment-417614166

juha-h

10:07 a.m.

Daniel-Constantin Mierla writes:

...

Optional is fine with me. I just want the build to work without errors. Also, if an error occurs, the build must stop, This is not the case now.

-- Juha

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/1624#issuecomment-417618299

Daniel-Constantin Mierla

11 a.m.

Pushed a patch to catch when the tool from docbook to mack conversion is not available and just print an info message. I think this should do it for now, if one wants to start generating man pages, can do it by installing the tool.

I will close this issue and if one wants to package man pages for modules' readme, open a feature request. To my knowledge, so far they were never packaged. If some new issues after the patch, just reopen.

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/1624#issuecomment-417630049

Daniel-Constantin Mierla

11 a.m.

Closed #1624.

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/1624#event-1820321904

juha-h

2:52 p.m.

Daniel-Constantin Mierla writes:

...

Thanks, debian package now built OK without docbook2x.

-- Juha

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/1624#issuecomment-417688685

2447

Age (days ago)

2455

Last active (days ago)

sr-dev@lists.kamailio.org

8 comments

3 participants

tags (0)

participants (3)

Daniel-Constantin Mierla
juha-h
Victor Seva