Closed Bug 454774 Opened 16 years ago Closed 7 years ago

make install doesn't work on Solaris

Categories

(Firefox Build System :: General, defect)

x86
Solaris
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INCOMPLETE

People

(Reporter: ginnchen+exoracle, Assigned: ginnchen+exoracle)

References

Details

Attachments

(1 obsolete file)

/usr/bin/find on Solaris doesn't have -xtype option.
"-type f" is same to "-xtype f" on both Linux and Solaris.
Attached patch patch (obsolete) — Splinter Review
Assignee: nobody → ginn.chen
Status: NEW → ASSIGNED
Attachment #338081 - Flags: review?(benjamin)
xtype is most definitely not the same as type. xtype dereferences any symlinks before checking the type, while type doesn't. xtype is not part of the posix standard, but -L is, so perhaps we should use that?
Attachment #338081 - Attachment is obsolete: true
Attachment #338081 - Flags: review?(benjamin)
Comment on attachment 338081 [details] [diff] [review]
patch

Oops, I misread the manpage.

But find -L ... -type f doesn't work as expected, either.
the difference of
find -L . -type f
and
find . -xtype f
is the latter won't enter a symbolic directory.

We have such a link at dist/include, it is system_wrappers.

I think we don't want to install system_wrappers, at least we don't flatten it, because it would overwrite the real header.

so maybe we should use
find -L $(DIST)/include -type f ! -wholename '*/system_wrapers/*' ...
BTW:

We have vorbis/codec.h and theora/codec.h, oggz/config_win32.h and oggplay/config_win32.h.

We could not install both into one directory.
9 year old bug involving Solaris. There is litting value in keeping this open.
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → INCOMPLETE
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: