OS/2 build brakes after rename of npupp.h to npfunctions.h

RESOLVED FIXED

Status

()

Core
Plug-ins
--
major
RESOLVED FIXED
9 years ago
9 years ago

People

(Reporter: Walter Meinl, Assigned: Peter Weilbacher)

Tracking

Trunk
x86
OS/2
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 obsolete attachment)

(Reporter)

Description

9 years ago
User-Agent:       Mozilla/5.0 (OS/2; U; Warp 4.5; en-US; rv:1.9.1b2pre) Gecko/20081010 Minefield/3.1b2pre
Build Identifier: 

E:/usr/src/hg/comm-central/mozilla/modules/plugin/os2wrapper/wrap_XPCOM.cpp:65:19: error: npupp.h: No such file or directory
E:/usr/src/hg/comm-central/mozilla/modules/plugin/os2wrapper/wrap_XPCOM.cpp: In member function 'virtual nsresult UpOutputStream::WriteSegments(nsresult (*)(nsIOutputStream*, void*, char*, PRUint32, PRUint32, PRUint32*), void*, PRUint32, PRUint32*)':
E:/usr/src/hg/comm-central/mozilla/modules/plugin/os2wrapper/wrap_XPCOM.cpp:4035: warning: overflow in implicit constant conversion
E:/usr/src/hg/comm-central/mozilla/modules/plugin/os2wrapper/wrap_XPCOM.cpp:4039: warning: overflow in implicit constant conversion
[snip]
modules/plugin/os2wrapper/wrap_XPCOM.cpp doesn't find the renamed header.

Reproducible: Always

Steps to Reproduce:
1.
2.
3.
(Reporter)

Comment 1

9 years ago
Created attachment 342638 [details] [diff] [review]
correct the header designation
Attachment #342638 - Flags: review?(mozilla)
(Reporter)

Updated

9 years ago
Depends on: 455458
Version: unspecified → Trunk
(Assignee)

Comment 2

9 years ago
Removing that include line was already enough for me, seems that we don't use the content of that file (any more).
(Assignee)

Comment 3

9 years ago
Comment on attachment 342638 [details] [diff] [review]
correct the header designation

I checked in the line removal instead, see http://hg.mozilla.org/mozilla-central/rev/18ad520738e5
Attachment #342638 - Attachment is obsolete: true
Attachment #342638 - Flags: review?(mozilla)
(Assignee)

Updated

9 years ago
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
(Assignee)

Updated

9 years ago
Assignee: nobody → mozilla
You need to log in before you can comment on or make changes to this bug.