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

Fixed "double-dot" extensions in output bin directory #37

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pcrockett
Copy link

When you install the NuGet package in a project and build, all the Mapnik files will be copied to the output bin directory. However all the files in the mapnik and share directories will have an extra dot in their names:

image

This change should fix that.

Sorry, I wasn't able to actually test my changes because I don't have .NET 4.5.2 specifically installed on my development machine. But I've done a similar thing for a separate project and I'm pretty sure this should fix the issue.

@Charmatzis
Copy link

related with issue #22

@pcrockett
Copy link
Author

pcrockett commented Dec 28, 2016

Not sure why the CI build is failing. It doesn't seem to have anything to do with my changes. Something about a mutex?

<mutex> is not supported when compiling with /clr or /clr:pure.

@jbrwn
Copy link
Owner

jbrwn commented Dec 28, 2016

👍 on the fix. I need the CI build to work before I can merge though. Ultimately, the CI build artifact nuget packages are what gets released on nuget.org.

Looks like the build is failing on the MSVC standard library patches. Not sure what is up with that. Unfortunately i don't have time time to debug this at the moment. If you can figure out what is going on i would gladly except your fix as part of this pull request. In the past i found the best way to debug this kind of thing is to actually login to the appveyor build worker.

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