Closed
Bug 190483
Opened 23 years ago
Closed 23 years ago
Do not need short name for layout/build/Makefile.in
Categories
(SeaMonkey :: Build Config, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla1.3beta
People
(Reporter: jhpedemonte, Assigned: mkaply)
Details
Attachments
(1 file)
|
1.71 KB,
patch
|
netscape
:
review+
dbaron
:
approval1.3b+
|
Details | Diff | Splinter Review |
Currently, OS/2 is the only platform to use SHORT_LIBNAME in
layout/build/Makefile.in; WIN32 specifically ifdefs it out. However, since the
default library name is already 8 characters ("gklayout"), there is no reason to
keep using the SHORT_LIBNAME ("gkhtml") on OS/2.
Along with this, we need to make the appropriate packaging changes.
| Assignee | ||
Comment 1•23 years ago
|
||
| Assignee | ||
Updated•23 years ago
|
Attachment #112523 -
Flags: review?(cls)
| Assignee | ||
Comment 2•23 years ago
|
||
Comment on attachment 112523 [details] [diff] [review]
Rename gkhtml to gklayout
I have no review on this (cls is swamped) but I would really like it for 1.3b.
My team has looked at it.
Attachment #112523 -
Flags: approval1.3b?
Comment 3•23 years ago
|
||
Comment on attachment 112523 [details] [diff] [review]
Rename gkhtml to gklayout
please use the "blocking1.3b?" flag to nominate that drivers hold the release
for a particular bug. use the approval1.3b? flag when you have reviews and are
ready to land.
Attachment #112523 -
Flags: approval1.3b?
Updated•23 years ago
|
Flags: blocking1.3b?
Attachment #112523 -
Flags: approval1.3b+
Flags: blocking1.3b?
Comment 4•23 years ago
|
||
Comment on attachment 112523 [details] [diff] [review]
Rename gkhtml to gklayout
r=cls
Attachment #112523 -
Flags: review?(cls) → review+
Comment 5•23 years ago
|
||
Patch has been checked in.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.3beta
Updated•21 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•