build error due to SEC_BEGIN_PROTOS / SEC_END_PROTOS are undefined

RESOLVED FIXED in 3.12

Status

NSS
Libraries
RESOLVED FIXED
10 years ago
10 years ago

People

(Reporter: surkov, Assigned: surkov)

Tracking

trunk
3.12
x86
Windows XP

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

1.01 KB, patch
Nelson Bolyard (seldom reads bugmail)
: review+
Details | Diff | Splinter Review
(Assignee)

Description

10 years ago
Created attachment 272282 [details] [diff] [review]
patch

It looks "seccomon.h" should be included to avoid the build error (see a bug 118830)
Attachment #272282 - Flags: review?(nelson)
what software fails to build due to this issue?
Is this fix needed on any NSS branches? or only the trunk?
Comment on attachment 272282 [details] [diff] [review]
patch

r=nelson for trunk.
I think this is only needed on the trunk.
Attachment #272282 - Flags: review?(nelson) → review+
Target Milestone: --- → 3.12
Version: unspecified → trunk
(Assignee)

Comment 3

10 years ago
(In reply to comment #1)
> what software fails to build due to this issue?
> Is this fix needed on any NSS branches? or only the trunk?
> 

Yes, it's on trunk only. I build firefox, the sources I get by cvs update (not by make checkout).
(Assignee)

Updated

10 years ago
Assignee: nobody → surkov.alexander
Alexander, Browser builds are NEVER supposed to use the tip of the NSS trunk.
NSS supplies stable static tags for use by the browser.  
The browser's makefile  knows these tags and pulls the NSS sources using them.
If you pull NSS sources from the trunk, you will run into a lot of problems
that no other browser builders see, because they are all pulling from the 
approved tag.  The NSS team does not promise at any time that the tip of 
the trunk will build and work in the browser.  That promise is only made for
the tags that the NSS team has approved for use by the browser.  

I'm not trying to rebuke you for using NSS trunk.  I just want you to know
that there is a way to build the browser that is much less painful than the
method of pulling NSS from the tip of the trunk.  

OTOH, sometimes reports like yours help us to notice and fix problems sooner
than we otherwise would.  So, if you don't mind enduring the pain and the 
risk of using the NSS trunk, please feel free to continue.  :)

I think your patch will get checked in this weekend.  Thanks for that patch.
(Assignee)

Comment 5

10 years ago
(In reply to comment #4)
> 
> OTOH, sometimes reports like yours help us to notice and fix problems sooner
> than we otherwise would.  So, if you don't mind enduring the pain and the 
> risk of using the NSS trunk, please feel free to continue.  :)

I use trunk browser only as html/xul renderer to check accessibility so usually trunk sources doesn't bring me any pain :).

> I think your patch will get checked in this weekend.  Thanks for that patch.
> 

No problem. I guess I can check in it myself if it doesn't require any special CVS rights.
NSS does require special CVS rights.  Thanks again for the patch.

Checking in util/secdert.h; new revision: 1.4; previous revision: 1.3
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED

Updated

10 years ago
Blocks: 118830
You need to log in before you can comment on or make changes to this bug.