Skip to content
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.

Libsass 3.0.2 #510

Closed
HamptonMakes opened this issue Nov 4, 2014 · 30 comments
Closed

Libsass 3.0.2 #510

HamptonMakes opened this issue Nov 4, 2014 · 30 comments

Comments

@HamptonMakes
Copy link
Member

I know we are torturing you all with our weekly release cycle... but...

https://github.com/sass/libsass/releases/tag/3.0.2

@benflodge
Copy link

I have been attempting to compile this but am getting the following error:-
node: symbol lookup error: /u2/home/ben.lodge/sass/node-sass/build/Release/binding.node: undefined symbol: _Z13json_mkobjectv

The tests are failing after running

.renderFile(options)
✓ should compile with renderFile

@am11
Copy link
Contributor

am11 commented Nov 4, 2014

@hcatlin, thanks for the heads-up! 😄👍

@benflodge, we will fix this. We need to make some fixes. I have them locally.

@kevva, @andrew, @nschonni, @keithamus, should we address #332 before release vNext? Also, there are couple of new source-map features added by @mgreter that we need to add (in both API and CLI).

am11 added a commit to am11/node-sass that referenced this issue Nov 5, 2014
* Additions in the binding.gyp.
* Updated CLI test for line endings.
* Updated Appveyor.yml as per @FeodorFitsner's
  suggestion.
andrew added a commit that referenced this issue Nov 5, 2014
Submodules: Updated submodules (#510)
@keithamus
Copy link
Member

@am11 I'd personally like to see issues like #497 dealt with as a priority over features like #332. I think we should release a new version with libsass 3.0.2 and the new work done around the binaries, and wait for feedback.

@andrew
Copy link
Contributor

andrew commented Nov 5, 2014

I agree with @keithamus, we should get a release out to fix the install issues first

@kevva
Copy link
Member

kevva commented Nov 5, 2014

Yeah, d4b8f6b fixes the install issues for most users (unless the binary doesn't work, then they'll have to build it). Is the build stuff (flag issues etc.) fixed?

@keithamus
Copy link
Member

@kevva The majority of flag issues I've seen are just users with old versions of GCC. Are there any others?

@kevva
Copy link
Member

kevva commented Nov 5, 2014

Hm, not that I'm aware of. So, are we good to go then? How are we with the Windows issue (tests taking forever when installing) @am11? We need to compile some binaries too.

@am11
Copy link
Contributor

am11 commented Nov 5, 2014

Yes. I think we are good to go. Please make sure compile binaries on separate machines / VMs for each architecture. See #511 (comment).

@am11
Copy link
Contributor

am11 commented Nov 5, 2014

@kevva, should I build the Windows binaries again for https://github.com/sass/node-sass-binaries/commits/master, since you pushed some commits to node-sass after sass/node-sass-binaries@c4f9e9c and sass/node-sass-binaries@64c65fd?

@kevva
Copy link
Member

kevva commented Nov 5, 2014

Nah, I think we're fine. Those changes has no effect on the binaries.

@am11
Copy link
Contributor

am11 commented Nov 5, 2014

Yup! I can push Linux x64 binaries. Can you do x32 linux? @andrew will take care of Darwin! 💡

@kevva
Copy link
Member

kevva commented Nov 5, 2014

Don't have access to a x86 system ~~.

@keithamus
Copy link
Member

Virtualbox will "emulate" 32-bit for you. Unfortunately I recently had to hose my 32-bit images from my machine 😦

@kevva
Copy link
Member

kevva commented Nov 5, 2014

Yeah, but I haven't got Vagrant etc set up and I'm too lazy to do that ;pPp.

@keithamus
Copy link
Member

Does anyone know if we can configure Travis/Appveyor to get us the 32-bit linux binaries?

@andrew
Copy link
Contributor

andrew commented Nov 5, 2014

I can do Linux 32+64 and Mac this afternoon if still needed

@am11
Copy link
Contributor

am11 commented Nov 5, 2014

We can have 64 bit OS, just need 32 bit node.js on Linux (installed one not the standalone one <-- this is important)

@kevva
Copy link
Member

kevva commented Nov 5, 2014

@keithamus, AppVeyor is Windows only. Travis runs on 64 bit images. But yeah, we could probably write a script that installs node-32bit on Travis and runs the build + commits the binaries.

@am11
Copy link
Contributor

am11 commented Nov 5, 2014

@andrew Just Linux 32 and Mac. I am building x64 Linux as we speak.. 😄

@am11
Copy link
Contributor

am11 commented Nov 5, 2014

LOL look at this: sass/node-sass-binaries@faa2e3d.. vagrant?

How, why, what da...!

@kevva
Copy link
Member

kevva commented Nov 5, 2014

You've probably setup a image with pre-installed git settings?

@am11
Copy link
Contributor

am11 commented Nov 5, 2014

Nah, I installed everything myself. Ruby, Node, Python, GCC. :D

Just had to do git config --global user.email " .. " thingy. But still, it committed with my account so.. Anyway that was crazy, now it's like this: sass/node-sass-binaries@7a2eb55.

@andrew
Copy link
Contributor

andrew commented Nov 5, 2014

Looks like all the binaries have been built for 3.0.2, are we going to wait for #515 to be merged or should we 🚢 a release this evening?

@am11
Copy link
Contributor

am11 commented Nov 5, 2014

I think lets merge #515 and release. #515 is ready and now node-sass-binaries are isolated from the actual npm. (no mode ../node-sass-binaries folder required) 😄

@andrew
Copy link
Contributor

andrew commented Nov 5, 2014

So do we need to tag the node-sass-binaries git repo or something?

@nschonni
Copy link
Contributor

nschonni commented Nov 5, 2014

@am11 has the issue of the binaries not being pinned to a release now been resolved?

@am11
Copy link
Contributor

am11 commented Nov 6, 2014

@nschonni, I think we just need to create a tag on node-sass-binaries. Then we need to assign it to this line. This way it will be locked to one set/version of binaries.

@andrew
Copy link
Contributor

andrew commented Nov 6, 2014

Still having issues with linux binaries: #517 (comment)

@andrew
Copy link
Contributor

andrew commented Nov 19, 2014

Finally published a release that appears to work everywhere with libsass 3.0.2: https://www.npmjs.org/package/node-sass/1.2.3

@andrew andrew closed this as completed Nov 19, 2014
@xzyfer
Copy link
Contributor

xzyfer commented Nov 20, 2014

Completely missed this. Y'all keep rocking!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

8 participants