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 installer/ dir from tarball #576

Closed
apoleon opened this issue Oct 17, 2015 · 3 comments
Labels

Comments

@apoleon
Copy link
Contributor

@apoleon apoleon commented Oct 17, 2015

Hi,

please consider to provide source-only tarballs again as until recently. (version 0.237) The windows binaries in the new installer directory are of no use on non-Windows systems. They just take up space.

According to installer/nsis_plugins/README.txt, some of the plugin licenses are unknown. That makes them non-distributable due to copyright law. Debian also requires that all binaries can be built from source and therefore the sources must be at least provided together with the binaries. (I know the GPL allows you to put the source code for the (L)GPL licensed plugins on a different site but then you also have to make sure that the source remains available for as long as you distribute the dll files". Debian cannot rely on this because it is also distributed on physical media for example.)

So I have removed the installer directory for now but I would rather like to see the old source tarballs again. That was perfect. Thanks for considering.

@abma abma changed the title Pre-built windows binaries should not be shipped with source tarball remove installer/ dir from tarball Oct 17, 2015
@abma abma added the enhancement label Oct 17, 2015
@abma abma closed this in b8325a9 Oct 19, 2015
@abma

This comment has been minimized.

Copy link
Member

@abma abma commented Oct 19, 2015

thanks for reporting this!

@abma abma reopened this Oct 25, 2015
@abma

This comment has been minimized.

Copy link
Member

@abma abma commented Oct 25, 2015

damn, fix didn't work :-|

@apoleon

This comment has been minimized.

Copy link
Contributor Author

@apoleon apoleon commented Sep 22, 2016

Hi,

is there anything that can be done to resolve this bug? I mean it worked very well before and the windows files are clearly not needed on other platforms.

@abma abma closed this in 155ed73 Dec 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.