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

Make test project target .net core and framework #76

Conversation

frederik-h
Copy link
Contributor

@frederik-h frederik-h commented Oct 10, 2018

In PR #74, @TChatzigiannakis mentioned that his tests don't work because they are no longer compatible with dotnet core. I have replaced the test project with a dotnet core style project
which targets both dotnet core and dotnet framework.

You can run the dotnet core tests with dotnet test Tests.csproj /p:TargetFramework=netcoreapp2.0.
For the dotnet framework tests, you should replace netcoreapp2.0 by net461.

frederik-h and others added 2 commits October 10, 2018 20:22
Change the test project to a dotnet core style project
which targets both dotnet core and dotnet framework.

Further changes:
* Adapt LLVMSharp.csproj correspondingly
* Remove unnecessary packages.config
  (package references are now in project file)
  and AssemblyInfo
With the older version, the project would no longer
build on Windows (probably due to missing package references)
after the change introduced by the last commit.
src/LLVMSharp.csproj Outdated Show resolved Hide resolved
Copy link
Contributor

@mjsabby mjsabby left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

frederik-h added 2 commits November 14, 2018 15:33
Build only LLVMSharp project intially
and then run tests with explicit target framework
netcoreapp2.0
@frederik-h
Copy link
Contributor Author

@mjsabby Is this change still relevant now that the OOAPI has been merged (for the master branch) or should I close the PR?

@TChatzigiannakis
Copy link
Contributor

TChatzigiannakis commented Dec 4, 2018

Is this change still relevant now that the OOAPI has been merged (for the master branch) or should I close the PR?

Yes, I think this PR isn't relevant anymore. However, if you could find a way to move files around so that we can run the tests with just dotnet test instead of having to specify the project file (as we do now), that would be very welcome I think.

@frederik-h
Copy link
Contributor Author

Yes, I think this PR isn't relevant anymore. However, if you could find a way to move files around so that we can run the tests with just dotnet test instead of having to specify the project file (as we do now), that would be very welcome I think.

I suppose the problem is that dotnet testtries to take up test from the LLVMSharp project file as well, right? I have implemented a workaround for this. Works for me. See #86.

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

Successfully merging this pull request may close these issues.

3 participants