#19273 closed defect (fixed)
yelp 2.24.0 configure and build errors after firefox update
| Reported by: | bskaplan14@… | Owned by: | macports-tickets@… |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | ports | Version: | 1.7.1 |
| Keywords: | yelp xulrunner firefox | Cc: | thedoobs@…, brentaar@… |
| Port: | yelp |
Description
After updating firefox-x11 to Firefox 3, yelp stopped working because it was looking for the firefox 2 libs. Unfortunately, Firefox 3 doesn't create firefox.pc, which Yelp looks for while configuring. (this error is identified around comment # 10 on ticket 19048). If I install Xulrunner and switch the configure args in Yelp's portfile to use libxul-embedding for gecko instead of firefox, then it configures without a problem.
When building Yelp using libxul-embedding, I got an error about nscore.h (from Xulrunner) not finding prtypes.h (from nspr). Adding -I/opt/local/include/nspr to Cflags in libxul-embedding.pc fixed this. I don't know if that's supposed to be there or not, but it's in the file on Ubuntu Jaunty and it got Yelp to work. I'm not including a patch for this one because I don't know how to write the patch without hard coding the path.
Attachments (4)
Change History (12)
Changed 15 years ago by bskaplan14@…
| Attachment: | yelp-portfile.diff added |
|---|
Changed 15 years ago by bskaplan14@…
| Attachment: | configure-problem.txt added |
|---|
Configure problem using unmodified portfile
comment:1 follow-up: 2 Changed 15 years ago by jeremyhu (Jeremy Huddleston Sequoia)
I think you might want libxul instead of libxul-embedding... but I'm not sure there.
Changed 15 years ago by axel@…
| Attachment: | libxul-embedding.txt added |
|---|
Changed 15 years ago by axel@…
| Attachment: | libxul.txt added |
|---|
comment:2 follow-up: 3 Changed 15 years ago by axel@…
Replying to jeremyhu@…:
I think you might want libxul instead of libxul-embedding... but I'm not sure there.
Both versions fail for me. See libxul-embedding.txt and libxul.txt above for my results with each flavor of the portfile.
comment:3 Changed 15 years ago by bskaplan14@…
Replying to axel@…:
Replying to jeremyhu@…:
I think you might want libxul instead of libxul-embedding... but I'm not sure there.
Both versions fail for me. See libxul-embedding.txt and libxul.txt above for my results with each flavor of the portfile.
I don't know how to get libxul working, but I couldn't do that on Ubuntu either so I don't think that's a macports problem. The libxul-embedding failure is the second problem I mentioned, the one that I don't know how to write a patch for. Go to /opt/local/lib/pkgconfig/libxul-embedding.pc and add "-I/opt/local/include/nspr" to the Cflags to get it to build and install.
comment:7 Changed 15 years ago by jeremyhu (Jeremy Huddleston Sequoia)
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
a patch for Yelp's portfile to use Xulrunner and libxul-embedding instead of Firefox