dsmythies | shaunm_: are you around? | 16:20 |
---|---|---|
shaunm_ | dsmythies: what's up? | 16:27 |
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:29 |
shaunm_ | mmm, referencing icons by absolute path | 16:33 |
shaunm_ | probably a bug in yelp-build. I'll take a look later today | 16:33 |
dsmythies | shaumn_: Thanks very much. | 16:34 |
shaunm_ | dsmythies: that bug is against an old version of yelp-tools | 17:41 |
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:42 |
shaunm_ | and I'm not really sure the best way to solve that | 17:44 |
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:47 |
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:48 |
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:50 |
dsmythies | Now, that might just be me. | 17:51 |
shaunm_ | dsmythies: oh, that might be part of your customization layer. I was just calling yelp-build without any customizations. I'll look | 17:54 |
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:17 |
shaunm_ | dsmythies: there's probably a ppa with the latest packages | 18:19 |
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:20 |
dsmythies | shaunm_: where can I get the tarball? | 18:21 |
shaunm_ | http://ftp.gnome.org/pub/gnome/sources/yelp-tools/ | 18:23 |
shaunm_ | or git clone git://git.gnome.org/yelp-tools | 18:23 |
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:24 |
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:25 |
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:26 |
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:27 |
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:28 |
shaunm_ | the other one is hard | 18:29 |
dsmythies | shaunm_: O.K. Understood. | 18:29 |
* dsmythies goes off to get and try tarball | 18:30 | |
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:25 |
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. | 19:39 |
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 | 20:54 |
=== dpm_ is now known as dpm | ||
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:00 |
dsmythies | KI7MT: thanks for the suggestions. In the end, I want to be sure I can revert, if needed. | 21:17 |
KI7MT | dsmythies, that's the purpose of checkinstall, it updates the pkg-manager also. Removal is then something like dpkg -r <package-name> | 21:19 |
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:21 |
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:22 |
shaunm_ | or somewhere in /opt or whatever | 21:23 |
shaunm_ | but then you'll have to manage $PATH for your Makefile to work | 21:23 |
dsmythies | hmmm... A saga is developing... | 21:24 |
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:54 |
KI7MT | dsmythies, interesting .. did it say why it failed? | 21:55 |
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. | 21:59 |
KI7MT | dsmythies, Ok, well seems all is well in the end :-) | 22:00 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!