Closed Bug 331833 Opened 18 years ago Closed 18 years ago

Build error on Solaris 11 in mailnews/base/build directory

Categories

(SeaMonkey :: Build Config, defect)

Product:
SeaMonkey
Component:
Build Config
Platform:
x86
SunOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: alfred.peng, Unassigned)

Details

Attachments

(1 file, 1 obsolete file)

Patch v1 for trunk
872 bytes, patch
benjamin
: review+
wtc
: review+
Details | Diff | Splinter Review 
There is a build error on Solaris 11 in mailnews/base/build directory:
ld: fatal: library -lmsgbaseutil: not found
ld: fatal: File processing errors. No output written to libmailnews.so
gmake[1]: *** [libmailnews.so] Error 1

It seems that the EXTRA_DSO_LDOPTS still needs $(LIBS_DIR).
Add $(LIBS_DIR) to EXTRA_DSO_LDOPTS to fix the build error.
Attachment #216385 - Flags: review?(benjamin)
Attached patch Patch v1 for trunk (obsolete) — — Splinter Review 
Add $(LIBS_DIR) to EXTRA_DSO_LDOPTS to fix the build error.
Attachment #216386 - Flags: review?(benjamin)
Attachment #216386 - Attachment is obsolete: true
Attachment #216386 - Flags: review?(benjamin)
Comment on attachment 216385 [details] [diff] [review]
Patch v1 for trunk

yuck... this lib will be moving shortly
Attachment #216385 - Flags: review?(benjamin) → review+
Do we need to check in this one? Or wait for its moving?
Attachment #216385 - Flags: superreview?(wtchang)
Comment on attachment 216385 [details] [diff] [review]
Patch v1 for trunk

r=wtc.
Attachment #216385 - Flags: superreview?(wtchang) → review+
Do I need a sr for this bug?
Benjamin, or I should wait for lib change?
Benjamin, can we check this code in to get rid of the build error currently?
Leon helps to check in the patch.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: