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

Please remove the hidden .mvn directory from the source tarball #57

Closed
apoleon opened this issue Jun 24, 2017 · 7 comments
Closed

Please remove the hidden .mvn directory from the source tarball #57

apoleon opened this issue Jun 24, 2017 · 7 comments
Assignees
Labels
Milestone

Comments

@apoleon
Copy link

apoleon commented Jun 24, 2017

Hi,

please consider to remove the hidden .mvn/wrapper directory and the included jar file from the source releases. It is actually not needed to build objenesis and comes without source and thus has to be removed when it is distributed by Debian or other distributions.

@henri-tremblay henri-tremblay self-assigned this Jun 25, 2017
@henri-tremblay
Copy link
Contributor

henri-tremblay commented Jun 25, 2017

These files are necessary to build objenesis with maven without having maven installed. With ./mvnw. And to fix a maven version for which we know the build is working. You should probably build using it instead. The tar.gz is done automatically by github when the release is done. I don't think I can filter it. Why is it a trouble to have those files?

@apoleon
Copy link
Author

apoleon commented Jun 25, 2017 via email

@apoleon
Copy link
Author

apoleon commented Jun 25, 2017

Prebuilt jar files should never be part of any source release. Debian requires source code for everything that should be shipped in its main distribution. A jar file with class files is obviously not source. We either need the corresponding *.java files or I have to repack the tarball and remove it. Our upstream guide sums it up:

https://wiki.debian.org/UpstreamGuide

I'm quite sure there is a way in Github to exclude files before they are included in the release tarballs. Jars just increase the size of the tarball and Linux distributions will always build Objenesis with the packaged Maven version anyway. I had no problems with building the latest release without the maven-wrapper.jar.

@henri-tremblay
Copy link
Contributor

In general it works. But nothing is guaranteed. I get what Debian needs. But it means to be less nice for everyone else. I'm looking for exclusions on the github git archive. Is it complicated for you to ignore?

@henri-tremblay
Copy link
Contributor

Ok. I should have fixed it. Can you do a git archive from HEAD (or a tag that I create) or I need to do an actual release?

@apoleon
Copy link
Author

apoleon commented Jun 26, 2017 via email

@henri-tremblay
Copy link
Contributor

Excellent. Should be the case

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants