Skip to content
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

Build for .NET 4.5 does not work, fails to find/restore packages #94

Closed
aliostad opened this issue Jul 21, 2017 · 10 comments
Closed

Build for .NET 4.5 does not work, fails to find/restore packages #94

aliostad opened this issue Jul 21, 2017 · 10 comments

Comments

@aliostad
Copy link

To reproduce:

  • git clone
  • open zipkin4net.sln in Visual Studio (2017)
  • build

I see packages.config files but they are not part of the solution. I added them to get around the build issue but for some reason does not work.

@aliostad
Copy link
Author

OK, this is fixed in my new PR.

@aliostad
Copy link
Author

aliostad commented Jul 24, 2017

OK, I am re-opening this for coordination. Can you please respond here rather than in the mother thread.

As I said before, you need to let me know whether pursue fixing this myself or it is something you will be doing your end and whether you need my help.

@fedj
Copy link
Collaborator

fedj commented Jul 24, 2017

@aliostad Thanks for reporting this issue. I'm currently working on a solution where net45 is available for 2017 csproj but travis will still compile netstandard until we find a solution.
Nuget can therefore be published supporting net45 and netstandard

@fedj
Copy link
Collaborator

fedj commented Jul 24, 2017

@aliostad It works on my VS2017 with this fix. Can you test ?

@aliostad
Copy link
Author

What I care about most is command line build and having nuget. Does it work on those?

@aliostad
Copy link
Author

I can see you closed this issue.

I checked and build works but does not create 4.5 binaries. The PR I sent was generating them. Am I missing something?

@fedj
Copy link
Collaborator

fedj commented Jul 24, 2017

@aliostad The PR allows to generate a nuget with both net standard and net45 supported. I will release a new nuget tomorrow as I don't have a windows machine at home. Isn't that what you needed ?

The PR you sent didn't have a green build due to some incompatibilities with linux for now. I guess we can't say your PR was generating them then.

What about creating a PR creating a script for windows ?

@fedj
Copy link
Collaborator

fedj commented Jul 25, 2017

Nuget published with version 0.4.1

@aliostad
Copy link
Author

Thanks a lot.
Fine with that PR, my point was that if you want to pick some of that work.
Yeah, I can work on a command line build for windows and .NET 4.5. I will create an issue.

Cheers

@fedj
Copy link
Collaborator

fedj commented Jul 25, 2017

Please come back to us if you need anything related to this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants