jsxc icon indicating copy to clipboard operation
jsxc copied to clipboard

Git submodules should be changed to bower packages

Open dimaursu opened this issue 9 years ago • 11 comments

There's no point in having 2 ways of managing dependencies: "i18next" and "otr" have bower packages, with these names. "stophe.jingle" doesn't, the package in bower is way older. (It's the original repo, not the one forked by sulko)


Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

dimaursu avatar Jan 02 '15 00:01 dimaursu

agree with i18next.

after I wrote you on diaspora I remembered why I haven't moved otr and strophe to bower; sulko touched the code.

but it is still a valid point. let me think about it, maybe we find a proper solution for this.

zauberstuhl avatar Jan 03 '15 00:01 zauberstuhl

how about jquery and plugins? Since they are already served via rails-assets.org as gems, why do we have to embed them? gemspec already has those dependencies. sprockets will find them anyway.

pravi avatar Jan 11 '15 05:01 pravi

Any plans for fixing this?

pravi avatar Jan 29 '16 07:01 pravi

@Zauberstuhl can we at least remove minified js from jsxc.dep.js immediately? Currently strophe.min.js is used in jsxc.dep.js. Also can you document how this file is created (which are the dependencies used)? This is blocking diaspora package update to 0.6.0.0 in debian.

pravi avatar Sep 25 '16 14:09 pravi

Also why do we need to create jsxc.dep.js? Doesn't assets:precompile step do concatenation already?

pravi avatar Sep 26 '16 04:09 pravi

All dependencies with the correct license is documented here https://github.com/diaspora/jsxc/blob/develop/dep.json

Why is it blocking th debian package release?

Doesn't assets:precompile step do concatenation already?

Many stuff is not available in rails assets or cannot be used via the assets pipeline (e.g. web worker)

zauberstuhl avatar Sep 26 '16 10:09 zauberstuhl

On 2016, സെപ്റ്റംബർ 26 4:10:45 PM IST, Lukas Matt [email protected] wrote:

All dependencies with the correct license is documented here https://github.com/diaspora/jsxc/blob/develop/dep.json

Thanks, I will check this.

Why is it blocking th debian package release?

Because minified files are not considered source code and every binary should have its corresponding source code.

So we minify the non-minified source during package build.

I think jsxc upstream has the non-minified jsxc.dep.js I'll try to take it from there directly.

Doesn't assets:precompile step do concatenation already?

Many stuff is not available in rails assets or cannot be used via the assets pipeline (e.g. web worker)

Is it true for all remaining dependencies?

pravi avatar Sep 26 '16 11:09 pravi

I think jsxc upstream has the non-minified jsxc.dep.js I'll try to take it from there directly.

not only upstream -> https://github.com/diaspora/jsxc/blob/develop/build/jsxc.dep.js

Is it true for all remaining dependencies?

haven't checked. OTR and webworker is the first thing which comes to my mind.

zauberstuhl avatar Oct 01 '16 11:10 zauberstuhl

@Zauberstuhl can't we use rails-assets-jsxc directly? Though their bower package is not yet updated to 3.0 https://github.com/jsxc/jsxc/issues/371

pravi avatar Oct 01 '16 12:10 pravi

not only upstream -> https://github.com/diaspora/jsxc/blob/develop/build/jsxc.dep.js

No, that uses strophe.min.js, see and compare https://raw.githubusercontent.com/jsxc/jsxc/master/build/lib/jsxc.dep.js

which uses strophe.js (non minimized).

pravi avatar Oct 01 '16 12:10 pravi

can't we use rails-assets-jsxc directly

unfortunately no, the dependencies are broken: https://github.com/jsxc/jsxc/blob/master/bower.json#L30

zauberstuhl avatar Oct 04 '16 10:10 zauberstuhl