Opened 10 years ago

Closed 10 years ago

#28561 closed defect (fixed)

nodejs @0.4.1_0: build fails when tidy is installed

Reported by: ryandesign (Ryan Schmidt) Owned by: ceager@…
Priority: Normal Milestone:
Component: ports Version: 1.9.2
Keywords: Cc:
Port: nodejs

Description

The nodejs build fails when the tidy port is installed.

:info:build ../src/node.cc: In function 'void node::CheckStatus(ev_timer*, int)':
:info:build ../src/node.cc:1468: error: 'Platform' has not been declared
:info:build ../src/node.cc: In function 'v8::Handle<v8::Value> node::MemoryUsage(const v8::Arguments&)':
:info:build ../src/node.cc:1493: error: 'Platform' has not been declared
:info:build ../src/node.cc: In function 'v8::Handle<v8::Value> node::ProcessTitleGetter(v8::Local<v8::String>, const v8::AccessorInfo&)':
:info:build ../src/node.cc:1805: error: 'Platform' has not been declared
:info:build ../src/node.cc: In function 'void node::ProcessTitleSetter(v8::Local<v8::String>, v8::Local<v8::Value>, const v8::AccessorInfo&)':
:info:build ../src/node.cc:1815: error: 'Platform' has not been declared
:info:build ../src/node.cc: In function 'void node::Load(int, char**)':
:info:build ../src/node.cc:1996: error: 'Platform' has not been declared
:info:build ../src/node.cc: In function 'int node::Start(int, char**)':
:info:build ../src/node.cc:2226: error: 'node::Platform' has not been declared

I think this is because the tidy port provides /opt/local/include/platform.h, and nodejs uses

#include <platform.h>

but it expects to be getting the platform.h provided in its source files instead. Perhaps nodejs should use

#include "platform.h"

instead.

Attachments (1)

main.log.bz2 (7.0 KB) - added by ryandesign (Ryan Schmidt) 10 years ago.
build log

Download all attachments as: .zip

Change History (3)

Changed 10 years ago by ryandesign (Ryan Schmidt)

Attachment: main.log.bz2 added

build log

comment:1 Changed 10 years ago by ryandesign (Ryan Schmidt)

Summary: nodejs: build fails when tidy is installednodejs @0.4.1_0: build fails when tidy is installed

comment:2 Changed 10 years ago by ryandesign (Ryan Schmidt)

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.