search for: firefox_cflags

> I've tried to separate out the general bugfixes from the new > implementation work in this patch: Sorry, I should have done that, I do tend to fix stuff I find while coding and neglect splitting up afterwards. Bad me. > I've not yet committed this to trunk. It does sound pretty cool > though. I don't see any problem with adding pango etc. support, it's > pretty

...d the second one to test for NULL before actually using it: That didn't compile though, sorry, the attached patch fixes that. I edited the original patch by hand to remove the Kate stuff and it seems I ended up with a stray comma. Also, I'd entirely forgotten about that, but I need to set FIREFOX_CFLAGS and FIREFOX_LIBS to (resp) "-I$(ffpath)/include" and "-L$(ffpath)/lib" for configure to not whine it can't find Firefox. I've got the Gecko SDK, but I can't see any pc files in there, so the configure.ac check's got me puzzled. Since it all builds and run if I do...

..., the attached patch fixes that. I edited > the original patch by hand to remove the Kate stuff and it seems I ended > up with a stray comma. sorry, that was my fault with the merge; I fixed it a few hours later in r3576 > Also, I'd entirely forgotten about that, but I need to set FIREFOX_CFLAGS > and FIREFOX_LIBS to (resp) "-I$(ffpath)/include" and "-L$(ffpath)/lib" for > configure to not whine it can't find Firefox. I've got the Gecko SDK, but > I can't see any pc files in there, so the configure.ac check's got me > puzzled. Since it al...