Skip to content

Commit

Permalink
fix compile error : mkpeephole related
Browse files Browse the repository at this point in the history 
issuse:
 nodejs/node#10303
 nodejs/node#9707

fix code:
 https://github.com/artynet/openwrt-git/blob/openwrt-1505-setup-03/package/linino/node7/Makefile
  • Loading branch information
Hirokazu MORIKAWA committed Dec 22, 2016
1 parent 4f2388c commit b7d8153
Showing 1 changed file with 9 additions and 1 deletion.
10 changes: 9 additions & 1 deletion node/Makefile
View file
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ PKG_SOURCE_URL:=http://nodejs.org/dist/${PKG_VERSION}
PKG_MD5SUM:=f7e8924c13c74576f6c0688bcf1bbfde

HOST_BUILD_DEPENDS:=python/host
PKG_BUILD_DEPENDS:=python/host
PKG_BUILD_DEPENDS:=python/host +node/host
PKG_INSTALL:=1
PKG_USE_MIPS16:=0

Expand Down Expand Up @@ -99,6 +99,14 @@ HOST_CONFIGURE_ARGS:= \

HOST_CONFIGURE_CMD:=python ./configure

MKTOOL=$(HOST_BUILD_DIR)/out/Release/mkpeephole
FILE_V8_GYP:=$(PKG_BUILD_DIR)/deps/v8/src/v8.gyp

define Build/Prepare
$(Build/Prepare/Default)
$(SED) "s#<(mkpeephole_exec)#$(MKTOOL)#g" $(FILE_V8_GYP)
endef

define Build/InstallDev
$(INSTALL_DIR) $(1)/usr/include
$(CP) $(PKG_INSTALL_DIR)/usr/include/* $(1)/usr/include/
Expand Down

3 comments on commit b7d8153

@artynet
Copy link
Contributor

@artynet artynet commented on b7d8153 Jun 20, 2017 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hello @nxhack, please update the above link to :

https://github.com/artynet/openwrt-git/blob/openwrt-1505-setup-04/package/linino/node7/Makefile

thanks, arturo

@nxhack
Copy link
Owner

@nxhack nxhack commented on b7d8153 Jun 20, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! @artynet

@artynet
Copy link
Contributor

@artynet artynet commented on b7d8153 Jun 21, 2017 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nxhack I have added again the node7 folder with its makefile since time ago I decided to temporarily drop its build.

Bests again

Please sign in to comment.