Opened 6 years ago

Closed 5 years ago

#55655 closed defect (fixed)

nodejs7 @7.10.1: error: return type of out-of-line definition of 'v8_inspector::InspectedContext::createInjectedScript' differs from that in the declaration

Reported by: TrejGun (TrejGun) Owned by: ci42
Priority: Normal Milestone:
Component: ports Version:
Keywords: highsierra Cc:
Port: nodejs7

Description (last modified by ryandesign (Ryan Carsten Schmidt))

port nodejs@nodejs7-7.10.1_0

Fails to build every time, even with cleaned cache

clean --dist and clean --all

sudo rm -rf /opt/local/var/macports/build/*
$ sudo port install nodejs7 
--->  Computing dependencies for nodejs7
--->  Fetching archive for nodejs7
--->  Attempting to fetch nodejs7-7.10.1_0.darwin_17.x86_64.tbz2 from http://jog.id.packages.macports.org/macports/packages/nodejs7
--->  Attempting to fetch nodejs7-7.10.1_0.darwin_17.x86_64.tbz2 from https://pek.cn.packages.macports.org/macports/packages/nodejs7
--->  Attempting to fetch nodejs7-7.10.1_0.darwin_17.x86_64.tbz2 from http://kmq.jp.packages.macports.org/nodejs7
--->  Fetching distfiles for nodejs7
--->  Verifying checksums for nodejs7
--->  Extracting nodejs7
--->  Applying patches to nodejs7
--->  Configuring nodejs7
--->  Building nodejs7
Error: Failed to build nodejs7: command execution failed
Error: See /opt/local/var/macports/logs/_opt_local_var_macports_sources_rsync.macports.org_macports_release_tarballs_ports_devel_nodejs7/nodejs7/main.log for details.
Error: Follow https://guide.macports.org/#project.tickets to report a bug.
Error: Processing of port nodejs7 failed

nodejs6 and nodejs8 are ok

Attachments (1)

main.log.zip (264.7 KB) - added by TrejGun (TrejGun) 6 years ago.

Download all attachments as: .zip

Change History (6)

Changed 6 years ago by TrejGun (TrejGun)

Attachment: main.log.zip added

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

Description: modified (diff)
Keywords: highsierra added
Owner: set to ci42
Port: nodejs7 added
Status: newassigned
Summary: nodejs7 fails to install on high sierranodejs7 @7.10.1: error: return type of out-of-line definition of 'v8_inspector::InspectedContext::createInjectedScript' differs from that in the declaration

It built ok on our Mavericks-through-Sierra buildbot workers, so maybe it's related to High Sierra's newer version of clang.

comment:2 Changed 6 years ago by awbacker (Andrew Backer)

I am having this issue as well. Says it is a duplicate declaration as the reason for that if it helps. Rebuilding now but I don't expect any change.

:info:build ../deps/v8_inspector/src/inspector/inspected-context.cc:77:24: error: return type of out-of-line definition of 'v8_inspector::InspectedContext::createInjectedScript' differs from that in the declaration
:info:build void InspectedContext::createInjectedScript() {
:info:build ~~~~                   ^
:info:build ../deps/v8/src/inspector/inspected-context.h:38:8: note: previous declaration is here
:info:build   bool createInjectedScript();
:info:build   ~~~~ ^
:info:build 1 error generated.

I don't have v8 installed/active.

Apple LLVM version 9.0.0 (clang-900.0.39.2)

comment:3 Changed 5 years ago by kencu (Ken)

changing the type to bool everywhere seems to gets past this.

comment:4 Changed 5 years ago by kencu (Ken)

Not that it matters, as nodejs7 is EOL and about to be deleted out of MacPorts anyway.

comment:5 Changed 5 years ago by yan12125 (Chih-Hsuan Yen)

Resolution: fixed
Status: assignedclosed

In 08408fd864685be540dc63399ebc445458904cae/macports-ports (master):

nodejs*: drop nodejs{5,7,9}

Those versions are non-LTS and have reached their end-of-life [1].

Closes: #53799
Closes: #54083
Closes: #55655
Closes: #55656

[1] https://github.com/nodejs/Release

Note: See TracTickets for help on using tickets.