coala-html icon indicating copy to clipboard operation
coala-html copied to clipboard

Run bower install during pip install

Open tushar-rishav opened this issue 8 years ago • 11 comments

Presently, bower installs dependencies when user runs coala-html first time. We should implement this installation process when user installs coala-html via pip.

tushar-rishav avatar May 21 '16 09:05 tushar-rishav

Thanks for reporting this issue!

@coala-analyzer/coala-contributors, your aid is required, fellow coalaian. Help us triage and solving this issue!

gitmate-bot avatar May 21 '16 09:05 gitmate-bot

This issue is absurd. bower install need not be run while installing coala-html. Use of bower is only when user runs coala-html first time - when angular app has been copied. Closing this issue as it's not required anymore

tushar-rishav avatar May 22 '16 12:05 tushar-rishav

The reason I thought it makes sense to do bower install with pip install is because installing should happen in one step. I hate downloading "installers" which then again start downloading and installing more stuff Looks at his windows laptop ...

But that is definitely a user preference. Some people prefer things that install lazily on demand. Hence I had mentioned you may want to have a vote about this.

AbdealiLoKo avatar May 22 '16 13:05 AbdealiLoKo

strongly disagree with this being a user preference, the user installs coala-html and thus wants to use it, we shouldn't defer steps to do them lazily

sils avatar May 22 '16 19:05 sils

i.e. I agree very much with abdeali that if possible we should do this upon installation

sils avatar May 22 '16 19:05 sils

@AbdealiJK @sils1297 So what happens now is that the user installs coala-html that contains angular packages including bower.json. Now the app requires static files that are being fetched using bower upon bower install. It's essential that those bower components are located within the directory that stores angular app. This particular directory is specified by user only when coala-html is run upon installation.
Thoughts?

tushar-rishav avatar May 22 '16 19:05 tushar-rishav

Maybe we can download the stuff once and then copy it whenever it's needed.

This is not important and shouldn't be addressed before we get a main version out of the door.

sils avatar May 22 '16 19:05 sils

Alright :+1:

tushar-rishav avatar May 22 '16 19:05 tushar-rishav

I was looking at code and found that jupyterlab does it in a fairly nice way. They have some method where if I install from git, the setup.py auto downloads things from bower/node and builds a single JS file. This JS file is inside the jupyterlab module, and it gets packages with PyPI. So, when running pip install upload and so on, it auto runs bower on the system and only adds the built files in the .tar.gz on PyPI.

AbdealiLoKo avatar Aug 29 '16 11:08 AbdealiLoKo

Strongly agree that running bower install after pip install is wrong. The pip package creation process should include all dependencies, including any bower components available at the time of release. Those are the versions which it was tested with , and may disappear later (especially as bower is being decommissioned)

jayvdb avatar Apr 18 '18 12:04 jayvdb

Unassigning due to inactivity

jayvdb avatar Apr 18 '18 12:04 jayvdb