Skip to content

[FIX] Armhf snap build#11268

Merged
rodrigok merged 11 commits into
developfrom
snap-node-prepare-script
Jun 27, 2018
Merged

[FIX] Armhf snap build#11268
rodrigok merged 11 commits into
developfrom
snap-node-prepare-script

Conversation

@geekgonecrazy

@geekgonecrazy geekgonecrazy commented Jun 27, 2018

Copy link
Copy Markdown
Contributor

Closes #8555
Closes #10653

Remove the emoji one node module that depends on phantomjs.

It appears that in our CI when we do npm install it installs devDependencies. Which on its own is fine. But when we do meteor build, it compiles and copies over the dev dependencies also. So it installs the grunt-contrib-qunit package, which has hard depend on phantomjs.

We should look at adding NODE_ENV=production to our ci to prevent it from installing dev dependencies. Would likely reduce the size of our resulting bundle.

Also remove the sharp/vendor folder so that during the npm install it will download the pre-compiled binaries for arm. It seems like it downloads and fails if that folder is there.

@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-11268 June 27, 2018 03:13 Inactive
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-11268 June 27, 2018 03:14 Inactive
@geekgonecrazy

Copy link
Copy Markdown
Contributor Author

Launchpad is still giving some issues... Its arm64 hardware trying to emulate armhf... Most of the time does a good job. But sharp its not working great.

@geekgonecrazy

geekgonecrazy commented Jun 27, 2018

Copy link
Copy Markdown
Contributor Author

Builds via launchpad just aren't going to be a possibility right now. But as is, will work if built on a raspberry pi. So for now... will manually do periodically until we can get maybe some help from launchpad guys to figure out why its not faking armhf good enough :)

@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-11268 June 27, 2018 17:21 Inactive
Comment thread .snapcraft/resources/prepareRocketChat Outdated
curl -SLf "https://releases.rocket.chat/#{RC_VERSION}/download/" -o rocket.chat.tgz

tar xvf rocket.chat.tgz --strip 1
tar xvf rocket.chat.tgz --strip 1 > /dev/null

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think removing v option should be enough.. no need to > /dev/null

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Doh! Duh. Habit to do tar xvf forget about the v 😁

Comment thread .snapcraft/resources/preparenode Outdated
fi


wget https://nodejs.org/dist/$node_version/node-$node_version-linux-$node_arch.tar.xz; tar xvf node-$node_version-linux-$node_arch.tar.xz --strip 1 > /dev/null;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

any reason to no split this into two lines?

same thing here, just removing v option from tar will make it silent, no need to > /dev/null

@geekgonecrazy geekgonecrazy Jun 27, 2018

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

No reason. Copied directly from the inline in the snapcraft.yaml . fixed :)

@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-11268 June 27, 2018 17:46 Inactive
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-11268 June 27, 2018 17:48 Inactive
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-11268 June 27, 2018 21:15 Inactive
@rodrigok rodrigok merged commit 445ef8b into develop Jun 27, 2018
@rodrigok rodrigok deleted the snap-node-prepare-script branch June 27, 2018 23:34
@rodrigok rodrigok mentioned this pull request Jun 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants