Magics-Group / wcjs-renderer

Renderer JS API for WebChimera.js
http://webchimera.org/
MIT License
40 stars 24 forks source link

use wcjs-prebuilt? #6

Open jaruba opened 9 years ago

jaruba commented 9 years ago

I made a branch that uses wcjs-prebuilt, should we merge this to master?

It can also be used alternatively by linking to the prebuilt branch instead of the master branch from package.json.

@Ivshti is this the correct way of implementing wcjs-prebuilt?

@RSATom can you think of any reason why it shouldn't be merged?

@luigiplr are you eating pizza right now?

RSATom commented 9 years ago

I just afraid I could not keep .node files up-to-date, since for example Electron releases out too frequently.

RSATom commented 9 years ago

Maybe the best solution could be merge wcjs-prebuilt to main repo, and use regular build as failback

RSATom commented 9 years ago

@Ivshti, @luigiplr how much work such merge could require?

Ivshti commented 9 years ago

@RSATom I think not a lot

But it's okay for electron, because we only have to re-build when chromium major version is bumped, which means every 3-4 (or even more) major electron releases. E.g. there's need for recompile between 0.29 and 0.30, because it changes chromium's major, but no need to re-compile between 0.26 - 0.30 because they all have the same chromium major.

@RSATom - I think your concern is not that relevant.

RSATom commented 9 years ago

@Ivshti, But do wcjs-prebuilt use only major version on binary download?

Ivshti commented 9 years ago

@RSATom no. But you can override electron version passed to wcjs-prebuilt to something compatible of what you are using

Ivshti commented 9 years ago

If you remember, I had made separate electron runtime names electron44 and electron45 to fix this issue. Luigi's latest improvements wiped that out, but we can return it

RSATom commented 9 years ago

I think we need more generic solution

RSATom commented 9 years ago

btw, another solution could be trigger CI build by watching new releases on https://github.com/mafintosh/electron-prebuilt But don't sure how to do it yet

RSATom commented 8 years ago

I think better solution will be remove direct dependency from wcjs. I did it already in https://github.com/RSATom/webgl-video-renderer, but it has a little bit changed api

luigiplr commented 8 years ago

@jaruba I... Might be. :neutral_face:

@RSATom should not be to hard to setup a ci task to "watch" the github releases of the electron repo.

RSATom commented 8 years ago

@luigiplr, yes, but excluding direct dependency is still good idea, since some time later we will have wcjs with gstreamer, and I want use same renderer for both wcjs and wcjs-gs (that's why I did webgl-video-renderer)

luigiplr commented 8 years ago

Ahhh cant wait for gStreamer variant of wcjs :dancer:

RSATom commented 8 years ago

It's very possible it will be only for linux...

luigiplr commented 8 years ago

It's very possible it will be only for linux...

Not Aumsed

RSATom commented 8 years ago

I just think libvlc is better alternative on Windows.