126163 - -lSM -lICE are hard coded

Status: RESOLVED FIXED

(SeaMonkey :: Build Config, defect, P3)

Description

[Colin Blake]

In /embedding/browser/gtk/tests/Makefile.in (see URL link) the SM and ICE
libraries are unconditionally loaded, even though we have tests for the
existence of the libraries in configure. OpenVMS doesn't have the libraries and
so our linker complains.

Comment 1

[hacker formerly known as [e-mail]]

The hardcoded -lSM & -lICE disappeared with the static-*.mk reorg.  Leaving this
bug open as we do need to make those libraries available to the makefiles as
-lXt requires them on some platforms.

Comment 2

[Colin Blake]

Changing O/S from OpenVMS to Linux so that this bug disappears off my radar.

Comment 3

[hacker formerly known as [e-mail]]

Attachment: Conditionally check for -lSM & -lICE if -lXt test fails

Comment 4

[Brian Ryner (not reading)]

Comment on attachment 83306 [details] [diff] [review]
Conditionally check for -lSM & -lICE if -lXt test fails

r=bryner

Comment 5

[hacker formerly known as [e-mail]]

http://mpt.phrasewise.com/2002/06/07

Comment 6

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Comment on attachment 83306 [details] [diff] [review]
Conditionally check for -lSM & -lICE if -lXt test fails

Please land this on the 1.0.1 branch.  Once there, replace the
"mozilla1.0.1+" keyword with the "fixed1.0.1" keyword.

(Note:	the inconsistent tabs/spaces make this patch a real pain to read.)