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

Add the Data files needed for Windows to work properly. #63

Closed
wants to merge 3 commits into from

Conversation

rgleason
Copy link
Contributor

@rgleason rgleason commented Apr 6, 2019

Sean Windows downloads of the data files are not working properly, therefore Paul added the data files back into the code.

You don't need the fix for Wind Rose font size because you have merged it,ClimatologyOverlayFactory.cpp change . This compiles and works fine but there is an error code on appeveyor, however I used the file created and tested it.

Please see
#52 (comment)

Can we fix the windows download of the data files so the binary data files are not needed in the exe?

dotPauldot and others added 3 commits March 28, 2019 17:27
These files will be incluuded in the Windows setup package.
The climatology data will be installed and ready to be used by
climatology_pi as soon as climatology_pi-1.4.1-ov50-win32.exe finishes
installing the plugin. This eliminates the need for downloading the data
from some other site.
Add climatology files to data folder.
@seandepagnier
Copy link
Owner

I cannot allow binary files this size into git

If I have a slow connection and want to clone in the future, it will have to download every version of every binary file to get a full repository. Git clone cannot resume timeouts making this worse.

Notice all my plugins clone quick even will full history and OpenCPN is impossible for me to clone unless I have a good connection (I usually have unreliable) In my opinion, OpenCPN should be rebased without any binary data.

So, you need to download the data files separate. The code itself is meant to do this automatically but it's been claimed it's broken. Alternately a script or it could even be built into the build system, but not dumped in git, sorry.

@nohal
Copy link
Contributor

nohal commented Apr 6, 2019

Make it a git submodule (which you yourself do not have to clone in case you don't have good enough connection) if you are concerned about the size of this very repository.
The download is totally unusable as you probably see from the endless support requests and is making the plugin unusable for users, which I kind of thought is your target audience.

@rgleason
Copy link
Contributor Author

rgleason commented Apr 6, 2019

Ok, as I said earlier, we need to fix the automatic binary file download and file decompression. Pavel got this working in another plugin I think, Don't remember which one right now.

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.

None yet

4 participants