Mobx: Bower

[noxecane]

Could you please make effort to host it on bower. I have my build tool(gulp + browserify + babel) setup to use bower as a repository of vendor dependencies

[richotaylor]

+1

[mweststrate]

I added the package to bower @danceasarxx. Can you verify the current setup is working as expected?

[noxecane]

actually there's no bower.json hence no main file. What I actually do is compile all vendor files into one using a gulp plugin, and it uses the main attribute in bower.json to know what file to add to the list. So I wrote mine

{
  "name": "mobservable",
  "homepage": "https://github.com/mweststrate/mobservable",
  "version": "1.2.1",
  "main": "dist/mobservable.js"
}

[mweststrate]

Apologies, forgot to push a new tag. Should be fixed now

[PhiLhoSoft]

Head ups:
bower lookup mobx answers "Package not found".
If I search on bower.io (http://bower.io/search/?q=mobx), I get the mobservable entry...
Indeed, bower lookup mobservable answers "mobservable git://github.com/mweststrate/mobservable.git".
bower home mobservable answers:

bower mobservable#*         not-cached git://github.com/mweststrate/mobservable.git#*
bower mobservable#*            resolve git://github.com/mweststrate/mobservable.git#*
bower mobservable#*           download https://github.com/mweststrate/mobservable/archive/2.0.4.tar.gz
bower mobservable#*            extract archive.tar.gz
bower mobservable#*           resolved git://github.com/mweststrate/mobservable.git#2.0.4

so it looks reasonably up-to-date...
Apparently, Bower hasn't updated fully to the new bowser.json file, or something like that.

[PhiLhoSoft]

I stand corrected. After Bower installation, I get no single .js file to load in our project. Nothing at root, not dist folder, etc. Not sure how to deal with that. We don't want to use a CDN...

[mweststrate]

I'll take a look tomorrow!

Op wo 30 mrt. 2016 om 17:01 schreef Philippe Lhoste <
[email protected]>:

I stand corrected. After Bower installation, I get no single .js file to
load in our project. Nothing at root, not dist folder, etc. Not sure how
to deal with that. We don't want to use a CDN...

—
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub
https://github.com/mobxjs/mobx/issues/85#issuecomment-203477718

[PhiLhoSoft]

Thanks. I would love to add MobX to our AngularJS project, still relying on Bower for front-end libraries, and using Jenkins for auto-build on CI server, so we can't tinker with original files and what's not. Apparently we need your UMD build, as I used in http://plnkr.co/edit/QGhtGu

As I struggled all day with complex UI state (toolbar button disabling depending on several asynchronous (or not) conditions), I would love to use MobX to clean up this mess... :laughing:

Note: maybe this article can help you: https://medium.com/@kentcdodds/why-i-don-t-commit-generated-files-to-master-a4d76382564#.b0u1b929h

[mweststrate]

@PhiLhoSoft yes preferrably the UMD builds are not available through github, does bower install https://npmcdn.com/mobx/lib/mobx.umd.js work for you? I'll figure out whether that can be standardized through bower.

[mweststrate]

Even better: bower install https://npmcdn.com/mobx/bower.zip

[mweststrate]

@PhiLhoSoft is the above link sufficient as bower installation? It seems that bower provides no installation mechanism that doesn't require storing a generated umd build in git (which is an ugly thing to do imho)

[andykog]

@mweststrate, as far as I know, The only way to deal with this is having seperate repo for bower, like http://github.com/facebook/react-bower.git

[PhiLhoSoft]

I did a quick test at home, bower install https://npmcdn.com/mobx/bower.zip delivers the mobx.umd.js file, so it looks OK.
I will do a test tomorrow, at work. Many thanks for the quick answer.

[PhiLhoSoft]

If that's not too much asking, it would be nice to include a mobx.umd.min.js (and probably its corresponding .map file) in this Zip file. Thanks.

[mweststrate]

Updated the build scripts accordingly. Will be available from the next release (I can release it if you need it now)

[mweststrate]

@PhiLhoSoft, done, the file should be available now (from v 2.1.3)

[PhiLhoSoft]

Ah, sorry, but I can see the minified file at https://npmcdn.com/[email protected]/lib/, but for some reason, npmcdn doesn't package it in its bower.json (https://npmcdn.com/mobx/bower.zip).

We should really move out of using Bower, but currently we are stuck with it... :grimacing:

[PhiLhoSoft]

I have an issue with the latest release.
If I go to https://npmcdn.com/[email protected]/ I see bower.zip listed, but if I click on this link, I get a page showing:

Server error: ENOENT: no such file or directory, lstat '/tmp/[email protected]/tmp/[email protected]/bower.zip'

Same if I click on the link in my previous message.
I don't know if it is an issue with npmcdn or your deployment script, but it is quite blocking. :-/

If you can take a look, I would be grateful. :smile:

[mweststrate]

Server error: ENOENT: no such file or directory, lstat '/tmp/[email protected]/tmp/[email protected]/bower.zip' on https://npmcdn.com/[email protected]/bower.zip

@PhiLhoSoft seems like an issue on npmcdn to me at first sight: /tmp/[email protected]/tmp/[email protected]/bower.zip should probably be /tmp/npm-http-server/[email protected]/bower.zip or something...

cc: @mjackson

[PhiLhoSoft]

Yes, I pinged @mjackson via Twitter about this issue.
Meanwhile, I bite the bullet and get MobX (and another library with lesser Bower support) via npm, then copied the files among the other Bower libraries via Gulp (already had a gulp.js in my Maven-based build process, anyway):

gulp.task('copy-libs', function()
{
    gulp.src('./node_modules/mobx/**/*.*')
        .pipe(gulp.dest('./src/main/webapp/assets/libs/mobx'));
    gulp.src('./node_modules/ally.js/*.*')
        .pipe(gulp.dest('./src/main/webapp/assets/libs/ally.js'));
});

Seems to work so far... (easier because of no dependencies).
Would probably be a good idea to drop Bower entirely, anyway, but that's some work.

[mjackson]

Ah, sorry for the regression. It's fixed now.

Would probably be a good idea to drop Bower entirely

Yep :)

[mweststrate]

Awesome, thanks!

Op ma 30 mei 2016 17:08 schreef Michael Jackson [email protected]:

Ah, sorry for the regression. It's fixed
https://github.com/mjackson/npm-http-server/commit/ae0f9387999fd1933b91e466fda07adf9d0629c0
now.

Would probably be a good idea to drop Bower entirely

Yep :)

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/mobxjs/mobx/issues/85#issuecomment-222511530, or mute
the thread
https://github.com/notifications/unsubscribe/ABvGhOG41AtZk-8zl447DGQhmlWHhN8vks5qGv19gaJpZM4G9xvt
.