Skip to content
This repository has been archived by the owner on Mar 14, 2019. It is now read-only.

mrt add collectionFS fails #97

Closed
Rebolon opened this issue Oct 28, 2013 · 57 comments
Closed

mrt add collectionFS fails #97

Rebolon opened this issue Oct 28, 2013 · 57 comments

Comments

@Rebolon
Copy link

Rebolon commented Oct 28, 2013

Trying to add collectionFS to a new project :
mrt create testApp
mrt add collectionFS

✓ cfs-public-folder
tag: https://github.com/nooitaf/meteor-cfs-public-folder.git#v0.0.3

/home/action/.nvm/v0.10.11/lib/node_modules/meteorite/lib/sources/git.js:151
hrow "There was a problem checking out " + self.checkoutType + ": " + (self.co
^
There was a problem checking out tag: v0.0.3

I also have the problem on previous installed app : collectionFS and cfs-public-folder are listed in packages folder but mrt always return the same problem.
When i edit the cfs-public-folder/smart.json of that app it shows me i'm on v0.0.2 and not in v0.0.3, so it seems that update fails sinc some time ago.

@nooitaf
Copy link
Contributor

nooitaf commented Oct 28, 2013

try mrt remove --system

this will remove all mrt files on your system..

and then mrt update in the folder of your app..

@Rebolon
Copy link
Author

Rebolon commented Oct 28, 2013

i did a mrt remove --system in one folder
then mrt update in app folder
But i have the same error

@nooitaf
Copy link
Contributor

nooitaf commented Oct 28, 2013

i'm sorry .. it's mrt uninstall --system
then mrt update

if that doesnt work try npm install -g meteorite or the sudo equvalent sudo -H npm install -g meteorite

@Rebolon
Copy link
Author

Rebolon commented Oct 28, 2013

ok it works,
thx

@Rebolon Rebolon closed this as completed Oct 28, 2013
@Rebolon
Copy link
Author

Rebolon commented Oct 28, 2013

Ok it works correctly now


Benjamin RICHARD
about.me/benjamin.richard/

2013/10/28 nooitaf [email protected]

i'm sorry .. it's mrt uninstall --system
then mrt update

if that doesnt work try npm install -g meteorite or the sudo equvalent sudo
-H npm install -g meteorite


Reply to this email directly or view it on GitHubhttps://github.com//issues/97#issuecomment-27210653
.

@nooitaf
Copy link
Contributor

nooitaf commented Oct 28, 2013

nice, ... i hope the next version bump will be smoother..

@pnunn
Copy link

pnunn commented Jan 16, 2014

I can't get collectionFS to install. I did mrt install cfs-public-folder first as I thought it was a dependancy, but then I got the error above when I tried to do mrt install collectionFS. I tried the mrt uninstall and update step, which simply re-installes roles, iron-router and errors that I had installed previously, then when I do mrt install collectionFS I don't get any errors, but the package is still not installed.
Any ideas?

@copleykj
Copy link
Member

Use mrt add collectionFS instead of mrt install

@pnunn
Copy link

pnunn commented Jan 16, 2014

How embarrassing... :( My excuse, its 43 C here today.. thanks

@copleykj
Copy link
Member

lol, its all good.. I think we've all had those moments :)

@pnunn
Copy link

pnunn commented Mar 20, 2014

Updated meteor to 0.7.2 then mrt update, but I'm getting errors from collectionFS.

Problem installing collectionFS
✘ [branch: https://github.com/CollectionFS/Meteor-CollectionFS.git#devel] conflicts with [latest]
Can't resolve dependencies! Use --force if you don't mrt taking a wild guess and running your app anway.

I tried mrt uninstall --system and mrt update, but same issue. Any ideas?

@raix
Copy link

raix commented Mar 20, 2014

@pnunn good question, we had to publish the new version to get tests working on travis. I'm guessing you are on the "old" api? Could try editing your app smart.json pointing collectionFS to the Master and tag v0.0.3? (I'm not a mrt expert) we will add a message/guide on the readme and meteor talk.

We are pushing the new version to Master before april.

There are massive speed and Security improvements in the new api, so when Ready we will recommend moving to the new api.

@pnunn
Copy link

pnunn commented Mar 20, 2014

Ok, I've added {"tag": "master/v0.0.3" } to collectionFS (also tried "master/0.0.3"), uninstall --system, mrt update but same error still. Not sure what you mean by "old" and "new" api's but is it easier to just update to what's new? Has it changed much?

@raix
Copy link

raix commented Mar 20, 2014

Its changed a lot - for the better, hope you guys agree - the new api should be more powerful, easy much faster since we are piping streams of data and more secure because we have a real rest point rigged - adding download to allow/deny rules - we do set headers now for forcing saveAs.

It also has a more flexible storage capability since you can easily switch between the current 3 storage adapters:

  • FS.Store.FileSystem
  • FS.Store.GridFS - (note that we use real GridFS now)
  • FS.StoreS3

We've made it fairly easy for others to create new storage adapters (I'm personally looking forward to storage adapters for stuff like dropbox/drive etc.)

We have added a FS.File object for handling files - this has many advantages:

  • We have a direct api on the file object making it easier to work with
  • We are ready for Blaze since a file contains helper oriented functions like {{url}}, {{url store="thumbnail"}, {{url download=true}}
  • If you add a FS.File to a document in a normal Meteor.Collection its converted to a ejson reference - when retrieved from the Meteor.Collection its converted back again to FS.File with api and all (may require that the FS.Collection is actually published for the user)
  • You can easily attachData like: file/blob/buffer/url/path (works at direct insert to a FS.Collection too)
  • Easy access to the streaming api etc.

We are writing the last parts of the transformation streams api - its basically file handlers / beforeSave but for streams - again making it possible to use gm streaming api or crypting/decrypting data etc.

The most important part: The new api already got more tests rigged that the old (it had no tests)

We are not releasing until we have a fair amount of test coverage - The last week of this month (March)/next week is test writing week - so the final api will be done this week.

But theres a bunch of devs already using/testing it - and we are close to a stable api - basically just the transform that is missing.

We are super low on documentation at the moment - but we have a place where early adapters can add comments what parts needs documentation #206

But when all is said - users should be able to use the old api if they want to, I'm sure its possible - I hope the new package system will resolve version dependencies better in the future.

Hmm, not 0.0.3 it should be 0.3.7 Maybe try:

{
  "meteor": {
    "tag": "v0.7.2"
  },
  "packages": {
    "collectionFS": "0.3.7",
  }
}

Feedback is much welcome

@pnunn
Copy link

pnunn commented Mar 20, 2014

Sounds really interesting raix, looking forward to trying it all out.
smart.json as it is just has { packages: { "collectionFS" : {} in it,
changing this to, as you suggested, "collectionFS: "0.3.7" seems to have done the trick (interestingly smart.lock already had 0.3.7 in it.. but there you go).

@raix
Copy link

raix commented Mar 20, 2014

Cool - I'll update the master readme - Thanks!!

@lap123
Copy link

lap123 commented Mar 21, 2014

fyi, I am on the devel branch and I have the same conflict problem as described earlier. Happened when I upgraded to meteor 0.7.2.

@raix
Copy link

raix commented Mar 21, 2014

ok, edit your smart.json use version: "collectionFS: "0.3.7"

@lap123
Copy link

lap123 commented Mar 21, 2014

huum, I have no intention to use the old api, thanks.
Just wanted to emphasize that the problem is not isolated.

@raix
Copy link

raix commented Mar 21, 2014

@lap123 sorry - whats the reference in smart.json? (we use mtr on the travis test server - so it should work)
@aldeed I think the problem you had to workaround could be related to this? (I think we can remove the smart.lock again - this issue is isolated to the smart.json files...)

@raix
Copy link

raix commented Mar 21, 2014

@aldeed I'm getting this result when trying mtr:

$ mrt add collectionFS
✓ cfs-base-package
    branch: https://github.com/CollectionFS/Meteor-cfs-base-package.git#master
✓ cfs-file
    branch: https://github.com/CollectionFS/Meteor-cfs-file.git#master
✓ cfs-ejson-file
    branch: https://github.com/CollectionFS/Meteor-cfs-ejson-file.git#master
✓ cfs-collection
    branch: https://github.com/CollectionFS/Meteor-cfs-collection.git#master
✓ cfs-access-point
    branch: https://github.com/CollectionFS/Meteor-cfs-access-point.git#master
✓ cfs-worker
    branch: https://github.com/CollectionFS/Meteor-cfs-worker.git#master
✓ cfs-upload-http
    branch: https://github.com/CollectionFS/Meteor-cfs-upload-http.git#master
✓ cfs-filesaver
    branch: https://github.com/CollectionFS/Meteor-cfs-filesaver.git#master
✓ cfs-storage-adapter
    branch: https://github.com/CollectionFS/Meteor-cfs-storage-adapter.git#master
✓ http-methods
    branch: https://github.com/CollectionFS/Meteor-http-methods.git#master
✓ cfs-upload-ddp
    branch: https://github.com/CollectionFS/Meteor-cfs-upload-ddp.git#master
✓ http-publish
    branch: https://github.com/CollectionFS/Meteor-http-publish.git#master
✓ cfs-gridfs
    branch: https://github.com/CollectionFS/Meteor-cfs-gridfs.git#master
✓ cfs-tempstore
    branch: https://github.com/CollectionFS/Meteor-cfs-tempstore.git#master

Problem installing collectionFS
  ✘ [branch: https://github.com/CollectionFS/Meteor-CollectionFS.git#devel] conflicts with [latest]
Can`t resolve dependencies! Use --force if you don`t mind mrt taking a wild guess and running your app anyway.

@lap123 is it the same result you get?

@lap123
Copy link

lap123 commented Mar 21, 2014

yes I get the same results. Sorry I should have pasted it from the beginning.
My smart.json is:

  "packages": {
    "collection2": {},
    "bootstrap-3": {},
    "moment": {},
    "iron-router": {},
    "collectionFS": {
      "git": "https://github.com/CollectionFS/Meteor-CollectionFS.git",
      "branch": "devel"
    },
    "cfs-gridfs": {
      "git": "https://github.com/CollectionFS/Meteor-cfs-gridfs.git",
      "branch": "master"
    },
    "cfs-filesystem": {
      "git": "https://github.com/CollectionFS/Meteor-cfs-filesystem.git",
      "branch": "master"
    },
    "cfs-handlebars": {
      "git": "https://github.com/CollectionFS/Meteor-cfs-handlebars.git",
      "branch": "master"
    },
    "bootstrap-wysiwyg": {}
  }

@aldeed
Copy link
Contributor

aldeed commented Mar 21, 2014

@lap123, if I'm understanding what's been happening correctly, you just need to change it to "collectionFS": {} and that should resolve your issues.

@raix, our issues with travis testing are a bit more complicated. As I said, we need to do mrt update --force and then commit the updated smart.lock.

@raix
Copy link

raix commented Mar 21, 2014

I can't add via mtr... Ive bumped all packages to make sure everything is up to date - and now its failing at cfs-file... This is a mtr issue right? (the new package system hopefully resolves this)

@aldeed
Copy link
Contributor

aldeed commented Mar 21, 2014

To put it clearly, as of right now, the correct way to use each collectionFS version right now is:

0.4.0 (new API) smart.json:

"collectionFS": {}

0.3.7 (old API) smart.json:

"collectionFS": "0.3.7"

Which also means that simply doing mrt add collectionFS adds 0.4.0.

@raix
Copy link

raix commented Mar 21, 2014

ughh - I have removed 2 smart.lock files in attept to debug..

@aldeed
Copy link
Contributor

aldeed commented Mar 21, 2014

It should work with mrt update --force. I will try out. I don't think this is really an issue with anything. It's just that we have a bit of a double dependency when running tests, and the testing dependency says to use "local" while our smart.json in collectionFS pkg says to use the master branch. So meteorite doesn't know which is correct.

By using --force, we tell it it's OK to use the "local" files for both dependencies, and then we lock that decision in the smart.lock file.

@raix
Copy link

raix commented Mar 21, 2014

mtr should be able to handle that packages may have same deps?

@aldeed
Copy link
Contributor

aldeed commented Mar 21, 2014

For an app, I don't think --force should be necessary, but for running tests it is. For an app, we just need to make sure everyone is using "collectionFS": {} in smart.json. The easy way to fix would be to do mrt remove collectionFS and then mrt add collectionFS.

@raix
Copy link

raix commented Mar 21, 2014

Ok, mrt is not actually checking or too clever, now I get:

Problem installing collectionFS
  ✘ [latest] conflicts with [branch: https://github.com/CollectionFS/Meteor-CollectionFS.git#devel];
keeping [branch: https://github.com/CollectionFS/Meteor-CollectionFS.git#devel]

its kinda exactly the same...
Edit: I'm using --force

@aldeed
Copy link
Contributor

aldeed commented Mar 21, 2014

Maybe we have some packages that still point to the devel branch in their smart.json. That should be changed.

@raix
Copy link

raix commented Mar 21, 2014

Nope checked all packages, none of them are doing circular references to collectionFS. Maybe do a cross check to see if there are circular deps between packages?

@raix
Copy link

raix commented Mar 21, 2014

Should we add --force to the read me?

@aldeed
Copy link
Contributor

aldeed commented Mar 21, 2014

Actually, not quite true. I just checked and found that the cfs-worker pkg was still referencing the devel branch. I changed it to reference the latest release, and that fixed this issue (for me).

@aldeed
Copy link
Contributor

aldeed commented Mar 21, 2014

So, no, --force should not be in the readme, unless you want to document how to run the tests in the contributor section.

@nooitaf
Copy link
Contributor

nooitaf commented Mar 21, 2014

just created a testapp, looks good:

This project is already at Meteor 0.7.2, the latest release.

$ mrt add collectionFS
✓ cfs-base-package
    branch: https://github.com/CollectionFS/Meteor-cfs-base-package.git#master
✓ cfs-file
    branch: https://github.com/CollectionFS/Meteor-cfs-file.git#master
✓ cfs-ejson-file
    branch: https://github.com/CollectionFS/Meteor-cfs-ejson-file.git#master
✓ cfs-collection
    branch: https://github.com/CollectionFS/Meteor-cfs-collection.git#master
✓ cfs-access-point
    branch: https://github.com/CollectionFS/Meteor-cfs-access-point.git#master
✓ cfs-worker
    branch: https://github.com/CollectionFS/Meteor-cfs-worker.git#master
✓ cfs-upload-http
    branch: https://github.com/CollectionFS/Meteor-cfs-upload-http.git#master
✓ cfs-filesaver
    branch: https://github.com/CollectionFS/Meteor-cfs-filesaver.git#master
✓ cfs-storage-adapter
    branch: https://github.com/CollectionFS/Meteor-cfs-storage-adapter.git#master
✓ http-methods
    branch: https://github.com/CollectionFS/Meteor-http-methods.git#master
✓ cfs-upload-ddp
    branch: https://github.com/CollectionFS/Meteor-cfs-upload-ddp.git#master
✓ http-publish
    branch: https://github.com/CollectionFS/Meteor-http-publish.git#master
✓ cfs-gridfs
    branch: https://github.com/CollectionFS/Meteor-cfs-gridfs.git#master
✓ cfs-tempstore
    branch: https://github.com/CollectionFS/Meteor-cfs-tempstore.git#master
✓ power-queue
    branch: https://github.com/CollectionFS/Meteor-power-queue.git#master
✓ reactive-list
    branch: https://github.com/CollectionFS/Meteor-reactive-list.git#master
✓ reactive-property
    branch: https://github.com/CollectionFS/Meteor-reactive-property.git#master
✓ micro-queue
    branch: https://github.com/CollectionFS/Meteor-micro-queue.git#master
✓ collectionFS
    tag: https://github.com/CollectionFS/Meteor-CollectionFS.git#devel#v0.4.1

Done installing smart packages

@raix
Copy link

raix commented Mar 21, 2014

@aldeed no no, btw good catch ive been looking at all smart.json too,
@nooitaf sounds good

@raix
Copy link

raix commented Mar 21, 2014

Yeah, it works here too - thanks guys!!
@aldeed tests are failing on travis, Ill test some more on this using mbr locally

@aldeed
Copy link
Contributor

aldeed commented Mar 21, 2014

I think the cfs-access-point tests are failing because you removed smart.lock. It was working yesterday after I committed smart.lock.

@aldeed
Copy link
Contributor

aldeed commented Mar 21, 2014

This is what worked for me:

$ mrt update --force
If `smart.lock` is changed:
$ git add smart.lock
$ git commit -m "message"

After you run mrt update --force, you will be able to run tests locally with mrt test-packages ./ and they will pass on travis as long as you commit the updated smart.lock.

@raix
Copy link

raix commented Mar 21, 2014

but we should not have to include smart.lock... I rather have it work like normal. Im thinking this issue we are experiencing could be due to the fact that we include info about git and branch - it should be ok to remove? e.g. like collectionFS: {} could perhaps work in general?

@raix
Copy link

raix commented Mar 21, 2014

Im turning in - I've refactored the transformWrite/read - it works when just piping the readStream and writeStream in the transform - so thats good news - its saved correctly etc. but the fileobj is not updated with an extra store (my qa got org + transform) - later...

@raix
Copy link

raix commented Mar 22, 2014

@aldeed I've fixed travis test script - Ive updated all .travis scripts to use our own version for this - using your mtr update --force

@aldeed
Copy link
Contributor

aldeed commented Mar 22, 2014

OK, nice. Yeah, I don't think that removing the git and branch references would help, because it still evaluates as "latest" whereas the tests try to use "local" of the package being tested. Since "latest" != "local", mrt doesn't know what to do. But I could be wrong, have been once or twice in my life. :)

@raix
Copy link

raix commented Mar 22, 2014

I still have smart.json open for all packages, I'll revert - it doesnt make any difference, modifying the test script solved it.

@nooitaf
Copy link
Contributor

nooitaf commented Mar 23, 2014

Did the same thing i did 2 days ago, but now it throws this error..

$ meteor create cfs-test
$ cd ctf-test
$ meteor --version
Release 0.7.2
$ mrt add collectionFS
Problem installing cfs-base-package
✘ [branch: https://github.com/CollectionFS/Meteor-cfs-base-package.git#master] conflicts with [latest]
Can't resolve dependencies! Use --force if you don't mrt taking a wild guess and running your app anway.

.. should i use the --force ?

@raix
Copy link

raix commented Mar 23, 2014

@nooitaf hmmm, maybe I did this? I did remove the git#master reference while testing making travis work - so I guess mrt tracks that as latest? I have then reverted to the git# reference creating a conflict?
So I think mrt update --force should solve it? (I'm not a mrt user, hope meteor will soon have real package support... should happen soon)

Let me know if it works - I'll add it to the readme.md

@nooitaf
Copy link
Contributor

nooitaf commented Mar 23, 2014

@raix the --force doesn't seem to solve it.. well, it goes on installing and i can start it but then it complains about an undefined function FS.Store.Filesystem(..

just a little side thought .. shouldn't those packages have tag numbers instead of #master?
i see tags as "stable" snapshots of the master, and i assume mrt gets confused when the same "branch/tag" changes.

$ mrt update --force
✓ cfs-base-package
    branch: https://github.com/CollectionFS/Meteor-cfs-base-package.git#master
✓ cfs-filesaver
    branch: https://github.com/CollectionFS/Meteor-cfs-filesaver.git#master
✓ cfs-storage-adapter
    branch: https://github.com/CollectionFS/Meteor-cfs-storage-adapter.git#master
✓ cfs-collection
    branch: https://github.com/CollectionFS/Meteor-cfs-collection.git#master
✓ http-methods
    branch: https://github.com/CollectionFS/Meteor-http-methods.git#master
✓ cfs-file
    branch: https://github.com/CollectionFS/Meteor-cfs-file.git#master
✓ cfs-filesaver
    branch: https://github.com/CollectionFS/Meteor-cfs-filesaver.git#master
✓ cfs-storage-adapter
    branch: https://github.com/CollectionFS/Meteor-cfs-storage-adapter.git#master
✓ cfs-collection
    branch: https://github.com/CollectionFS/Meteor-cfs-collection.git#master
✓ http-methods
    branch: https://github.com/CollectionFS/Meteor-http-methods.git#master
✓ cfs-upload-ddp
    branch: https://github.com/CollectionFS/Meteor-cfs-upload-ddp.git#master
✓ cfs-upload-http
    branch: https://github.com/CollectionFS/Meteor-cfs-upload-http.git#master
✓ http-publish
    branch: https://github.com/CollectionFS/Meteor-http-publish.git#master
✓ cfs-gridfs
    branch: https://github.com/CollectionFS/Meteor-cfs-gridfs.git#master
✓ cfs-tempstore
    branch: https://github.com/CollectionFS/Meteor-cfs-tempstore.git#master
✓ cfs-access-point
    branch: https://github.com/CollectionFS/Meteor-cfs-access-point.git#master
✓ http-publish
    branch: https://github.com/CollectionFS/Meteor-http-publish.git#master
✓ cfs-gridfs
    branch: https://github.com/CollectionFS/Meteor-cfs-gridfs.git#master
✓ cfs-tempstore
    branch: https://github.com/CollectionFS/Meteor-cfs-tempstore.git#master
✓ power-queue
    branch: https://github.com/CollectionFS/Meteor-power-queue.git#master
✓ reactive-list
    branch: https://github.com/CollectionFS/Meteor-reactive-list.git#master
✓ reactive-property
    branch: https://github.com/CollectionFS/Meteor-reactive-property.git#master
✓ micro-queue
    branch: https://github.com/CollectionFS/Meteor-micro-queue.git#master
Problem installing cfs-base-package
  ✘ [branch: https://github.com/CollectionFS/Meteor-cfs-base-package.git#master] conflicts with [latest]; overriding with [branch: https://github.com/CollectionFS/Meteor-cfs-base-package.git#master]
Problem installing cfs-collection
  ✘ [branch: https://github.com/CollectionFS/Meteor-cfs-collection.git#master] conflicts with [latest]; overriding with [branch: https://github.com/CollectionFS/Meteor-cfs-collection.git#master]
Problem installing cfs-file
  ✘ [branch: https://github.com/CollectionFS/Meteor-cfs-file.git#master] conflicts with [latest]; overriding with [branch: https://github.com/CollectionFS/Meteor-cfs-file.git#master]
Problem installing cfs-upload-http
  ✘ [branch: https://github.com/CollectionFS/Meteor-cfs-upload-http.git#master] conflicts with [latest]; overriding with [branch: https://github.com/CollectionFS/Meteor-cfs-upload-http.git#master]
Problem installing cfs-access-point
  ✘ [branch: https://github.com/CollectionFS/Meteor-cfs-access-point.git#master] conflicts with [latest]; overriding with [branch: https://github.com/CollectionFS/Meteor-cfs-access-point.git#master]
✓ collectionFS
    tag: https://github.com/CollectionFS/Meteor-CollectionFS.git#v0.4.3
✓ cfs-ejson-file
    tag: https://github.com/CollectionFS/Meteor-cfs-ejson-file.git#v0.0.5
✓ cfs-worker
    tag: https://github.com/CollectionFS/Meteor-cfs-worker.git#v0.0.11

Done installing smart packages

@raix
Copy link

raix commented Mar 23, 2014

FS.FileSystem

@nooitaf
Copy link
Contributor

nooitaf commented Mar 23, 2014

Your example gist https://gist.github.com/raix/9720064 said FS.Store.Filesystem... which seems to be working now. I just forgot to install the cfs-filesystem, also using the --force ofc :D

mrt add collectionFS --force throws problems, but seems to be the wonder call
mrt add cfs-filesystem --force throws problems, but adds that missing function of mine

i made a little repo of the working thingy. https://github.com/nooitaf/meteor-cfs-example-image-upload

@raix
Copy link

raix commented Mar 23, 2014

Yep, sorry was on iPhone, FS.Store.FileSystem (with cap. "S") but true one have to add the storage adapter packages, I'll just put a note in the gist about it.
Yep mrt is not clever enough to resolve correctly.

@raix
Copy link

raix commented Mar 23, 2014

Cool about the repo, we should link from the readme :)
And, the api is not likly to change, we need some work in the fileworker area and test writing, but the api should be ok.

@solshark
Copy link

I have similar issue.
It just doing nothing.

$ mrt add collectionFS

and silens. No errors. No messages. Nothing.

--force

helpless too

@aldeed
Copy link
Contributor

aldeed commented May 30, 2014

@solshark, I think that sometimes happens temporarily due to network issues. Maybe see if it works later.

@solshark
Copy link

Might be, but all the rest of atmosphere packages installs ok.

@solshark
Copy link

And magically it works now :)

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