Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Dependencies still appear to be off #12

Open
EricWVGG opened this issue Aug 18, 2018 · 14 comments
Open

Dependencies still appear to be off #12

EricWVGG opened this issue Aug 18, 2018 · 14 comments

Comments

@EricWVGG
Copy link

EricWVGG commented Aug 18, 2018

building with 3.0.0-RC3.0.1

backgroundExecute(`
    code: 1,
    error: "error: the package PackageReference(identity: \"s3\", name: nil, path: \"https://github.com/LiveUI/S3.git\", isLocal: false) @ 3.0.0-alpha.1 contains revisioned dependencies:\n    PackageReference(identity: \"vaportesttools\", name: nil, path: \"https://github.com/LiveUI/VaporTestTools.git\", isLocal: false) @ master\n"
@Joebayld
Copy link
Collaborator

Having the same issue here..

@rafiki270
Copy link
Collaborator

Should be fixed in #13, could you guys check?

@Joebayld
Copy link
Collaborator

I'm still having problems here. Do I need the VaporTestTools dependency? I'd rather just get rid of it for my use.

@rafiki270 rafiki270 reopened this Sep 11, 2018
@rafiki270
Copy link
Collaborator

rafiki270 commented Sep 11, 2018

I see, we need it as it will be easiest way to test/mock S3 lib in your apps. I understand there is not much at the moment but there is quite lot of work coming into this ... but yeah, if there is a compatibility issue will fit that ASAP! @Joebayld

@Joebayld
Copy link
Collaborator

Got it. I'll try and sort out why this is being problematic and report back.

@rafiki270
Copy link
Collaborator

If you having issues I would assume problem will be somewhere downstream on the test tools dependencies

@jdmcd
Copy link

jdmcd commented Sep 15, 2018

@rafiki270 I am hitting this problem as well. Since this is (to my knowledge) the only Vapor 3 compatible S3 library I would classify this as a pretty critical problem. Any chance you can look into why dependencies might not be resolving? Thank you!

@rafiki270
Copy link
Collaborator

@mcdappdev really don't see what could be off :( ... any ideas?

@jdmcd
Copy link

jdmcd commented Sep 16, 2018

I'm not sure, unfortunately. Your package files seem to be ok but maybe it's something with Package.resolved?

@rausnitz
Copy link

I made a fork that only depends on Vapor and therefore avoids the aforementioned dependency problems. The fork only provides the S3 signer functionality, but if that's all you need, then it works: https://github.com/rausnitz/S3.

(I'm not sure why I had to specify the branch in Package.swift. When I tried doing from: "3.0.0" then my vapor update command would hang.)

@jdmcd
Copy link

jdmcd commented Sep 16, 2018

@rausnitz Awesome! I'll use yours. Thanks :)

@rafiki270
Copy link
Collaborator

guys, do you by any chance have any logs to share or does the process just freeze?

@lluuaapp
Copy link
Collaborator

I think this is an issue with the Swift Package Manager. I had this too, but updated the dependencies of the main project (all the latest version of the packages, some pinned to the latest version) and deleted Package.resolved. After that, run

vapor build --verbose --clean

and you will see the package that the package manager gets stuck resolving.

@EricWVGG
Copy link
Author

EricWVGG commented Oct 1, 2018

This is working for me now with RC2 (and Mojave, if that makes any difference).

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

No branches or pull requests

6 participants