[16:20] <dsmythies> shaunm_: are you around?
[16:27] <shaunm_> dsmythies: what's up?
[16:29] <dsmythies> shaunm_: There is another bug report which I originally diagnosed as an issue with the html/Makefile.
[16:29] <dsmythies> Shaunm_: However, and after observing some discussion about it between godbyk and KI7MT on this IRC channel,
[16:29] <dsmythies> shaunm_: it has been determined that it is an upstream issue also, and my original diagnosis was incorrect.
[16:29] <dsmythies> shaunm_: reference: https://bugs.launchpad.net/ubuntu-docs/+bug/1239162
[16:29] <ubot2> Launchpad bug 1239162 in Ubuntu Documentation "yelp-build incorrect construction of path to .svg files. Copy step fails" [Low,Confirmed]
[16:29] <dsmythies> shaunm_: At this point, I am trying to determine if the root issue is in xsltproc or yelp-tools (yelp-build).
[16:29] <dsmythies> shaunm_: I would appreciate any insight / opinion you might have.
[16:29] <dsmythies> shaunm_: I realize a reply might take some time. If you are willing to take a look, perhaps reply on the bug report itself.
[16:33] <shaunm_> mmm, referencing icons by absolute path
[16:33] <shaunm_> probably a bug in yelp-build. I'll take a look later today
[16:34] <dsmythies> shaumn_: Thanks very much.
[17:41] <shaunm_> dsmythies: that bug is against an old version of yelp-tools
[17:42] <shaunm_> the good news is, there are no longer any errors with those files
[17:42] <shaunm_> the bad news is, yelp-build completely ignores them. so unless help.ubuntu.com/usr/share/icons/... exists and has the files, you'll have broken images
[17:44] <shaunm_> and I'm not really sure the best way to solve that
[17:47] <dsmythies> shaunm_: Currently (at least whatever yelp-build I am ausing now) does build the html properly, and the links are to, for example:
[17:48] <dsmythies> <p class="p"><span class=" em">Bluetooth menu</span> <span class="media media-image"><img src="img/bluetooth-active.svg" alt="Bluetooth icon"></span></p>
[17:48] <dsmythies> which is a realtive link to the correct place (or desired palce)
[17:48] <dsmythies> are you saying it will be different now?
[17:50] <dsmythies> shaunm_: as for the copy step is question here. In my opinion, it is better that it doesn't even try to copy the files.
[17:50] <dsmythies> shaunm_: as for the copy step in question here. In my opinion, it is better that it doesn't even try to copy the files.
[17:50] <dsmythies> why? becuase sometimes one is compiling on a computer that doesn't actually have the files.
[17:51] <dsmythies> Now, that might just be me.
[17:54] <shaunm_> dsmythies: oh, that might be part of your customization layer. I was just calling yelp-build without any customizations. I'll look
[18:17] <dsmythies> shaunm_: I better figure out how to try the newest version of yelp stuff, for testing. It seems, a few times now, I bring up issues that are already solved.
[18:19] <shaunm_> dsmythies: there's probably a ppa with the latest packages
[18:20] <shaunm_> otherwise, honestly, I often override the package manager and install from the tarball
[18:20] <shaunm_> not something I generally recommend for most packages, but yelp-tools is pretty safe
[18:21] <dsmythies> shaunm_: where can I get the tarball?
[18:23] <shaunm_> http://ftp.gnome.org/pub/gnome/sources/yelp-tools/
[18:23] <shaunm_> or git clone git://git.gnome.org/yelp-tools
[18:24] <shaunm_> oh yikes, you guys are overriding yelp.page
[18:24] <shaunm_> err, html.pge
[18:24] <shaunm_> * html.page
[18:24] <shaunm_> sheesh
[18:24] <shaunm_> and mal2html.page.linktrails.trail
[18:25] <shaunm_> actually, I know why you're overriding mal2html.page.linktrails.trail, and I added a hook in the xsl quite some time ago so you could stop doing that
[18:26] <dsmythies> shaunm_: Ugh... Ugh... Not guility. These are before my time.
[18:26] <shaunm_> the html.page override is much much trickier
[18:26] <shaunm_> I know
[18:26] <shaunm_> probably mdke
[18:27] <dsmythies> and or Jeremy
[18:27] <shaunm_> yeah
[18:27] <shaunm_> anyway, if it's the only way to do what you need to do, then it is what it is
[18:27] <shaunm_> I've tried to add extension points whenever I see overrides like these, so people can use those instead
[18:28] <dsmythies> shaunm_: Please let us know if we should change things. Myself, I would never know otherwise.
[18:28] <shaunm_> I'll cook up a patch for the mal2html.page.linktrails.trail override. that one's easy to fix, because I already added an extension point for exactly that purpose
[18:29] <shaunm_> the other one is hard
[18:29] <dsmythies> shaunm_: O.K. Understood.
[18:30]  * dsmythies goes off to get and try tarball
[19:25] <dsmythies> shaunm_: can I not build a .deb directly from the git clone? Note: my only experience is with kernel.org where I just do "git pull" then "make deb-pkg" then install using "sudo dpkg -i linux-bla-bla.deb"
[19:39] <dsmythies> shaunm_: I guess not, and I guess that is why you said you "often override the package manager and install from the tarball" above.
[20:54] <KI7MT> dsmythies, another option for the traditional ./configure .. make && make install .. look into checkintall package with auto-apt .. makes removal and / or updating from sources pretty easy: https://help.ubuntu.com/community/CheckInstall
[21:00] <KI7MT> Or, one can just build it in $USER/bin or something, forgoing make install and sudo ln -s /usr/local/bin for testing  .. somthing along those lines.
[21:17] <dsmythies> KI7MT: thanks for the suggestions. In the end, I want to be sure I can revert, if needed.
[21:19] <KI7MT> dsmythies, that's the purpose of checkinstall, it updates the pkg-manager also. Removal is then something like dpkg -r <package-name>
[21:21] <KI7MT> dsmythies,  Use it for testing Hamlib and wsjtx builds .. you could also use /opt and and in .bashrc add path var's .. I use that method for testing Intel and Portland compilers while still maintaining standard system gcc installs.ff.
[21:21] <KI7MT> * I use it for testing .. ..
[21:22] <shaunm_> that's kind of slick
[21:22] <dsmythies> KI7MT: Aghh... Thanks
[21:22]  * dsmythies goes off to study up
[21:22] <shaunm_> dsmythies: for yelp-tools, you could just install with --prefix=/usr/local
[21:22] <shaunm_> should work just fine, and won't interfere with the packaged yelp-tools in /usr
[21:23] <shaunm_> or somewhere in /opt or whatever
[21:23] <shaunm_> but then you'll have to manage $PATH for your Makefile to work
[21:24] <dsmythies> hmmm... A saga is developing...
[21:54] <dsmythies> KI7MT: checkinstall seems to be exactly what I was looking for. However, I got "Building Debian package... FAILED!". Everything turns into a saga.
[21:55] <KI7MT> dsmythies, interesting .. did it say why it failed?
[21:59] <dsmythies> KI7MT: It asked if I wanted to look at the log. I said yes. version did not start with a digit. Did it again, forcing version instead of default, and now it seems happy.
[22:00] <KI7MT> dsmythies, Ok, well seems all is well in the end :-)