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

Remove tarballs from repository #6

Closed
peanutfun opened this issue Nov 8, 2021 · 2 comments
Closed

Remove tarballs from repository #6

peanutfun opened this issue Nov 8, 2021 · 2 comments

Comments

@peanutfun
Copy link

Description

This repository currently contains tarballs of the main software dependencies and a pre-compiled GUI. Tarballs are binary code that cannot be interpreted by Git. This leads to a bloat in repository size, especially when multiple versions of the tarballs are committed. In my perception, including the tarballs is not necessary: The dependencies can be installed through usual package managers (see #5), or downloaded from their respective repositories or websites.

The GUI, on the other hand, is pre-compiled for Windows and cannot be used for other platforms. This goes against the idea of a repository containing platform-independent source code. While I very much like the idea of ready-to-use binaries, I recommend uploading the GUI package somewhere else, e.g. directly to https://zenodo.org/ or simply to a Google Drive.

Proposal

  • Remove tarballs of dependency packages, updating installation instructions accordingly (in line with Add OS-dependent installation instructions #5).
  • Remove GUI package tarball. Upload it to a suitable different repository, and provide a link in the README.md.

Related issues

@mithun218
Copy link
Owner

I have removed the tar balls as you suggested.

@peanutfun
Copy link
Author

Thank you! Sourceforge is a great place to store the GUI 👍

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