New issue

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

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

Already on GitHub? Sign in to your account

Framework resolution strategy for Google.Apis.Oauth2.v2 #1663

Closed
sergey-tihon opened this Issue May 5, 2016 · 11 comments

Comments

Projects
None yet
2 participants
@sergey-tihon
Member

sergey-tihon commented May 5, 2016

Description

Cannot use Google.Apis.Oauth2.v2 with Paket for Net45 project
Paket does not setup references properly in this case

Expected behavior

image

<?xml version="1.0" encoding="utf-8"?>
<packages>
  <package id="BouncyCastle" version="1.7.0" targetFramework="net45" />
  <package id="Google.Apis" version="1.11.1" targetFramework="net45" />
  <package id="Google.Apis.Auth" version="1.11.1" targetFramework="net45" />
  <package id="Google.Apis.Core" version="1.11.1" targetFramework="net45" />
  <package id="Google.Apis.Oauth2.v2" version="1.12.0.454" targetFramework="net45" />
  <package id="log4net" version="2.0.3" targetFramework="net45" />
  <package id="Newtonsoft.Json" version="7.0.1" targetFramework="net45" />
  <package id="Zlib.Portable.Signed" version="1.11.0" targetFramework="net45" />
</packages>

Actual behavior

paket.dependencies

source https://nuget.org/api/v2

nuget Google.Apis.Oauth2.v2

paket.lock

NUGET
  remote: https://www.nuget.org/api/v2
  specs:
    BouncyCastle (1.8.1)
    Google.Apis (1.10) - framework: portable-net40+sl5+win8+wp8+wpa81
      Google.Apis.Core (>= 1.10)
      log4net (>= 2.0.3) - framework: >= net40
      Zlib.Portable.Signed (>= 1.11)
    Google.Apis.Auth (1.11.1) - framework: portable-net45+win8+wp8+wpa81
      BouncyCastle (>= 1.7) - framework: >= net45
      Google.Apis.Core (>= 1.11.1)
    Google.Apis.Core (1.11.1) - framework: portable-net40+sl5+win8+wp8+wpa81, portable-net45+win8+wp8+wpa81
      Microsoft.Bcl (>= 1.1.10) - framework: net10, net11, net20, net30, net35, net40, net40-full
      Microsoft.Bcl.Async (>= 1.0.168) - framework: net10, net11, net20, net30, net35, net40, net40-full
      Microsoft.Bcl.Build (>= 1.0.21) - framework: net10, net11, net20, net30, net35, net40, net40-full
      Microsoft.Net.Http (>= 2.2.29) - framework: net10, net11, net20, net30, net35, net40, net40-full
      Newtonsoft.Json (>= 7.0.1)
    Google.Apis.Oauth2.v2 (1.12.0.454)
      Google.Apis (>= 1.11.1) - framework: portable-net45+win8+wp8+wpa81
      Google.Apis (1.10) - framework: portable-net40+sl5+win8+wp8+wpa81
      Google.Apis.Auth (>= 1.11.1) - framework: portable-net45+win8+wp8+wpa81
      Google.Apis.Auth (1.10) - framework: portable-net40+sl5+win8+wp8+wpa81
    log4net (2.0.5)
    Microsoft.Bcl (1.1.10)
      Microsoft.Bcl.Build (>= 1.0.14)
    Microsoft.Bcl.Async (1.0.168)
      Microsoft.Bcl (>= 1.1.8)
    Microsoft.Bcl.Build (1.0.21) - import_targets: false
    Microsoft.Net.Http (2.2.29)
      Microsoft.Bcl (>= 1.1.10)
      Microsoft.Bcl.Build (>= 1.0.14)
    Newtonsoft.Json (8.0.3) - framework: portable-net40+sl5+win8+wp8+wpa81, portable-net45+win8+wp8+wpa81
    Zlib.Portable.Signed (1.11) - framework: portable-net40+sl5+win8+wp8+wpa81

image

image

Known workarounds

Use NuGet =(

Related information

  • Paket 2.64.0.0

@forki forki closed this May 9, 2016

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki May 9, 2016

Member

fixed in 2.64.2

Member

forki commented May 9, 2016

fixed in 2.64.2

@sergey-tihon

This comment has been minimized.

Show comment
Hide comment
@sergey-tihon

sergey-tihon May 9, 2016

Member

Hmm.. Looks like it does not work for indirect dependencies
image
image
Instead of
image

Member

sergey-tihon commented May 9, 2016

Hmm.. Looks like it does not work for indirect dependencies
image
image
Instead of
image

@sergey-tihon sergey-tihon reopened this May 9, 2016

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki May 9, 2016

Member

can you try update -f ?

Member

forki commented May 9, 2016

can you try update -f ?

@sergey-tihon

This comment has been minimized.

Show comment
Hide comment
@sergey-tihon

sergey-tihon May 9, 2016

Member

Better but still not all
image
image

Member

sergey-tihon commented May 9, 2016

Better but still not all
image
image

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki May 9, 2016

Member

What is still missing? Will look into it tomorrow.
On May 9, 2016 7:10 PM, "Sergey Tihon" notifications@github.com wrote:

Better but still not all
[image: image]
https://cloud.githubusercontent.com/assets/1197905/15121185/fff53470-1621-11e6-94d4-cd869713a833.png
[image: image]
https://cloud.githubusercontent.com/assets/1197905/15121189/07523812-1622-11e6-9168-7b1899915a7e.png


You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub
#1663 (comment)

Member

forki commented May 9, 2016

What is still missing? Will look into it tomorrow.
On May 9, 2016 7:10 PM, "Sergey Tihon" notifications@github.com wrote:

Better but still not all
[image: image]
https://cloud.githubusercontent.com/assets/1197905/15121185/fff53470-1621-11e6-94d4-cd869713a833.png
[image: image]
https://cloud.githubusercontent.com/assets/1197905/15121189/07523812-1622-11e6-9168-7b1899915a7e.png


You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub
#1663 (comment)

@sergey-tihon

This comment has been minimized.

Show comment
Hide comment
@sergey-tihon

sergey-tihon May 9, 2016

Member

All yellow warn messages Google.Apis, log4net & Zlib

Member

sergey-tihon commented May 9, 2016

All yellow warn messages Google.Apis, log4net & Zlib

@sergey-tihon

This comment has been minimized.

Show comment
Hide comment
@sergey-tihon

sergey-tihon May 10, 2016

Member

Very interesting case here... Look at lock file

    Google.Apis.Oauth2.v2 (1.12.0.454)
      Google.Apis (>= 1.11.1) - framework: >= net45, portable-net45+win8+wp8+wpa81
      Google.Apis (1.10) - framework: net40, portable-net40+sl5+win8+wp8+wpa81
      Google.Apis.Auth (>= 1.11.1) - framework: >= net45, portable-net45+win8+wp8+wpa81
      Google.Apis.Auth (1.10) - framework: net40, portable-net40+sl5+win8+wp8+wpa81

Google.Apis has special version with net45 support.

Google.Apis (1.10):
image

Google.Apis (1.11.1):
image

But Paket by design have to choose one version for solution, and here it choose min version 1.10

    Google.Apis (1.10) - framework: net40, portable-net40+sl5+win8+wp8+wpa81
      Google.Apis.Core (>= 1.10)
      log4net (>= 2.0.3) - framework: >= net40
      Zlib.Portable.Signed (>= 1.11)

and this decisions does not work for net45 projects.

Member

sergey-tihon commented May 10, 2016

Very interesting case here... Look at lock file

    Google.Apis.Oauth2.v2 (1.12.0.454)
      Google.Apis (>= 1.11.1) - framework: >= net45, portable-net45+win8+wp8+wpa81
      Google.Apis (1.10) - framework: net40, portable-net40+sl5+win8+wp8+wpa81
      Google.Apis.Auth (>= 1.11.1) - framework: >= net45, portable-net45+win8+wp8+wpa81
      Google.Apis.Auth (1.10) - framework: net40, portable-net40+sl5+win8+wp8+wpa81

Google.Apis has special version with net45 support.

Google.Apis (1.10):
image

Google.Apis (1.11.1):
image

But Paket by design have to choose one version for solution, and here it choose min version 1.10

    Google.Apis (1.10) - framework: net40, portable-net40+sl5+win8+wp8+wpa81
      Google.Apis.Core (>= 1.10)
      log4net (>= 2.0.3) - framework: >= net40
      Zlib.Portable.Signed (>= 1.11)

and this decisions does not work for net45 projects.

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki May 10, 2016

Member

yes. we don't support such constellation.

Member

forki commented May 10, 2016

yes. we don't support such constellation.

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki May 10, 2016

Member

I will try to make it work for

source https://nuget.org/api/v2
framework: net45
nuget Google.Apis.Oauth2.v2 1.12.0.454
Member

forki commented May 10, 2016

I will try to make it work for

source https://nuget.org/api/v2
framework: net45
nuget Google.Apis.Oauth2.v2 1.12.0.454
@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki May 10, 2016

Member

it's giving me

FRAMEWORK: NET45
NUGET
  remote: https://www.nuget.org/api/v2
  specs:
    BouncyCastle (1.8.1)
    Google.Apis (1.11.1)
      Google.Apis.Core (>= 1.11.1)
      log4net (>= 2.0.3)
      Zlib.Portable.Signed (>= 1.11)
    Google.Apis.Auth (1.11.1)
      BouncyCastle (>= 1.7)
      Google.Apis.Core (>= 1.11.1)
    Google.Apis.Core (1.11.1)
      Newtonsoft.Json (>= 7.0.1)
    Google.Apis.Oauth2.v2 (1.12.0.454)
      Google.Apis (>= 1.11.1)
      Google.Apis.Auth (>= 1.11.1)
    log4net (2.0.5)
    Newtonsoft.Json (8.0.3)
    Zlib.Portable.Signed (1.11)

that sounds ok

Member

forki commented May 10, 2016

it's giving me

FRAMEWORK: NET45
NUGET
  remote: https://www.nuget.org/api/v2
  specs:
    BouncyCastle (1.8.1)
    Google.Apis (1.11.1)
      Google.Apis.Core (>= 1.11.1)
      log4net (>= 2.0.3)
      Zlib.Portable.Signed (>= 1.11)
    Google.Apis.Auth (1.11.1)
      BouncyCastle (>= 1.7)
      Google.Apis.Core (>= 1.11.1)
    Google.Apis.Core (1.11.1)
      Newtonsoft.Json (>= 7.0.1)
    Google.Apis.Oauth2.v2 (1.12.0.454)
      Google.Apis (>= 1.11.1)
      Google.Apis.Auth (>= 1.11.1)
    log4net (2.0.5)
    Newtonsoft.Json (8.0.3)
    Zlib.Portable.Signed (1.11)

that sounds ok

@sergey-tihon

This comment has been minimized.

Show comment
Hide comment
@sergey-tihon

sergey-tihon May 10, 2016

Member

Thank you, I think that it is OK for now (same result on my machine)
Now I have to understand how Paket modified all my proj files again)

Member

sergey-tihon commented May 10, 2016

Thank you, I think that it is OK for now (same result on my machine)
Now I have to understand how Paket modified all my proj files again)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment