-
Notifications
You must be signed in to change notification settings - Fork 5
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
Made this package available for .net45 clients, because it is good #1
Conversation
This is compiling just fine on vsts build server. I have never tried appveyor |
This reverts commit 6d964bc.
It's because I'm using a custom build script, so it would be necessary to target the dotnet buidl with just the correct project and then you can call msbuild for the .net45 project. In addition nuget restore must be called for the .net45 as well. So can you please add the changed to the appveyor.yml (you can easily sign up for free to test everything with your fork). I'm happy to add a .net45 version in the nuget as well. |
Thanks for your updates so the correct path in yml file must be CoreHelpers.WindowsAzure.Storage.Table\CoreHelpers.WindowsAzure.Storage.Table.csproj instead of just CoreHelpers.WindowsAzure.Storage.Table.csproj I guess |
I think the build for the projects are correct now, I dont know how the version string are generated or why they are not working now? That I did not touch. |
Not sure why it's breaking but could be that you need to use in msbuild and dotnet build and restore also the AppVeyor variable: %APPVEYOR_BUILD_FOLDER% |
Ah I guess you are referencing the Assembly.cs in the MSBuild Script, just a guessing. This make the Assembly.cs now compatible with msbuild but not with core anymore. The Assmebly.cs is patched with the powershell script. |
So I recommend to different Assembly.cs files for the two different projects |
Maybe, but actually it is the dotnet build that fails: Because the version string is: That might be because this is a PR, and not simply the master branch. I have both compiling just fine, here and on vsts with the same assembly.cs |
Did you pull the change I did a couple minutes ago? I added in the appveyor.yml the information that a pull request should not increase the version number? This could be the root cause |
get this fork up to date
…do not want the .nuget folder
And we have liftoff :-) but did your build process just upload the package to nuget automatically? |
The changes are now published to NuGet as Version 1.6.45, thanks for contribution |
Cool! And you are welcome |
I found this package and have been using it in my MVC5 projects running on .net45 and later, the code is fully compatible so it is simple a matter of creating a second dll and adding it to the nupack.